Project

Profile

Help

Task #4720

Remove publishers from pulp_file

Added by daviddavis 5 months ago. Updated 4 months ago.

Status:
MODIFIED
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 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 5 months ago

  • Description updated (diff)

#2 Updated by bmbouter 5 months ago

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

#3 Updated by amacdona@redhat.com 5 months ago

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

#4 Updated by bmbouter 5 months ago

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

+1 to this story.

#5 Updated by daviddavis 5 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 4 months ago

  • Sprint set to Sprint 53

#7 Updated by daviddavis 4 months ago

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

#8 Updated by daviddavis 4 months ago

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

#10 Updated by daviddavis 4 months ago

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

#11 Updated by kersom 3 months ago

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

Please register to edit this issue

Also available in: Atom PDF