Actions
Bug #38460
closedComposite content views can update twice due to a single incremental update of a child content view
Status:
Closed
Priority:
Normal
Assignee:
Category:
Content Views
Target version:
Difficulty:
Triaged:
Yes
Description
When a composite content view is both set to auto-publish and contains a content view set to 'Always update to latest version', it will update twice when an incremental update is applied on the child content view and the --propagate-all-composites flag is set.
- Create and publish a content view with missing errata.
- Add that content view to a composite content view and set the CV to 'Always update to latest version'. Set the CCV to auto-publish.
- Run an incremental update on the CV with the
--propagate-all-compositesflag set:hammer content-view version incremental-update --content-view-version-id <cvv-id> --errata-ids <erratum-id> --propagate-all-composites true - Look at the new versions of the CCV. There will be a new minor version tracking the incremental update (X.1) and a new major version created in response to the new CV minor version (X+1).
Expected output
There are a few options for fixing this bug. Either
- CCVs should not auto-publish on child CV minor version updates when the
--propagate-all-compositesflag is set, --propagate-all-compositesshould not have an effect on CCVs set to auto-publish, or- auto-publishing CCVs should abort new publications when the new version's contents match the old version.
Option #3's approach is probably the cleanest but is also the most expensive. Any approach has lots of edge cases we need to worry about, unfortunately.
Updated by Pavan Soma Shekar 10 months ago
- Target version changed from Katello 4.18.0 to Katello 4.20.0
- Triaged changed from Yes to No
Updated by The Foreman Bot 5 months ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/Katello/katello/pull/11629 added
Updated by Quinn James 5 months ago
- Status changed from Ready For Testing to Closed
Applied in changeset katello|4723d11af7b48754af9bb6286527979bdb645e4d.
Updated by The Foreman Bot 5 months ago
- Assignee set to Zach Huntington-Meath
- Pull request https://github.com/theforeman/forklift/pull/1932 added
Updated by Quinn James 4 months ago
- Pull request deleted (
https://github.com/theforeman/forklift/pull/1932)
Updated by The Foreman Bot 4 months ago
- Pull request https://github.com/theforeman/forklift/pull/1939 added
Actions