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

Remove cpu_mode option from Libvirt provisioning tests #15883

Merged
merged 1 commit into from
Aug 9, 2024

Conversation

Gauravtalreja1
Copy link
Collaborator

@Gauravtalreja1 Gauravtalreja1 commented Aug 7, 2024

Problem Statement

cpu_mode option is removed in SAT-22491 and upstream PR theforeman/foreman#9963, causing Libvirt provisioning tests to fail

Solution

Remove cpu_mode option from Libvirt provisioning tests

Dependant PRs:

SatelliteQE/airgun#1486

@Gauravtalreja1 Gauravtalreja1 added Easy Fix :) Easiest Fix to review and quick merge request. No-CherryPick PR doesnt need CherryPick to previous branches Stream Introduced in or relating directly to Satellite Stream/Master labels Aug 7, 2024
@Gauravtalreja1 Gauravtalreja1 self-assigned this Aug 7, 2024
@Gauravtalreja1 Gauravtalreja1 requested a review from a team as a code owner August 7, 2024 18:16
@Gauravtalreja1 Gauravtalreja1 removed the Easy Fix :) Easiest Fix to review and quick merge request. label Aug 7, 2024
@Gauravtalreja1
Copy link
Collaborator Author

trigger: test-robottelo
pytest: tests/foreman/{cli,ui}/test_computeresource_libvirt.py -k test_positive_provision_end_to_end
airgun: 1486
provisioning: true
env:
     BROKER_SSH_BACKEND: hussh

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8013
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/{cli,ui}/test_computeresource_libvirt.py -k test_positive_provision_end_to_end --external-logging --include-onprem-provisioning
Test Result : ========== 29 deselected, 66 warnings, 6 errors in 798.57s (0:13:18) ===========

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

trigger: test-robottelo
pytest: tests/foreman/{cli,ui}/test_computeresource_libvirt.py -k test_positive_provision_end_to_end
airgun: 1486
provisioning: true
env:
     BROKER_SSH_BACKEND: hussh

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8052
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/{cli,ui}/test_computeresource_libvirt.py -k test_positive_provision_end_to_end --external-logging --include-onprem-provisioning
Test Result : ==== 2 failed, 4 passed, 29 deselected, 966 warnings in 10867.32s (3:01:07) ====

@Gauravtalreja1
Copy link
Collaborator Author

PRT is expected to fail here for EL9 host due to known issue with SAT-22491

@shubhamsg199 shubhamsg199 merged commit fe6a4b2 into SatelliteQE:master Aug 9, 2024
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No-CherryPick PR doesnt need CherryPick to previous branches PRT-Failed Indicates that latest PRT run is failed for the PR Stream Introduced in or relating directly to Satellite Stream/Master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants