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 Cli/API/UI tests for bootc #17634

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Add Cli/API/UI tests for bootc #17634

wants to merge 8 commits into from
+141 −1

Conversation

sambible
Copy link
Contributor

Problem Statement

Adds tests for CLI/UI/API for bootc, using dummy facts

Related Issues

Relies on: SatelliteQE/airgun#1745 and SatelliteQE/nailgun#1277

@sambible sambible added No-CherryPick PR doesnt need CherryPick to previous branches Stream Introduced in or relating directly to Satellite Stream/Master labels Feb 19, 2025
@sambible sambible requested a review from a team February 19, 2025 14:02
@sambible sambible self-assigned this Feb 19, 2025
@sambible sambible requested review from a team as code owners February 19, 2025 14:02
This was referenced Feb 19, 2025
Merged
Merged
@sambible
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_host.py::test_bootc_booted_container_images tests/foreman/cli/test_host.py::test_positive_register_read_bootc tests/foreman/api/test_host.py::test_positive_bootc_api_actions --co
airgun: 1745
nailgun: 1277

@LadislavVasina1
Copy link
Contributor 

trigger: test-robottelo
pytest: tests/foreman/ui/test_host.py::test_bootc_booted_container_images tests/foreman/cli/test_host.py::test_positive_register_read_bootc tests/foreman/api/test_host.py::test_positive_bootc_api_actions
airgun: 1745
nailgun: 1277

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10262
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_host.py::test_bootc_booted_container_images tests/foreman/cli/test_host.py::test_positive_register_read_bootc tests/foreman/api/test_host.py::test_positive_bootc_api_actions --external-logging
Test Result : ================== 30 warnings, 3 errors in 542.21s (0:09:02) ==================

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Feb 20, 2025
vijaysawant
vijaysawant requested changes Feb 24, 2025
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.

Please confirm :CaseComponent: before merging. Rest of things looks good.

@sambible
Copy link
Contributor Author 

trigger: test-robottelo
pytest: tests/foreman/ui/test_host.py::test_bootc_booted_container_images tests/foreman/cli/test_host.py::test_positive_register_read_bootc tests/foreman/api/test_host.py::test_positive_bootc_api_actions
airgun: 1745
nailgun: 1277

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10292
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_host.py::test_bootc_booted_container_images tests/foreman/cli/test_host.py::test_positive_register_read_bootc tests/foreman/api/test_host.py::test_positive_bootc_api_actions --external-logging
Test Result : ================== 35 warnings, 3 errors in 730.98s (0:12:10) ==================

vsedmik
vsedmik requested changes Feb 25, 2025
View reviewed changes
Copy link
Contributor

@vsedmik vsedmik left a comment

Choose a reason for hiding this comment

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

A hosts count question - one or more?

@sambible
Copy link
Contributor Author 

trigger: test-robottelo
pytest: tests/foreman/ui/test_host.py::test_bootc_booted_container_images tests/foreman/cli/test_host.py::test_positive_register_read_bootc tests/foreman/api/test_host.py::test_positive_bootc_api_actions
airgun: 1745
nailgun: 1277

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10321
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_host.py::test_bootc_booted_container_images tests/foreman/cli/test_host.py::test_positive_register_read_bootc tests/foreman/api/test_host.py::test_positive_bootc_api_actions --external-logging
Test Result : ================= 271 warnings, 3 errors in 2294.80s (0:38:14) =================

@sambible
Copy link
Contributor Author 

trigger: test-robottelo
pytest: tests/foreman/ui/test_host.py::test_bootc_booted_container_images tests/foreman/cli/test_host.py::test_positive_register_read_bootc tests/foreman/api/test_host.py::test_positive_bootc_api_actions
airgun: 1745
nailgun: 1277

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10323
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/ui/test_host.py::test_bootc_booted_container_images tests/foreman/cli/test_host.py::test_positive_register_read_bootc tests/foreman/api/test_host.py::test_positive_bootc_api_actions --external-logging
Test Result : ================= 3 passed, 227 warnings in 1728.96s (0:28: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 Feb 26, 2025
@sambible sambible added CherryPick PR needs CherryPick to previous branches 6.17.z and removed No-CherryPick PR doesnt need CherryPick to previous branches labels Feb 26, 2025
@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Feb 26, 2025
@Satellite-QE Satellite-QE mentioned this pull request Feb 27, 2025
Open
@Satellite-QE Satellite-QE mentioned this pull request Feb 27, 2025
Open
JacobCallahan
JacobCallahan requested changes Feb 27, 2025
View reviewed changes
Copy link
Member

@JacobCallahan JacobCallahan left a comment

Choose a reason for hiding this comment

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

Please address @shweta83's suggestion

@vsedmik
Copy link
Contributor

vsedmik commented Mar 3, 2025 

trigger: test-robottelo
pytest: tests/foreman/ui/test_host.py::test_bootc_booted_container_images tests/foreman/cli/test_host.py::test_positive_register_read_bootc tests/foreman/api/test_host.py::test_positive_bootc_api_actions
airgun: 1745
nailgun: 1277

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10362
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_host.py::test_bootc_booted_container_images tests/foreman/cli/test_host.py::test_positive_register_read_bootc tests/foreman/api/test_host.py::test_positive_bootc_api_actions --external-logging
Test Result : ============ 2 failed, 1 passed, 202 warnings in 1977.13s (0:32:57) ============

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Mar 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shweta83 shweta83 shweta83 left review comments

@vsedmik vsedmik vsedmik approved these changes

@JacobCallahan JacobCallahan JacobCallahan requested changes

@vijaysawant vijaysawant vijaysawant approved these changes

@LadislavVasina1 LadislavVasina1 LadislavVasina1 approved these changes

@damoore044 damoore044 damoore044 approved these changes

Assignees

@sambible sambible

Labels
6.17.z CherryPick PR needs CherryPick to previous branches PRT-Failed Indicates that latest PRT run is failed for the PR Stream Introduced in or relating directly to Satellite Stream/Master
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@sambible @LadislavVasina1 @Satellite-QE @vsedmik @JacobCallahan @vijaysawant @shweta83 @damoore044