Project

Profile

Help

Task #4720

Remove publishers from pulp_file

Added by daviddavis over 1 year ago. Updated 9 months ago.

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

100%

Estimated time:
Platform Release:
Groomed:
Yes
Sprint Candidate:
Yes
Tags:
Sprint:
Sprint 53
Quarter:

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<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>
Related to Pulp - Test #4418: Publish lazy synced reposCLOSED - COMPLETE<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>
Blocked by Pulp - Story #4678: As a plugin writer, I have Master/Detail PublicationsCLOSED - CURRENTRELEASE

<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

History

#1 Updated by daviddavis over 1 year ago

  • Description updated (diff)

#2 Updated by bmbouter over 1 year ago

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

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

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

#4 Updated by bmbouter over 1 year ago

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

+1 to this story.

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

  • Sprint set to Sprint 53

#7 Updated by daviddavis over 1 year ago

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

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

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

#11 Updated by kersom over 1 year ago

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

#12 Updated by bmbouter 9 months ago

  • Sprint/Milestone set to 0.1.0

#13 Updated by bmbouter 9 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF