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

ReportTemplates Coverage for RHEL10 hosts #17635

Merged
merged 2 commits into from
Feb 24, 2025
Merged

ReportTemplates Coverage for RHEL10 hosts #17635

merged 2 commits into from
Feb 24, 2025
+13 −13

Conversation

damoore044
Copy link
Contributor

@damoore044 damoore044 commented Feb 19, 2025 •
edited
Loading

Problem Statement

Add parametrization for RHEL10, for the rhel_contenthost's in these ReportTemplates cases.
Some RT tests are already running with a RHEL10 contenthost param, but found a few more we would want also covered.
Also fix AK creation issue seen locally and in Stream, for api/test_reporttemplates.py::test_positive_installable_errata

PRT Case

trigger: test-robottelo
pytest: tests/foreman/api/test_reporttemplates.py tests/foreman/cli/test_reporttemplates.py -k 'rhel10'

@damoore044 damoore044 added No-CherryPick PR doesnt need CherryPick to previous branches Stream Introduced in or relating directly to Satellite Stream/Master labels Feb 19, 2025
@damoore044 damoore044 self-assigned this Feb 19, 2025
@damoore044 damoore044 requested a review from a team as a code owner February 19, 2025 17:58
@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_reporttemplates.py -k 'rhel10'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10245
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_reporttemplates.py -k rhel10 --external-logging
Test Result : = 1 failed, 4 passed, 36 deselected, 501 warnings, 1 error in 1952.74s (0:32:32) =

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Feb 19, 2025
@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_reporttemplates.py -k 'rhel10'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10246
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_reporttemplates.py -k rhel10 --external-logging
Test Result : ========= 5 passed, 37 deselected, 508 warnings in 2032.02s (0:33:52) ==========

@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 19, 2025
@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_reporttemplates.py tests/foreman/ui/test_reporttemplates.py -k 'rhel10'

@damoore044 damoore044 force-pushed the ReportTemplates_RHEL10 branch from a5b8e13 to a0122f9 Compare February 19, 2025 19:39
@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Feb 19, 2025
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10247
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/cli/test_reporttemplates.py tests/foreman/ui/test_reporttemplates.py -k rhel10 --external-logging
Test Result : ==== 1 failed, 41 deselected, 195 warnings, 3 errors in 1025.63s (0:17:05) =====

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Feb 19, 2025
@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_reporttemplates.py tests/foreman/cli/test_reporttemplates.py -k 'rhel10'

@Satellite-QE
Copy link
Collaborator

Satellite-QE commented Feb 19, 2025 •
edited by damoore044
Loading

PRT Result

Build Number: 10248
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_reporttemplates.py tests/foreman/cli/test_reporttemplates.py -k rhel10 --external-logging
Test Result : ===== 5 passed, 60 deselected, 535 warnings, 1 error in 2126.36s (0:35:26) =====

[David]: ^ not sure why the CLI case fails to checkout in rhel_contenthost fixture , local pass for me

@damoore044 damoore044 force-pushed the ReportTemplates_RHEL10 branch from a0122f9 to 44f5190 Compare February 20, 2025 16:44
@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_reporttemplates.py -k 'rhel10'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10264
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/cli/test_reporttemplates.py -k rhel10 --external-logging
Test Result : =========== 23 deselected, 32 warnings, 1 error in 668.18s (0:11:08) ===========

@damoore044 damoore044 force-pushed the ReportTemplates_RHEL10 branch from 44f5190 to e3ae83d Compare February 20, 2025 17:21
@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_reporttemplates.py -k 'rhel10'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10266
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/cli/test_reporttemplates.py -k rhel10 --external-logging
Test Result : =========== 23 deselected, 32 warnings, 1 error in 654.93s (0:10:54) ===========

@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_reporttemplates.py -k 'rhel10'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10267
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/cli/test_reporttemplates.py -k rhel10 --external-logging
Test Result : =========== 23 deselected, 32 warnings, 1 error in 641.87s (0:10:41) ===========

@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_reporttemplates.py -k 'rhel10'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10268
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/cli/test_reporttemplates.py -k rhel10 --external-logging
Test Result : =========== 23 deselected, 32 warnings, 1 error in 629.67s (0:10:29) ===========

@damoore044 damoore044 force-pushed the ReportTemplates_RHEL10 branch from e3ae83d to 603d605 Compare February 21, 2025 00:03
@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_reporttemplates.py -k 'rhel10'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10269
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/cli/test_reporttemplates.py -k rhel10 --external-logging
Test Result : =========== 23 deselected, 32 warnings, 1 error in 625.99s (0:10:25) ===========

@damoore044 damoore044 force-pushed the ReportTemplates_RHEL10 branch from 603d605 to 9942a7d Compare February 24, 2025 13:48
@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_reporttemplates.py -k 'rhel10'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10282
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/cli/test_reporttemplates.py -k rhel10 --external-logging
Test Result : =========== 23 deselected, 32 warnings, 1 error in 615.52s (0:10:15) ===========

@damoore044 damoore044 force-pushed the ReportTemplates_RHEL10 branch from 9942a7d to 953d785 Compare February 24, 2025 16:20
@damoore044 damoore044 force-pushed the ReportTemplates_RHEL10 branch from 953d785 to 5a6dc7c Compare February 24, 2025 16:21
@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_reporttemplates.py -k 'rhel10'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10290
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/cli/test_reporttemplates.py -k rhel10 --external-logging
Test Result : ========== 1 passed, 23 deselected, 32 warnings in 751.60s (0:12:31) ===========

@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 24, 2025
ColeHiggins2
ColeHiggins2 reviewed Feb 24, 2025
View reviewed changes
Copy link
Member

@ColeHiggins2 ColeHiggins2 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Which test are we trying to run via prt here? @damoore044

@damoore044
Copy link
Contributor Author

@ColeHiggins2 cli/test_positive_generate_installed_packages_report[rhel10-ipv4]
was getting BrokerError failing to checkout RHEL10 , until merger of Broker#353 .

@ColeHiggins2
Copy link
Member

@damoore044 Ack, okay yes I see that is passing. Do we want to test api/test_reporttemplates.py::test_positive_installable_errata at all?

ColeHiggins2
ColeHiggins2 approved these changes Feb 24, 2025
View reviewed changes
Copy link
Member

@ColeHiggins2 ColeHiggins2 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

@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_reporttemplates.py -k 'rhel10'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10293
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_reporttemplates.py -k rhel10 --external-logging
Test Result : ========= 5 passed, 37 deselected, 576 warnings in 2493.97s (0:41:33) ==========

@JacobCallahan JacobCallahan merged commit 8282d4f into SatelliteQE:master Feb 24, 2025
12 checks passed
@damoore044 damoore044 mentioned this pull request Feb 25, 2025
Merged
damoore044 added a commit that referenced this pull request Feb 25, 2025
@damoore044
[6.17.z] CP ReportTemplates Coverage for RHEL10 hosts (#17687)
24fd64a 
Cherry-pick: ReportTemplates Coverage for RHEL10 hosts (#17635)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JacobCallahan JacobCallahan JacobCallahan approved these changes

@ColeHiggins2 ColeHiggins2 ColeHiggins2 approved these changes

Assignees

@damoore044 damoore044

Labels
No-CherryPick PR doesnt need CherryPick to previous branches PRT-Passed Indicates that latest PRT run is passed 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.
4 participants
@damoore044 @Satellite-QE @ColeHiggins2 @JacobCallahan