Bug 1160410

Summary:	Pulp uses '__/primary/__' string to indicate primary content source
Product:	[Retired] Pulp	Reporter:	Chris Duryee <cduryee>
Component:	API/integration	Assignee:	pulp-bugs
Status:	CLOSED UPSTREAM	QA Contact:	pulp-qe-list
Severity:	low	Docs Contact:
Priority:	medium
Version:	2.5	CC:	jortel
Target Milestone:	---	Keywords:	Task, Triaged
Target Release:	3.0.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2015-02-28 22:43:20 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 Chris Duryee 2014-11-04 18:04:44 UTC

Description of problem:

Currently, pulp's ContentSource class has an id field that represents source ID. In cases when the ContentSource is the primary content source, this is set to "__/primary/__".

Instead, this could just be set to "None". This would let us remove the PRIMARY_ID constant in pulp/common/constants.py.

This change is an API change so I propose that it is done with Pulp 3.0.

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

Note that there is also https://bugzilla.redhat.com/show_bug.cgi?id=1135589 which moves the PRIMARY_ID definition to avoid DB initialization. The patch for that bug is more of a workaround; this BZ is to get rid of the constant entirely.

Comment 1 Brian Bouterse 2015-02-28 22:43:20 UTC

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