Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes #36302 - Add option to skip dep solving during inc update in the UI #10522

Merged
merged 1 commit into from Apr 19, 2023

Conversation

ianballou
Copy link
Member

What are the changes introduced in this pull request?

Adds a checkbox to skip dependency solving during incremental update in the UI.

Considerations taken when implementing this change?

This was determined to be the best way to speed up incremental update for users.

To the reviewer: how does the wording sound?

What are the testing steps for this pull request?

  1. Subscribe a host to a content view version with errata filtered out.
  2. Apply some errata that were filtered out. First, skip dependency solving with the new checkbox.
  3. See that dependency solving false is sent to Pulp in the MultiCopyContents Dynflow.
  4. Apply the errata again (after promoting the original CVV to Library) using dependency solving this time.
  5. Check that dependency solving is true this time.

@theforeman-bot
Copy link

Issues: #36302

Copy link
Member

@sjha4 sjha4 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Screenshot from 2023-04-19 12-46-45
Screenshot from 2023-04-19 12-46-25

Able to control dep solving through the new checkbox. Ack 👍🏼

@ianballou ianballou merged commit ee56983 into Katello:master Apr 19, 2023
5 checks passed
@ianballou ianballou deleted the 36302-inc-update-box branch April 19, 2023 18:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants