Bug 1167456 - --checksum-type does not affect the checksum used in primary.xml
Summary: --checksum-type does not affect the checksum used in primary.xml
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: Pulp
Classification: Retired
Component: rpm-support
Version: 2.4.0
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
: ---
Assignee: pulp-bugs
QA Contact: pulp-qe-list
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2014-11-24 20:20 UTC by Randy Barlow
Modified: 2015-02-28 22:45 UTC (History)
1 user (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2015-02-28 22:45:31 UTC
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Pulp Redmine 627 0 None None None Never

Description Randy Barlow 2014-11-24 20:20:27 UTC
Description of problem:
Setting the --checksum-type on a repo does not seem to have an effect.

Version-Release number of selected component (if applicable):
2.4.3-1

How reproducible:
Every time

Steps to Reproduce:
1. Create a repo, perhaps the zoo repo, setting --checksum-type sha1
2. Sync & Publish

Actual results:
Open the primary.xml.gz file in the published repodata folder. Observe that sha256 was used. For example, I see this tag in the bear package:

<checksum pkgid="YES" type="sha256">7a831f9f90bf4d21027572cb503d20b702de8e8785b02c0397445c2e481d81b3</checksum>

Expected results:
I expected sha1 to have been used, since I explicitly chose sha1.

Additional info:
I wonder if this is because I sync'd the repo, and perhaps the upstream zoo repo used sha256 and that overrides the --checksum-type flag? I think it's a bug either way, since the --checksum-type flag helptext doesn't list any conditions on whether it gets used.

Comment 1 Randy Barlow 2014-11-25 01:15:22 UTC
I tried creating repos without feeds and uploading packages to them with the --checksum-type set to "sha", but the primary.xml still shows sha256 as the checksum type used.

Comment 2 Brian Bouterse 2014-12-01 16:59:37 UTC
This behavior should be documented in the help-text and docs. It should note that checksum types can be set in multiple places. ie: upload, repo create, etc.

Comment 3 Brian Bouterse 2015-02-28 22:45:31 UTC
Moved to https://pulp.plan.io/issues/627


Note You need to log in before you can comment on or make changes to this bug.