Project

General

Profile

Actions

Bug #28039

closed

Content View publish with dep solve and no filters adds recursive flag

Added by Ian Ballou over 4 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Content Views
Target version:
Difficulty:
Triaged:
Yes
Fixed in Releases:
Found in Releases:

Description

Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1750811

Description of problem:
During a content view publish, Katello directs Pulp to copy contents between 2 repos.
Now if the "solve dependencies" is set on the content view, Katello automatically sets "recursive_conservative: true" when that copy operation is executed.
This recursive_conservative flag is set even if no filters were specified on the content view. Meaning it could potentially end up dep solving in all rpms in a repo instead of just a subset. This could unnecessarily slow down the publish times (since depsolve takes a much longer time).

Version-Release number of selected component (if applicable):
Nightly

How reproducible:
100%

Steps to Reproduce:
1. Have a content view with solve dependencies set to true and no filters
2. Add a large repo to the cv
3. Publish

Actual results:
Publish that takes much longer even though there are no filters.

Expected results:
Publish that takes about the same time as publish without dep solving

Actions #1

Updated by The Foreman Bot over 4 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Ian Ballou
  • Pull request https://github.com/Katello/katello/pull/8382 added
Actions #2

Updated by The Foreman Bot over 4 years ago

  • Fixed in Releases Katello 3.14.0 added
Actions #3

Updated by Anonymous over 4 years ago

  • Status changed from Ready For Testing to Closed
Actions #4

Updated by Samir Jha over 4 years ago

  • Subject changed from Content View publish with dep solve and no filters adds recursive flag to Content View publish with dep solve and no filters adds recursive flag
  • Target version set to Katello 3.14.0
  • Triaged changed from No to Yes
Actions

Also available in: Atom PDF