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

Convert bookmarks upgrade scenarios to new format #16856

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

synkd
Copy link
Contributor

@synkd synkd commented Nov 4, 2024

As part of the upgrade scenarios conversion effort, this PR changes the bookmarks upgrade scenario tests to use the shared resource framework and converts the pre-upgrade scenarios to pytest fixtures. It also adds a new shared Satellite fixture for search component tests.

@synkd synkd added CherryPick PR needs CherryPick to previous branches 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 6.16.z Introduced in or relating directly to Satellite 6.16 labels Nov 4, 2024
@synkd
Copy link
Contributor Author

synkd commented Nov 4, 2024

trigger: test-robottelo
pytest: tests/new_upgrades/test_bookmarks.py
env:
BROKER_SSH_BACKEND: hussh

@synkd
Copy link
Contributor Author

synkd commented Nov 5, 2024

trigger: test-robottelo
pytest: tests/new_upgrades/test_bookmarks.py
env:
BROKER_SSH_BACKEND: hussh

1 similar comment
@synkd
Copy link
Contributor Author

synkd commented Nov 5, 2024

trigger: test-robottelo
pytest: tests/new_upgrades/test_bookmarks.py
env:
BROKER_SSH_BACKEND: hussh

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9254
Build Status: SUCCESS
PRT Comment: pytest tests/new_upgrades/test_bookmarks.py --external-logging
Test Result : ================= 2 passed, 134 warnings in 4070.81s (1:07:50) =================

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Nov 5, 2024
As part of the upgrade scenarios conversion effort, this PR changes the
bookmarks upgrade scenario tests to use the shared resource framework
and converts the pre-upgrade scenarios to pytest fixtures. It also adds
a new shared Satellite fixture for search component tests.
@synkd synkd force-pushed the convert_bookmarks_scenario_to_new_format branch from 1771cbf to 6ec3c3e Compare November 5, 2024 20:25
@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Nov 5, 2024
Copy link
Member

@JacobCallahan JacobCallahan left a comment

Choose a reason for hiding this comment

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

ACK, nice!

Copy link
Contributor

@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.

Looks neat and great!

The only thing - would you mind to update the CODEOWNERS file with the new additions? We use it for the team-based review requests / heads up. Thank you!

tests/new_upgrades/test_bookmarks.py Show resolved Hide resolved
@synkd synkd requested a review from a team as a code owner November 13, 2024 15:02
@synkd
Copy link
Contributor Author

synkd commented Nov 13, 2024

@vsedmik Just updated with the change to the CODEOWNERS file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 6.16.z Introduced in or relating directly to Satellite 6.16 CherryPick PR needs CherryPick to previous branches
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants