Refactor #8635
closedDeprecate ContentUnassociation stage
100%
Description
ContentUnassociation doesn't really need to be a separate stage, and making it one breaks the expectations of the "pipeline" by changing the semantics of what goes into and out of the queues.
The ContentAssociation stage puts querysets into the queue (instead of declarative content) for the sake of the ContentUnassociation stage, and also these querysets represent content to remove rather than content to add.
Furthermore this isn't necessary. Rather than toggling whether the ContentUnassociation stage is run as a whole, we can just pass a boolean parameter into the ContentAssociation stage to toggle the behavior.
Related issues
Added by dalley over 3 years ago
Updated by dalley over 3 years ago
- Status changed from POST to MODIFIED
- % Done changed from 0 to 100
Applied in changeset pulpcore|0ae327ab5de95020b97f9438f090849760ef8272.
Updated by dalley over 3 years ago
- Status changed from MODIFIED to NEW
Reverting to move to 3.14 after a deprecation cycle
Updated by pulpbot over 3 years ago
- Status changed from NEW to POST
Updated by pulpbot over 3 years ago
Added by dalley over 3 years ago
Revision 1db09595 | View on GitHub
Revert "Merge ContentUnassociation stage into the ContentAssociation stage"
Reverts commits: ec7afd2f5e34acacf40c1b59637438339dc56584 0ae327ab5de95020b97f9438f090849760ef8272
Added by dalley over 3 years ago
Revision 776f530e | View on GitHub
Deprecate ContentUnassociation stage
Updated by dalley over 3 years ago
- Subject changed from Merge ContentUnassociation stage into ContentAssociation stage to Deprecate ContentUnassociation stage
- Status changed from ASSIGNED to MODIFIED
Updated by pulpbot over 3 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Updated by dalley over 3 years ago
- Copied to Refactor #8827: Remove ContentUnassociation stage added
Merge ContentUnassociation stage into the ContentAssociation stage
ContentUnassociation doesn't really need to be a separate stage, and making it one breaks the expectations of the "pipeline" by changing the semantics of what goes into and out of the queues.
closes: #8635 https://pulp.plan.io/issues/8635