Project

Profile

Help

Task #608

Pulp uses '__/primary/__' string to indicate primary content source

Added by cduryee almost 6 years ago. Updated 6 months ago.

Status:
CLOSED - WONTFIX
Priority:
Normal
Assignee:
-
Category:
-
Sprint/Milestone:
-
Start date:
Due date:
% Done:

0%

Estimated time:
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:
Quarter:

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 +

History

#1 Updated by bmbouter over 1 year ago

  • Status changed from NEW to CLOSED - WONTFIX

#2 Updated by bmbouter over 1 year 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.

#3 Updated by bmbouter over 1 year ago

  • Tags Pulp 2 added

#4 Updated by bmbouter 6 months ago

  • Category deleted (14)

We are removing the 'API' category per open floor discussion June 16, 2020.

Please register to edit this issue

Also available in: Atom PDF