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.16.z] CP: rhel_ver_match(N-x) Optional Format #17090

Merged
merged 1 commit into from
Dec 6, 2024

Conversation

damoore044
Copy link
Contributor

@damoore044 damoore044 commented Dec 5, 2024

Problem Statement

Partial cherry-pick of #16807 , adding ('N-x') format to parametrize marker rhel_ver_match for rhel contenthosts/clients.
Just CPing the fixture changes, as some new tests are also being CP'd to 6.16/6.15.z, with the N-x marker format.
Otherwise, the format is invalid for regex, so all distros are picked including fips.

Note: In 6.15.z and 6.16.z , RHEL9 is N (newest) in supportability.yaml. RHEL10 is only in Stream (6.17)

Example:

@pytest.mark.rhel_ver_match('N-2') # rhel9, 8, and 7  in 6.16 and 6.15
def test_case(rhel_contenthost, target_sat, ...):

PRT Case

trigger: test-robottelo
pytest: tests/foreman/cli/test_rhcloud_inventory.py -k 'test_positive_register_insights_client_host'

or tests/foreman/api/test_rhcloud_inventory.py -k 'test_rhcloud_scheduled_insights_sync'

@damoore044 damoore044 added CherryPick PR needs CherryPick to previous branches 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 Dec 5, 2024
@damoore044 damoore044 self-assigned this Dec 5, 2024
@damoore044 damoore044 requested a review from a team as a code owner December 5, 2024 16:25
@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_rhcloud_inventory.py -k 'test_rhcloud_scheduled_insights_sync'

@damoore044 damoore044 requested a review from a team December 5, 2024 16:31
@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_rhcloud_inventory.py -k 'test_positive_register_insights_client_host'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9547
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/cli/test_rhcloud_inventory.py -k test_positive_register_insights_client_host --external-logging
Test Result : ========= 3 passed, 28 deselected, 126 warnings in 2045.90s (0:34:05) ==========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Dec 5, 2024
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9546
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_rhcloud_inventory.py -k test_rhcloud_scheduled_insights_sync --external-logging
Test Result : ========= 6 passed, 20 deselected, 795 warnings in 4909.79s (1:21:49) ==========

@damoore044 damoore044 added the AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing label Dec 5, 2024
@jyejare jyejare merged commit 42edc55 into SatelliteQE:6.16.z Dec 6, 2024
19 checks passed
github-actions bot pushed a commit that referenced this pull request Dec 6, 2024
CP rhel_ver_match(N-x) optional format

(cherry picked from commit 42edc55)
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 6.16.z Introduced in or relating directly to Satellite 6.16 AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing CherryPick PR needs 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.

4 participants