Project

Profile

Help

Task #4720

closed

Remove publishers from pulp_file

Added by daviddavis about 5 years ago. Updated over 4 years 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)CLOSED - WONTFIXActions
Related to Pulp - Test #4418: Publish lazy synced reposCLOSED - COMPLETEkersomActions
Blocked by Pulp - Story #4678: As a plugin writer, I have Master/Detail PublicationsCLOSED - CURRENTRELEASEdaviddavis

Actions
Actions #1

Updated by daviddavis about 5 years ago

  • Description updated (diff)
Actions #2

Updated by bmbouter about 5 years ago

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

Updated by amacdona@redhat.com about 5 years ago

  • Tracker changed from Issue to Task
  • % Done set to 0
Actions #4

Updated by bmbouter about 5 years ago

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

+1 to this story.

Actions #5

Updated by daviddavis about 5 years 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).

Actions #6

Updated by CodeHeeler about 5 years ago

  • Sprint set to Sprint 53
Actions #7

Updated by daviddavis about 5 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to daviddavis
Actions #8

Updated by daviddavis about 5 years ago

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

Added by daviddavis about 5 years ago

Revision 849810ea | View on GitHub

Remove file publisher stuff from tests

ref #4720 https://pulp.plan.io/issues/4720

Actions #10

Updated by daviddavis about 5 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100
Actions #11

Updated by kersom almost 5 years ago

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

Updated by bmbouter over 4 years ago

  • Sprint/Milestone set to 0.1.0
Actions #13

Updated by bmbouter over 4 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF