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

Add test for baremetal secureboot provisioning #15735

Merged
merged 2 commits into from
Nov 5, 2024
Merged

Add test for baremetal secureboot provisioning #15735

merged 2 commits into from
Nov 5, 2024

Conversation

Gauravtalreja1
Copy link
Collaborator

Problem Statement

Missing test for baremetal secureboot provisioning

Solution

Add test for baremetal secureboot provisioning

@Gauravtalreja1 Gauravtalreja1 added No-CherryPick PR doesnt need CherryPick to previous branches Stream Introduced in or relating directly to Satellite Stream/Master labels Jul 23, 2024
@Gauravtalreja1 Gauravtalreja1 self-assigned this Jul 23, 2024
@Gauravtalreja1 Gauravtalreja1 requested review from a team as code owners July 23, 2024 14:25
@Gauravtalreja1 Gauravtalreja1 marked this pull request as draft July 23, 2024 14:26
@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Jul 23, 2024
@Gauravtalreja1 Gauravtalreja1 force-pushed the bm-sb-provisioning branch 2 times, most recently from 2d9eef6 to 3278d8a Compare August 2, 2024 15:32
@JacobCallahan
Copy link
Member

@Gauravtalreja1 can you add that sleep to this PR, then we can test it in PRT.

@Gauravtalreja1 Gauravtalreja1 force-pushed the bm-sb-provisioning branch 2 times, most recently from 23f5866 to bef652b Compare August 9, 2024 19:52
@SatelliteQE SatelliteQE deleted a comment from Satellite-QE Aug 9, 2024
@SatelliteQE SatelliteQE deleted a comment from Satellite-QE Aug 9, 2024
@SatelliteQE SatelliteQE deleted a comment from Satellite-QE Aug 9, 2024
@SatelliteQE SatelliteQE deleted a comment from Satellite-QE Aug 9, 2024
@SatelliteQE SatelliteQE deleted a comment from Satellite-QE Aug 9, 2024
@SatelliteQE SatelliteQE deleted a comment from Satellite-QE Aug 9, 2024
@SatelliteQE SatelliteQE deleted a comment from Satellite-QE Aug 9, 2024
@SatelliteQE SatelliteQE deleted a comment from Satellite-QE Aug 9, 2024
@Gauravtalreja1 Gauravtalreja1 force-pushed the bm-sb-provisioning branch 2 times, most recently from 0255517 to dac5106 Compare September 19, 2024 23:19
@Gauravtalreja1 Gauravtalreja1 force-pushed the bm-sb-provisioning branch 2 times, most recently from 3ca4237 to ce895ec Compare October 22, 2024 09:41
@SatelliteQE SatelliteQE deleted a comment from Satellite-QE Oct 22, 2024
@SatelliteQE SatelliteQE deleted a comment from Satellite-QE Oct 22, 2024
@Gauravtalreja1 Gauravtalreja1 marked this pull request as ready for review October 22, 2024 22:32
@Gauravtalreja1 Gauravtalreja1 force-pushed the bm-sb-provisioning branch 2 times, most recently from e3fbc96 to 4373c00 Compare October 23, 2024 09:13
@Gauravtalreja1 Gauravtalreja1 added the 6.16.z Introduced in or relating directly to Satellite 6.16 label Oct 23, 2024
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9115
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_provisioning.py::test_rhel_pxe_provisioning_secureboot_enabled --external-logging --include-onprem-provisioning
Test Result : ============= 1 passed, 13 warnings, 1 error in 2398.19s (0:39:58) =============

@Gauravtalreja1
Copy link
Collaborator Author 

trigger: test-robottelo
pytest: tests/foreman/api/test_provisioning.py::test_rhel_pxe_provisioning_secureboot_enabled
provisioning: true
env:
     BROKER_SSH_BACKEND: hussh

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9122
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_provisioning.py::test_rhel_pxe_provisioning_secureboot_enabled --external-logging --include-onprem-provisioning
Test Result : ================= 1 failed, 13 warnings in 2969.40s (0:49:29) ==================

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9123
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_provisioning.py::test_rhel_pxe_provisioning_secureboot_enabled --external-logging --include-onprem-provisioning
Test Result : ================= 1 passed, 13 warnings in 2748.38s (0:45:48) ==================

@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 Oct 23, 2024
@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Oct 24, 2024
@Gauravtalreja1 Gauravtalreja1 added the PRT-Passed Indicates that latest PRT run is passed for the PR label Oct 24, 2024
@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Nov 5, 2024
@Gauravtalreja1 Gauravtalreja1 added the PRT-Passed Indicates that latest PRT run is passed for the PR label Nov 5, 2024
shweta83
shweta83 approved these changes Nov 5, 2024
View reviewed changes
Copy link
Contributor

@shweta83 shweta83 left a comment

Choose a reason for hiding this comment

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

Ack

@shweta83 shweta83 merged commit 3430887 into SatelliteQE:master Nov 5, 2024
10 checks passed
github-actions bot pushed a commit that referenced this pull request Nov 5, 2024
@Gauravtalreja1 @web-flow
Add test for baremetal secureboot provisioning (#15735)
df5f15e 
* Add test for baremetal secureboot provisioning

Signed-off-by: Gaurav Talreja <[email protected]>

* Update provisioning_vmware_host fixture to use Broker contextmanager

Signed-off-by: Gaurav Talreja <[email protected]>

---------

Signed-off-by: Gaurav Talreja <[email protected]>
(cherry picked from commit 3430887)
This was referenced Nov 5, 2024
Merged
Open
@Gauravtalreja1 Gauravtalreja1 deleted the bm-sb-provisioning branch November 5, 2024 13:16
damoore044 pushed a commit to damoore044/robottelo that referenced this pull request Nov 5, 2024
@Gauravtalreja1 @damoore044
Add test for baremetal secureboot provisioning (SatelliteQE#15735)
7d288df 
* Add test for baremetal secureboot provisioning

Signed-off-by: Gaurav Talreja <[email protected]>

* Update provisioning_vmware_host fixture to use Broker contextmanager

Signed-off-by: Gaurav Talreja <[email protected]>

---------

Signed-off-by: Gaurav Talreja <[email protected]>
damoore044 pushed a commit to damoore044/robottelo that referenced this pull request Nov 5, 2024
@Gauravtalreja1 @damoore044
Add test for baremetal secureboot provisioning (SatelliteQE#15735)
39eaa31 
* Add test for baremetal secureboot provisioning

Signed-off-by: Gaurav Talreja <[email protected]>

* Update provisioning_vmware_host fixture to use Broker contextmanager

Signed-off-by: Gaurav Talreja <[email protected]>

---------

Signed-off-by: Gaurav Talreja <[email protected]>
damoore044 pushed a commit to damoore044/robottelo that referenced this pull request Nov 8, 2024
@Gauravtalreja1 @damoore044
Add test for baremetal secureboot provisioning (SatelliteQE#15735)
13ba089 
* Add test for baremetal secureboot provisioning

Signed-off-by: Gaurav Talreja <[email protected]>

* Update provisioning_vmware_host fixture to use Broker contextmanager

Signed-off-by: Gaurav Talreja <[email protected]>

---------

Signed-off-by: Gaurav Talreja <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@amolpati30 amolpati30 amolpati30 approved these changes

@shubhamsg199 shubhamsg199 shubhamsg199 approved these changes

@shweta83 shweta83 shweta83 approved these changes

Assignees

@Gauravtalreja1 Gauravtalreja1

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 QETestCoverage Issues and PRs relating to a Satellite bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@Gauravtalreja1 @JacobCallahan @Satellite-QE @shweta83 @shubhamsg199 @amolpati30