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.12.z] Audit - Adding coverage for Discovery provisioning via CLI #12827

Closed
wants to merge 1 commit into from
Closed

[6.12.z] Audit - Adding coverage for Discovery provisioning via CLI #12827

wants to merge 1 commit into from

Conversation

Satellite-QE
Copy link
Collaborator

Cherrypick of PR: #12467

PR aims to add coverage for Pxe and Pxeless Discovery host provisioning via hammer(CLI).

@Satellite-QE Satellite-QE added 6.12.z Introduced in or relating directly to Satellite 6.12 Auto_Cherry_Picked Automatically cherrypicked PR using GHA No-CherryPick PR doesnt need CherryPick to previous branches labels Oct 6, 2023
@Satellite-QE
Copy link
Collaborator Author 

trigger: test-robottelo
pytest: tests/foreman/cli/test_discoveredhost.py
provisioning: true
env:
    ROBOTTELO_server__deploy_arguments__deploy_sat_version: '6.14.0'
    ROBOTTELO_server__deploy_arguments__deploy_snap_version: '19.0'

@Satellite-QE Satellite-QE added the AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing label Oct 6, 2023
@adarshdubey-star
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/cli/test_discoveredhost.py
provisioning: true

1 similar comment
@adarshdubey-star
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/cli/test_discoveredhost.py
provisioning: true

@Gauravtalreja1 Gauravtalreja1 force-pushed the cherry-pick-6.12.z-272a3aafb06da6bdc093da6b1e2791385d1f7e1d branch from d462716 to aa3d7ff Compare October 7, 2023 13:54
@Gauravtalreja1
Copy link
Collaborator

trigger: test-robottelo
pytest: tests/foreman/cli/test_discoveredhost.py
provisioning: true

1 similar comment
@Gauravtalreja1
Copy link
Collaborator

trigger: test-robottelo
pytest: tests/foreman/cli/test_discoveredhost.py
provisioning: true

@Gauravtalreja1
Copy link
Collaborator

@adarshdubey-star Can you investigate the failing PRT?

@adarshdubey-star
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/cli/test_discoveredhost.py
provisioning: true

1 similar comment
@adarshdubey-star
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/cli/test_discoveredhost.py
provisioning: true

@Gauravtalreja1
Copy link
Collaborator

@adarshdubey-star any update on the failing PRT?

@adarshdubey-star
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/cli/test_discoveredhost.py
provisioning: true

@adarshdubey-star @jyejare
Audit - Adding coverage for Discovery provisioning via CLI (#12467)
d1a4b37 
* Adding coverage for Discovery provisioning via CLI

Signed-off-by: Adarsh Dubey <[email protected]>

* Adding finalizer

---------

Signed-off-by: Adarsh Dubey <[email protected]>
(cherry picked from commit 272a3aa)
@jyejare jyejare force-pushed the cherry-pick-6.12.z-272a3aafb06da6bdc093da6b1e2791385d1f7e1d branch from aa3d7ff to d1a4b37 Compare January 2, 2024 08:31
@jyejare
Copy link
Member

jyejare commented Jan 2, 2024

trigger: test-robottelo
pytest: tests/foreman/cli/test_discoveredhost.py
provisioning: true

@adarshdubey-star
Copy link
Contributor

Closing this one, we don't need cherry pick this far

@jyejare jyejare deleted the cherry-pick-6.12.z-272a3aafb06da6bdc093da6b1e2791385d1f7e1d branch January 4, 2024 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@adarshdubey-star adarshdubey-star

@Satellite-QE Satellite-QE

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

Successfully merging this pull request may close these issues.
4 participants
@Satellite-QE @adarshdubey-star @Gauravtalreja1 @jyejare