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

Coverage for autoprovision and reboot of discovered host #12815

Merged
merged 3 commits into from
Oct 11, 2023
Merged

Coverage for autoprovision and reboot of discovered host #12815

merged 3 commits into from
Oct 11, 2023

Conversation

adarshdubey-star
Copy link
Contributor

@adarshdubey-star adarshdubey-star commented Oct 4, 2023 •
edited
Loading

  1. Adding coverage for the auto-provision and reboot API endpoints for a single host.
  2. Adding coverage for the auto-provision of multiple discovered hosts.

@adarshdubey-star adarshdubey-star added API Issues and PRs involving the API QETestCoverage Issues and PRs relating to a Satellite bug CherryPick PR needs CherryPick to previous branches 6.13.z Introduced in or relating directly to Satellite 6.13 6.14.z Introduced in or relating directly to Satellite 6.14 labels Oct 4, 2023
@adarshdubey-star adarshdubey-star self-assigned this Oct 4, 2023
@adarshdubey-star adarshdubey-star requested a review from a team as a code owner October 4, 2023 13:53
@adarshdubey-star adarshdubey-star force-pushed the discovery_4 branch 2 times, most recently from 070bdd8 to 3ef6d9c Compare October 4, 2023 13:57
@adarshdubey-star
Copy link
Contributor Author 

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

@adarshdubey-star
Copy link
Contributor Author 

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

Gauravtalreja1
Gauravtalreja1 reviewed Oct 5, 2023
View reviewed changes
tests/foreman/api/test_discoveredhost.py Outdated Show resolved Hide resolved
tests/foreman/api/test_discoveredhost.py Outdated Show resolved Hide resolved
tests/foreman/api/test_discoveredhost.py Outdated Show resolved Hide resolved
tests/foreman/api/test_discoveredhost.py Outdated Show resolved Hide resolved
tests/foreman/api/test_discoveredhost.py Outdated Show resolved Hide resolved
@adarshdubey-star adarshdubey-star requested a review from a team as a code owner October 10, 2023 07:29
@adarshdubey-star
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_discoveredhost.py -k test_positive_auto_provision_pxe_host

@adarshdubey-star
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_discoveredhost.py -k test_positive_auto_provision_pxe_host

@adarshdubey-star
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_discoveredhost.py -k test_positive_auto_provision_all

shubhamsg199
shubhamsg199 reviewed Oct 10, 2023
View reviewed changes
pytest_fixtures/component/discovery.py Outdated Show resolved Hide resolved
pytest_fixtures/component/discovery.py Outdated Show resolved Hide resolved
Gauravtalreja1
Gauravtalreja1 reviewed Oct 10, 2023
View reviewed changes
pytest_fixtures/component/discovery.py Outdated Show resolved Hide resolved
pytest_fixtures/component/discovery.py Outdated Show resolved Hide resolved
shubhamsg199
shubhamsg199 approved these changes Oct 11, 2023
View reviewed changes
Copy link
Contributor

@shubhamsg199 shubhamsg199 left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@adarshdubey-star
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_discoveredhost.py -k test_positive_auto_provision_all

Gauravtalreja1
Gauravtalreja1 approved these changes Oct 11, 2023
View reviewed changes
Copy link
Collaborator

@Gauravtalreja1 Gauravtalreja1 left a comment

Choose a reason for hiding this comment

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

Ack, waiting for PRT

@Gauravtalreja1 Gauravtalreja1 added 6.12.z Introduced in or relating directly to Satellite 6.12 AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing labels Oct 11, 2023
@Gauravtalreja1 Gauravtalreja1 merged commit ff06a9d into SatelliteQE:master Oct 11, 2023
7 checks passed
github-actions bot pushed a commit that referenced this pull request Oct 11, 2023
@adarshdubey-star @web-flow
Coverage for autoprovision and reboot of discovered host (#12815)
989a2fd 
* Coverage for autoprovision and reboot of discovered host

* Added one more test case for autoprovision of multiple discoveredhosts

* Making a discovery component helper module

(cherry picked from commit ff06a9d)
github-actions bot pushed a commit that referenced this pull request Oct 11, 2023
@adarshdubey-star @web-flow
Coverage for autoprovision and reboot of discovered host (#12815)
0e776b5 
* Coverage for autoprovision and reboot of discovered host

* Added one more test case for autoprovision of multiple discoveredhosts

* Making a discovery component helper module

(cherry picked from commit ff06a9d)
This was referenced Oct 11, 2023
Merged
Merged
github-actions bot pushed a commit that referenced this pull request Oct 11, 2023
@adarshdubey-star @web-flow
Coverage for autoprovision and reboot of discovered host (#12815)
8323c94 
* Coverage for autoprovision and reboot of discovered host

* Added one more test case for autoprovision of multiple discoveredhosts

* Making a discovery component helper module

(cherry picked from commit ff06a9d)
@Satellite-QE Satellite-QE mentioned this pull request Oct 11, 2023
Merged
LadislavVasina1 pushed a commit that referenced this pull request Oct 20, 2023
@adarshdubey-star @LadislavVasina1
Coverage for autoprovision and reboot of discovered host (#12815)
4587e3a 
* Coverage for autoprovision and reboot of discovered host

* Added one more test case for autoprovision of multiple discoveredhosts

* Making a discovery component helper module
shweta83 pushed a commit to shweta83/robottelo that referenced this pull request Apr 10, 2024
@adarshdubey-star @shweta83
Coverage for autoprovision and reboot of discovered host (SatelliteQE…
af63c92 
…#12815)

* Coverage for autoprovision and reboot of discovered host

* Added one more test case for autoprovision of multiple discoveredhosts

* Making a discovery component helper module
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shubhamsg199 shubhamsg199 shubhamsg199 approved these changes

@Gauravtalreja1 Gauravtalreja1 Gauravtalreja1 approved these changes

Assignees

@adarshdubey-star adarshdubey-star

Labels
6.12.z Introduced in or relating directly to Satellite 6.12 6.13.z Introduced in or relating directly to Satellite 6.13 6.14.z Introduced in or relating directly to Satellite 6.14 API Issues and PRs involving the API AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing CherryPick PR needs CherryPick to previous branches 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.
3 participants
@adarshdubey-star @Gauravtalreja1 @shubhamsg199