Project

Profile

Help

Task #4720

Remove publishers from pulp_file

Added by daviddavis about 1 year ago. Updated 6 months ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Sprint/Milestone:
Start date:
Due date:
% 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 manifest[0]. 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 CLOSED - CURRENTRELEASE Actions

History

#1 Updated by daviddavis about 1 year ago

  • Description updated (diff)

#2 Updated by bmbouter about 1 year ago

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

#3 Updated by amacdona@redhat.com about 1 year ago

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

#4 Updated by bmbouter about 1 year ago

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

+1 to this story.

#5 Updated by daviddavis about 1 year 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 year ago

  • Sprint set to Sprint 53

#7 Updated by daviddavis about 1 year ago

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

#8 Updated by daviddavis about 1 year 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 year ago

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

#11 Updated by kersom 12 months ago

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

#12 Updated by bmbouter 6 months ago

  • Sprint/Milestone set to 0.1.0

#13 Updated by bmbouter 6 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF