Bug 1168029

Summary:	Nodes does not use checksum_type when generating a publishing path
Product:	[Retired] Pulp	Reporter:	Randy Barlow <rbarlow>
Component:	nodes	Assignee:	pulp-bugs
Status:	CLOSED UPSTREAM	QA Contact:	pulp-qe-list
Severity:	medium	Docs Contact:
Priority:	medium
Version:	2.4.0	CC:	bmbouter, fdewaley
Target Milestone:	---	Keywords:	Triaged
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2015-02-28 22:45:44 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Randy Barlow 2014-11-25 22:10:38 UTC

Description of problem:
Nodes uses a checksum to make the publishing path for units, but the path does not also include the checksum type. This was the source of #1165355[0], though there was another way to fix that particular bug that had additional benefits that fixing this bug would not have. We should still add the checksum type to the publishing path of Nodes for correctness.

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

How reproducible:
Every time

Steps to Reproduce:
1. Publish a nodes repository, perhaps one that has the "bear" RPM.
2. Look at the path to the published bear RPM.

Actual results:
/var/lib/pulp/nodes/published/https/repos/confused-bear/content/rpm/bear/4.1/1/noarch/7cc8894d84696bfac328a0f7104daec7cbb0f5c4/bear-4.1-1.noarch.rpm

Expected results:
/var/lib/pulp/nodes/published/https/repos/confused-bear/content/rpm/bear/4.1/1/noarch/sha1-7cc8894d84696bfac328a0f7104daec7cbb0f5c4/bear-4.1-1.noarch.rpm, or similar (note the addition of checksum type).

Additional info:
[0] https://bugzilla.redhat.com/show_bug.cgi?id=1165355

Comment 1 Brian Bouterse 2014-12-01 17:10:56 UTC

It doesn't have to be that we add the checksum type to the path. The best thing to use would be the UUID of the database unit.

Comment 2 Brian Bouterse 2015-02-28 22:45:44 UTC

Moved to https://pulp.plan.io/issues/629