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

Testing Discovery #14274

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from
Draft

Testing Discovery #14274

wants to merge 1 commit into from

Conversation

shubhamsg199
Copy link
Contributor

No description provided.

@shubhamsg199 shubhamsg199 added No-CherryPick PR doesnt need CherryPick to previous branches Do Not Merge labels Mar 7, 2024
@shubhamsg199 shubhamsg199 self-assigned this Mar 7, 2024
@shubhamsg199 shubhamsg199 requested a review from a team as a code owner March 7, 2024 07:50
@shubhamsg199
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_discoveredhost.py -k test_rhel_pxe_discovery_provisioning
provisioning: true
pod_resources_size: extra_extra_large

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 5970
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/cli/test_discoveredhost.py -k test_rhel_pxe_discovery_provisioning --external-logging --include-onprem-provisioning
Test Result : ===== 2 passed, 15 deselected, 110 warnings, 1 error in 2640.28s (0:44:00) =====

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Mar 7, 2024
lhellebr
lhellebr reviewed Mar 7, 2024
View reviewed changes
tests/foreman/cli/test_discoveredhost.py Outdated
@@ -103,7 +103,7 @@ def test_rhel_pxeless_discovery_provisioning(
:Setup: Provisioning should be configured and a host should be
discovered via cli

:expectedresults: Host should be provisioned successfully
:expectedresults: Host should be provisioned successfully.
Copy link
Contributor

Choose a reason for hiding this comment

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

Why? This isn't even for the sake of consistency. The other tests don't have period here as well.

@lhellebr
Copy link
Contributor

lhellebr commented Mar 7, 2024

Maybe this should be a Draft?

@Gauravtalreja1 Gauravtalreja1 marked this pull request as draft March 8, 2024 09:45
@shubhamsg199 shubhamsg199 force-pushed the testing_discovery branch 2 times, most recently from 5de88c3 to 95010e5 Compare March 8, 2024 12:50
@shubhamsg199
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_discoveredhost.py tests/foreman/cli/test_discoveredhost.py tests/foreman/ui/test_discoveredhost.py
provisioning: true
pod_resources_size: extra_extra_large

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 5997
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_discoveredhost.py tests/foreman/cli/test_discoveredhost.py tests/foreman/ui/test_discoveredhost.py --external-logging --include-onprem-provisioning
Test Result : == 19 failed, 6 passed, 14 deselected, 4106 warnings in 38696.25s (10:44:56) ===

@shubhamsg199
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_provisioning.py tests/foreman/api/test_provisioning_puppet.py
provisioning: true
pod_resources_size: extra_extra_large

@shubhamsg199
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_provisioning.py tests/foreman/api/test_provisioning_puppet.py
provisioning: true
pod_resources_size: extra_extra_large

@shubhamsg199
Copy link
Contributor Author

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

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7064
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/cli/test_discoveredhost.py --external-logging --include-onprem-provisioning
Test Result : ==== 3 failed, 1 passed, 13 deselected, 503 warnings in 6179.37s (1:42:59) =====

@shubhamsg199
Copy link
Contributor Author

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

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7082
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/cli/test_discoveredhost.py --external-logging --include-onprem-provisioning
Test Result : = 2 failed, 2 passed, 13 deselected, 506 warnings, 2 errors in 3960.11s (1:06:00) =

@shubhamsg199
Copy link
Contributor Author

trigger: test-robottelo
pytest: -m on_premises_provisioning tests/foreman/ --include-onprem-provisioning
provisioning: true

@shubhamsg199
Copy link
Contributor Author 

trigger: test-robottelo
pytest: -m on_premises_provisioning tests/foreman/ --include-onprem-provisioning
provisioning: true
env:
     BROKER_SSH_BACKEND: hussh

@shubhamsg199
testing discovery provisioning failure
3fc53d2 
Signed-off-by: Shubham Ganar <shubhamsg123m@gmail.com>
Signed-off-by: Shubham Ganar <sganar@redhat.com>
@shubhamsg199
Copy link
Contributor Author

trigger: test-robottelo
pytest: -m on_premises_provisioning tests/foreman/ --include-onprem-provisioning
provisioning: true
env:
BROKER_SSH_BACKEND: hussh

@shubhamsg199
Copy link
Contributor Author 

trigger: test-robottelo
pytest: -m on_premises_provisioning tests/foreman/ --include-onprem-provisioning
provisioning: true
env:
     BROKER_SSH_BACKEND: hussh
     ROBOTTELO_server__deploy_arguments__deploy_sat_version: '6.16'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9099
Build Status: UNSTABLE
PRT Comment: pytest -m on_premises_provisioning tests/foreman/ --include-onprem-provisioning --external-logging --include-onprem-provisioning
Test Result : == 6 skipped, 5373 deselected, 5500 warnings, 90 errors in 4808.01s (1:20:08) ==

@shubhamsg199
Copy link
Contributor Author 

trigger: test-robottelo
pytest: -m on_premises_provisioning tests/foreman/ --include-onprem-provisioning
provisioning: true
env:
     BROKER_SSH_BACKEND: hussh
     ROBOTTELO_server__deploy_arguments__deploy_sat_version: '6.16.0'
     ROBOTTELO_server__deploy_arguments__deploy_snap_version: '9.0'

@shubhamsg199
Copy link
Contributor Author 

trigger: test-robottelo
pytest: -m on_premises_provisioning tests/foreman/ --include-onprem-provisioning --component ComputeResources-VMWare,DiscoveryImage
provisioning: true
env:
     BROKER_SSH_BACKEND: hussh
     ROBOTTELO_server__deploy_arguments__deploy_sat_version: '6.16.0'
     ROBOTTELO_server__deploy_arguments__deploy_snap_version: '9.0'

@shubhamsg199
Copy link
Contributor Author 

trigger: test-robottelo
pytest: -m on_premises_provisioning tests/foreman/ --include-onprem-provisioning --component DiscoveryImage
provisioning: true
env:
     BROKER_SSH_BACKEND: hussh
     ROBOTTELO_server__deploy_arguments__deploy_sat_version: '6.16.0'
     ROBOTTELO_server__deploy_arguments__deploy_snap_version: '9.0'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lhellebr lhellebr lhellebr left review comments

Assignees

@shubhamsg199 shubhamsg199

Labels
Do Not Merge No-CherryPick PR doesnt need CherryPick to previous branches PRT-Failed Indicates that latest PRT run is failed for the PR
Projects
None yet
Milestone
No milestone
Development

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