Task #608
closed
Pulp uses '__/primary/__' string to indicate primary content source
0%
Description
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.
+ This bug was cloned from Bugzilla Bug #1160410 +
Updated by bmbouter about 5 years ago
- Status changed from NEW to CLOSED - WONTFIX
Updated by bmbouter about 5 years ago
Pulp 2 is approaching maintenance mode, and this Pulp 2 ticket is not being actively worked on. As such, it is being closed as WONTFIX. Pulp 2 is still accepting contributions though, so if you want to contribute a fix for this ticket, please reopen or comment on it. If you don't have permissions to reopen this ticket, or you want to discuss an issue, please reach out via the developer mailing list.
Updated by bmbouter almost 4 years ago
- Category deleted (
14)
We are removing the 'API' category per open floor discussion June 16, 2020.
.