Task #4720
closed
Remove publishers from pulp_file
100%
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.
Related issues
Updated by bmbouter over 5 years ago
- Blocked by Story #4678: As a plugin writer, I have Master/Detail Publications added
Updated by amacdona@redhat.com over 5 years ago
- Tracker changed from Issue to Task
- % Done set to 0
Updated by bmbouter over 5 years ago
- Groomed changed from No to Yes
- Sprint Candidate changed from No to Yes
+1 to this story.
Updated by daviddavis over 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).
Added by daviddavis over 5 years ago
Updated by daviddavis over 5 years ago
- Status changed from NEW to ASSIGNED
- Assignee set to daviddavis
Updated by daviddavis over 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 over 5 years ago
Revision d4d58283 | View on GitHub
Update tests since file publishers were removed
ref #4720 https://pulp.plan.io/issues/4720
Required PR: https://github.com/pulp/pulp_file/pull/221 Required PR: https://github.com/pulp/pulp-certguard/pull/21
Updated by daviddavis over 5 years ago
- Status changed from ASSIGNED to POST
pulp_file PR: https://github.com/pulp/pulp_file/pull/221
pulpcore PR: https://github.com/pulp/pulpcore/pull/132
pulp-openapi-generator PR: https://github.com/pulp/pulp-openapi-generator/pull/14
pulp-certguard PR: https://github.com/pulp/pulp-certguard/pull/21
Added by daviddavis over 5 years ago
Revision 849810ea | View on GitHub
Remove file publisher stuff from tests
Added by daviddavis over 5 years ago
Revision bffdc106 | View on GitHub
Remove publishers from file plugin
fixes #4720 https://pulp.plan.io/issues/4720
Required PR: https://github.com/pulp/pulpcore/pull/132
Updated by daviddavis over 5 years ago
- Status changed from POST to MODIFIED
- % Done changed from 0 to 100
Applied in changeset bffdc106791034c617b92be22c101c32b15e03ec.
Updated by kersom over 5 years ago
- Related to Test #4418: Publish lazy synced repos added
Updated by bmbouter about 5 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
.
Update tests based on publication changes
ref #4720 https://pulp.plan.io/issues/4720
Required PR: https://github.com/pulp/pulpcore-plugin/pull/93 Required PR: https://github.com/pulp/pulpcore/pull/103/files Required PR: https://github.com/pulp/pulp_file/pull/207