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 #38042 - Pass LCE ID in Refresh Counts action from WebUI #11235

Merged
merged 1 commit into from
Nov 26, 2024

Conversation

pavanshekar
Copy link
Contributor

What are the changes introduced in this pull request?

Added the missing LCE ID to the parameters passed in the Refresh Counts action triggered from the WebUI.

Considerations taken when implementing this change?

Ensured that the LCE ID is correctly included without affecting existing functionality or breaking other API calls.

What are the testing steps for this pull request?

Setup:
1. Satellite with registered external Smart Proxy.
2. Disabled automatic content counts update.

Steps:
1. Create two LCEs, assign them to the Smart Proxy.
2. Create and sync a repo, publish it in two CVs, promote both CVs to both LCEs.
3. Ensure no counts were calculated yet for both CVs, both LCEs.
4. Refresh counts for the first CV in the first LCE, check entity IDs in the Update task.
5. Ensure the counts were updated for the first CV and the second CV stayed untouched, as well as the second LCE.

Copy link

@vsedmik vsedmik left a comment

Choose a reason for hiding this comment

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

LGTM, packit/PRT passed here: SatelliteQE/robottelo#16820 (comment)

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.

Ack.. 👍🏼

@sjha4 sjha4 merged commit 951fb57 into Katello:master Nov 26, 2024
26 of 27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants