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.

Sign up for GitHub

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

Jump to bottom

[6.13.z] Fix OEL Convert2Rhel tests #14200

Merged
merged 1 commit into from
Feb 29, 2024
Merged

[6.13.z] Fix OEL Convert2Rhel tests #14200

merged 1 commit into from
Feb 29, 2024

Conversation

Satellite-QE
Copy link
Collaborator

Cherrypick of PR: #14194

(cherry picked from commit 9f42080)
Manual cherrypick of #13236

Problem Statement

OEL Convert2Rhel tests failing

Solution

Use UBI repos required for subman packages, while registration instead of syncing.
Fix OEL tests by adding and modifying remediations for the inhibitors.
Replacing existing register_host() method with API method to generate register command - Fixes #11876

@Satellite-QE Satellite-QE added 6.13.z Introduced in or relating directly to Satellite 6.13 Auto_Cherry_Picked Automatically cherrypicked PR using GHA No-CherryPick PR doesnt need CherryPick to previous branches labels Feb 27, 2024
@Satellite-QE
Copy link
Collaborator Author

trigger: test-robottelo
pytest: tests/foreman/api/test_convert2rhel.py
pod_resources_size: extra_large

@Satellite-QE Satellite-QE added the AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing label Feb 27, 2024
@Gauravtalreja1 Gauravtalreja1 changed the title [6.13.z] [6.15.z] Fix OEL Convert2Rhel tests [6.13.z] Fix OEL Convert2Rhel tests Feb 27, 2024
@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 5866
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_convert2rhel.py
Test Result : ================= 4 failed, 565 warnings in 4287.02s (1:11:27) =================

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Feb 27, 2024
@Gauravtalreja1
Copy link
Collaborator

trigger: test-robottelo
pytest: tests/foreman/api/test_convert2rhel.py
pod_resources_size: extra_large

@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 5867
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_convert2rhel.py
Test Result : ================= 4 failed, 566 warnings in 4330.72s (1:12:10) =================

@shubhamsg199 shubhamsg199 force-pushed the cherry-pick-6.13.z-3a9e8a53f153162e3de5745b1851d9a08a72c8e8 branch from 2fc4f23 to d484602 Compare February 29, 2024 07:37
@Gauravtalreja1 @shubhamsg199
[6.15.z] Fix OEL Convert2Rhel tests (#14194)
b361514 
Fix OEL Convert2Rhel tests

Signed-off-by: Gaurav Talreja <[email protected]>
(cherry picked from commit 9f42080)
(cherry picked from commit 3a9e8a5)
@shubhamsg199 shubhamsg199 force-pushed the cherry-pick-6.13.z-3a9e8a53f153162e3de5745b1851d9a08a72c8e8 branch from d484602 to b361514 Compare February 29, 2024 07:38
@shubhamsg199
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/api/test_convert2rhel.py
pod_resources_size: extra_large

shubhamsg199
shubhamsg199 reviewed Feb 29, 2024
View reviewed changes
tests/foreman/api/test_convert2rhel.py
@@ -276,7 +287,6 @@ def test_convert2rhel_oracle(module_target_sat, oracle, activation_key_rhel, ver
'inputs': {
'Activation Key': activation_key_rhel.id,
'Restart': 'yes',
'Data telemetry': 'yes',
Copy link
Contributor

Choose a reason for hiding this comment

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

Removing this Data telemetry': 'yes' as it's not for 6.13.z

@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 5897
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_convert2rhel.py --external-logging
Test Result : ============ 3 passed, 769 warnings, 1 error in 6085.64s (1:41:25) =============

@shubhamsg199
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/api/test_convert2rhel.py
pod_resources_size: extra_large

@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 5902
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_convert2rhel.py --external-logging
Test Result : ================= 4 passed, 876 warnings in 6612.30s (1:50:12) =================

@Satellite-QE Satellite-QE added PRT-Passed Indicates that latest PRT run is passed for the PR and removed PRT-Failed Indicates that latest PRT run is failed for the PR labels Feb 29, 2024
@Satellite-QE Satellite-QE merged commit 7203474 into 6.13.z Feb 29, 2024
11 of 12 checks passed
@Satellite-QE Satellite-QE deleted the cherry-pick-6.13.z-3a9e8a53f153162e3de5745b1851d9a08a72c8e8 branch February 29, 2024 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shubhamsg199 shubhamsg199 shubhamsg199 left review comments

Assignees

@Gauravtalreja1 Gauravtalreja1

Labels
6.13.z Introduced in or relating directly to Satellite 6.13 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
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Satellite-QE @Gauravtalreja1 @shubhamsg199