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 #36871 - Fix incremental update concurrency #10781

Merged

Conversation

m-bucher
Copy link
Contributor

Do not run Incremental Updates on the same Content View in parallel

@m-bucher m-bucher force-pushed the disable_incr_update_concurrency_in_cv branch from 0a7aeda to 78b2a98 Compare October 27, 2023 13:08
Do not run Incremental Updates on the same Content View in parallel
@m-bucher m-bucher force-pushed the disable_incr_update_concurrency_in_cv branch from 78b2a98 to 776e08c Compare December 5, 2023 10:03
@jeremylenz jeremylenz requested a review from sjha4 February 6, 2024 21:14
@ianballou
Copy link
Member

Tested on Katello 4.7 and seems to be working fine. Tested with normal CVs and composite CVs. Going to test now on the latest code.

Copy link
Member

@ianballou ianballou left a comment

Choose a reason for hiding this comment

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

Working fine for me! Tested again on the latest Katello code. Created a CV with two versions that contain the same repo with different content. The content difference was created via RPM deletion in the UI. Each content view version has a different LCE.

I subscribed one host to each LCE and ensured that each had one applicable erratum to apply. I then ran the incremental update via the UI. The incremental update generate two new content view versions, each had the appropriate content in it. I also confirmed that applying the new content to the host after incremental update still works.

@ianballou ianballou merged commit fb6f707 into Katello:master Feb 15, 2024
6 checks passed
@m-bucher m-bucher deleted the disable_incr_update_concurrency_in_cv branch February 16, 2024 17:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants