Add a case for active_only box on the sync_status page #17413
+75
−0
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
vsedmik
added
CherryPick
|
|
PRT Result |
Satellite-QE
added
the
PRT-Passed
chris1984
reviewed
Jan 22, 2025
| f' and organization_id = {function_sca_manifest_org.id}' | ||
| f' and resource_id = {repo.id}' | ||
| ' and resource_type = Katello::Repository', | ||
| max_tries=12, |
There was a problem hiding this comment.
Is this enough tries with the kickstart repos? I feel like they take about 10 minutes to sync unless my broker vm's are slow.
There was a problem hiding this comment.
The kickstarts usually take less than the normal ones with on_demand policy, this one in particular took ~17 seconds, so we should have ~100 seconds for safety.
vijaysawant
reviewed
Jan 23, 2025
| max_tries=12, | ||
| search_rate=10, | ||
| ) | ||
| session.browser.refresh() |
There was a problem hiding this comment.
Is this refresh neccessary because we are on same page i.e sync_status page and check box Active only doesn't reflect changes on page dynamically?
There was a problem hiding this comment.
Yup it is. It also follows the steps to reproduce from that bug.
vijaysawant
approved these changes
Jan 23, 2025
synkd
approved these changes
Jan 23, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Problem Statement
There is a z-stream bug which needs to be automated, see the issue bellow for details.
TLDR: We need to verify the functionality of the Active only check-box on the Sync Status page.
Solution
This PR adds such UI test case.
It needs some Airgun changes too.
Related Issues
https://issues.redhat.com/browse/SAT-30291
PRT test Cases example