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 Libvirt CLI crud test #15543

Merged
merged 1 commit into from
Jul 3, 2024
Merged

Add Libvirt CLI crud test #15543

merged 1 commit into from
Jul 3, 2024

Conversation

shubhamsg199
Copy link
Contributor

Combining various Libvirt tests into one e2e Libvirt CLI crud test and marking it for upgrade.

@shubhamsg199 shubhamsg199 added CLI Issues and PRs involving the CLI CherryPick PR needs CherryPick to previous branches AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing 6.13.z Introduced in or relating directly to Satellite 6.13 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 labels Jul 1, 2024
@shubhamsg199 shubhamsg199 self-assigned this Jul 1, 2024
@shubhamsg199 shubhamsg199 requested a review from a team as a code owner July 1, 2024 08:07
@shubhamsg199 shubhamsg199 force-pushed the libvirt_crud_cr branch 3 times, most recently from 03514cb to 50d050e Compare July 1, 2024 08:13
@shubhamsg199
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_computeresource_libvirt.py::test_positive_crud_libvirt_cr

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7585
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/cli/test_computeresource_libvirt.py::test_positive_crud_libvirt_cr --external-logging
Test Result : ================== 1 failed, 9 warnings in 643.18s (0:10:43) ===================

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Jul 1, 2024
@shubhamsg199
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_computeresource_libvirt.py::test_positive_crud_libvirt_cr

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7586
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/cli/test_computeresource_libvirt.py::test_positive_crud_libvirt_cr --external-logging
Test Result : ================== 1 passed, 9 warnings in 665.52s (0:11:05) ===================

@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 Jul 1, 2024
Gauravtalreja1
Gauravtalreja1 reviewed Jul 1, 2024
View reviewed changes
tests/foreman/cli/test_computeresource_libvirt.py Outdated Show resolved Hide resolved
tests/foreman/cli/test_computeresource_libvirt.py Outdated Show resolved Hide resolved
tests/foreman/cli/test_computeresource_libvirt.py Outdated Show resolved Hide resolved
tests/foreman/cli/test_computeresource_libvirt.py Outdated Show resolved Hide resolved
tests/foreman/cli/test_computeresource_libvirt.py Outdated Show resolved Hide resolved
tests/foreman/cli/test_computeresource_libvirt.py Outdated Show resolved Hide resolved
@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Jul 1, 2024
@shubhamsg199
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_computeresource_libvirt.py::test_positive_crud_libvirt_cr

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7592
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/cli/test_computeresource_libvirt.py::test_positive_crud_libvirt_cr --external-logging
Test Result : ================== 1 passed, 9 warnings in 625.54s (0:10:25) ===================

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Jul 1, 2024
shweta83
shweta83 approved these changes Jul 1, 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

vijaysawant
vijaysawant reviewed Jul 2, 2024
View reviewed changes
Copy link
Contributor

@vijaysawant vijaysawant left a comment

Choose a reason for hiding this comment

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

Non-blocking, rest of things looks good.

tests/foreman/cli/test_computeresource_libvirt.py Outdated Show resolved Hide resolved
@shubhamsg199
Add libvirt cli crud test
aeabbad 
Signed-off-by: Shubham Ganar <shubhamsg123m@gmail.com>
@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Jul 3, 2024
@Gauravtalreja1 Gauravtalreja1 merged commit 94c0e7b into SatelliteQE:master Jul 3, 2024
9 checks passed
github-actions bot pushed a commit that referenced this pull request Jul 3, 2024
@shubhamsg199 @web-flow
Add Libvirt CLI crud test (#15543)
8af070d 
Add libvirt cli crud test

Signed-off-by: Shubham Ganar <shubhamsg123m@gmail.com>
(cherry picked from commit 94c0e7b)
@Satellite-QE Satellite-QE mentioned this pull request Jul 3, 2024
Merged
github-actions bot pushed a commit that referenced this pull request Jul 3, 2024
@shubhamsg199 @web-flow
Add Libvirt CLI crud test (#15543)
e704787 
Add libvirt cli crud test

Signed-off-by: Shubham Ganar <shubhamsg123m@gmail.com>
(cherry picked from commit 94c0e7b)
github-actions bot pushed a commit that referenced this pull request Jul 3, 2024
@shubhamsg199 @web-flow
Add Libvirt CLI crud test (#15543)
2c4aff0 
Add libvirt cli crud test

Signed-off-by: Shubham Ganar <shubhamsg123m@gmail.com>
(cherry picked from commit 94c0e7b)
This was referenced Jul 3, 2024
Merged
Merged
jyejare pushed a commit to jyejare/robottelo that referenced this pull request Oct 19, 2024
@shubhamsg199 @jyejare
Add Libvirt CLI crud test (SatelliteQE#15543)
8ef933b 
Add libvirt cli crud test

Signed-off-by: Shubham Ganar <shubhamsg123m@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Gauravtalreja1 Gauravtalreja1 Gauravtalreja1 approved these changes

@vijaysawant vijaysawant vijaysawant approved these changes

@shweta83 shweta83 shweta83 approved these changes

Assignees

@shubhamsg199 shubhamsg199

Labels
6.13.z Introduced in or relating directly to Satellite 6.13 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing CherryPick PR needs CherryPick to previous branches CLI Issues and PRs involving the CLI
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@shubhamsg199 @Satellite-QE @Gauravtalreja1 @vijaysawant @shweta83