Bug #28189
closedManifest refresh redundantly calls Actions::Pulp::Repository::Refresh for all repos
Description
When refreshing a manifest that updates few subscriptions or repos only (or performs no update at all, even), refreshing triggers updating of pulp importers of all RedHat repos in Default CV and Library. That redundantly slows down manifest refresh.
Version-Release number of selected component (if applicable):
Sat6.5
How reproducible:
100%
Steps to Reproduce:
1. Import a manifest, enable many repos, optionally synchronize them.
2. Refresh the manifest -twice in a row to be sure the 2nd does not update anything.
3. Check dynflow steps of the Manifest Refresh task
Actual results:
3. will show Actions::Pulp::Repository::Refresh steps for each and every RedHat repo enabled
Expected results:
3. to show those steps only for repos that were really affected by the manifest refresh
Additional info:
Particular upstream source code:
triggers all the tasks Katello::Repository::RefreshRepository that is in fact Actions::Pulp::Repository::Refresh (cf. https://github.com/Katello/katello/blob/master/app/lib/actions/katello/repository/refresh_repository.rb).
Updated by The Foreman Bot over 5 years ago
- Assignee set to Partha Aji
- Status changed from New to Ready For Testing
- Pull request https://github.com/Katello/katello/pull/8415 added
Updated by Justin Sherrill over 5 years ago
- Category set to Subscriptions
- Target version set to Katello 3.15.0
- Triaged changed from No to Yes
Updated by The Foreman Bot about 5 years ago
- Fixed in Releases Katello 3.15.0 added
Updated by Partha Aji about 5 years ago
- Status changed from Ready For Testing to Closed
Applied in changeset katello|1237135e38137300fcefa9990402181e6cf6f013.
Updated by Jonathon Turel about 5 years ago
- Target version changed from Katello 3.15.0 to Katello 3.14.0