Project

Profile

Help

Task #4720

Remove publishers from pulp_file

Added by daviddavis 2 months ago. Updated about 1 month ago.

Status:
MODIFIED
Priority:
Normal
Assignee:
Sprint/Milestone:
-
% Done:

100%

Platform Release:
Blocks Release:
Backwards Incompatible:
No
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
QA Contact:
Complexity:
Smash Test:
Verified:
No
Verification Required:
No
Target Release - File:
Sprint:
Sprint 53

Description

Publishers aren't used much in the pulp_file plugin. The only thing they store is a name for manifest0. I don't think most users will use this field so it basically just adds an extra step to most people's workflow. We could make manifest name an option on publish and remove publishers from pulp_file.

[0] https://github.com/pulp/pulp_file/blob/cc7c0bf04bf1b0090490db004b7f14aaf10e971d/pulp_file/app/models.py#L49


Related issues

Related to File Support - Test #4815: Publish a file repo with a manifest filename other than the default (PULP_MANIFEST) NEW Actions
Related to Pulp - Test #4418: Publish lazy synced repos CLOSED - COMPLETE Actions
Blocked by Pulp - Story #4678: As a plugin writer, I have Master/Detail Publications MODIFIED Actions

History

#1 Updated by daviddavis 2 months ago

  • Description updated (diff)

#2 Updated by bmbouter 2 months ago

  • Blocked by Story #4678: As a plugin writer, I have Master/Detail Publications added

#3 Updated by amacdona@redhat.com about 2 months ago

  • Tracker changed from Issue to Task
  • % Done set to 0

#4 Updated by bmbouter about 2 months ago

  • Groomed changed from No to Yes
  • Sprint Candidate changed from No to Yes

+1 to this story.

#5 Updated by daviddavis about 2 months ago

There is one issue that QE raised: if we remove publishers from pulp_file then no plugins will be using publishers and there'll be no way to test them functionally (ie with pulp-smash).

#6 Updated by CodeHeeler about 1 month ago

  • Sprint set to Sprint 53

#7 Updated by daviddavis about 1 month ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to daviddavis

#8 Updated by daviddavis about 1 month ago

  • Related to Test #4815: Publish a file repo with a manifest filename other than the default (PULP_MANIFEST) added

#10 Updated by daviddavis about 1 month ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100

#11 Updated by kersom 6 days ago

  • Related to Test #4418: Publish lazy synced repos added

Please register to edit this issue

Also available in: Atom PDF