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

[6.15.z] [stream] fixes for combined session failures, Errata UI #14929

Conversation

Satellite-QE
Copy link
Collaborator

Cherrypick of PR: #14822

Problem Statement

Initial PR #14773 raised by @LadislavVasina1, for 3 errata UI test fixes/updates.

  • troubleshoot PRT failures in combined session, running the 3 tests from above PR.

Error in setup for test_positive_check_errata_counts_by_type_on_host_details_page :

Locally have intermittently reproduced issue with the registered_contenthost UI fixture with adding repos to cv, and promotion of cv to lce, in between parameters of combined session. Issues with some associated activation keys not being deleted in cleanup.

^^ Issues fixed with 2nd commit.

PRT Case

trigger: test-robottelo
pytest: tests/foreman/ui/test_errata.py -k 'test_positive_check_errata or test_positive_errata_search_type or test_positive_check_errata_counts_by_type_on_host_details_page or test_positive_list and not test_positive_list_permission'
  • We run the 3 impacted tests from @LadislavVasina1 's PR and commits,
    along with test_positive_list, which fails in CI and is fixed by the changes.
    We exclude the otherwise collected test_positive_list_permissions, it is not impacted by these changes.

* Refactor and fix of some Errata UI tests

* UI fixture and other updates for combined sessions

---------

Co-authored-by: Ladislav Vasina <[email protected]>
(cherry picked from commit 9448d4b)
@Satellite-QE Satellite-QE requested a review from a team as a code owner May 1, 2024 16:11
@Satellite-QE Satellite-QE added 6.15.z Introduced in or relating directly to Satellite 6.15 Auto_Cherry_Picked Automatically cherrypicked PR using GHA No-CherryPick PR doesnt need CherryPick to previous branches labels May 1, 2024
@Satellite-QE
Copy link
Collaborator Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_errata.py -k 'test_positive_check_errata or test_positive_errata_search_type or test_positive_check_errata_counts_by_type_on_host_details_page or test_positive_list and not test_positive_list_permission'

@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 6758
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/ui/test_errata.py -k test_positive_check_errata or test_positive_errata_search_type or test_positive_check_errata_counts_by_type_on_host_details_page or test_positive_list and not test_positive_list_permission --external-logging
Test Result : ========= 4 passed, 12 deselected, 171 warnings in 1342.43s (0:22:22) ==========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label May 1, 2024
@damoore044 damoore044 added the AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing label May 1, 2024
@Satellite-QE Satellite-QE merged commit d4c7098 into 6.15.z May 1, 2024
18 checks passed
@Satellite-QE Satellite-QE deleted the cherry-pick-6.15.z-9448d4be9cfbe661103f1ff9382016a581bc5c88 branch May 1, 2024 16:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.15.z Introduced in or relating directly to Satellite 6.15 Auto_Cherry_Picked Automatically cherrypicked PR using GHA AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing No-CherryPick PR doesnt need CherryPick to previous branches PRT-Passed Indicates that latest PRT run is passed for the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants