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.

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

adjusting oscap tests for rhel10 #17055

Merged
merged 1 commit into from
Feb 6, 2025
Merged

Conversation

pondrejk
Copy link
Contributor

@pondrejk pondrejk commented Dec 3, 2024

Problem Statement

needs changes in conf/repos.yml

Solution

Related Issues

@pondrejk pondrejk added No-CherryPick PR doesnt need CherryPick to previous branches Stream Introduced in or relating directly to Satellite Stream/Master Do Not Merge labels Dec 3, 2024
@pondrejk pondrejk self-assigned this Dec 3, 2024
@pondrejk pondrejk requested a review from a team as a code owner December 3, 2024 12:18
@pondrejk pondrejk marked this pull request as draft December 3, 2024 12:18
@lhellebr
Copy link
Contributor

lhellebr commented Dec 4, 2024

Looks good but needs PRT. Will RHEL10 ssg download work out of the box?

@lhellebr
Copy link
Contributor

I don't see any conf MR open from you. Are you going to create one? Is something blocking it?

@pondrejk
Copy link
Contributor Author

pondrejk commented Jan 8, 2025

config changes were merged some time before (not by me), the targeted tests currently blocked by SAT-30043

@pondrejk pondrejk marked this pull request as ready for review January 27, 2025 14:06
@pondrejk pondrejk requested a review from a team as a code owner January 27, 2025 14:06
@pondrejk
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/longrun/test_oscap.py -k test_positive_oscap_run_via_ansible[rhel10-ipv4]

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9991
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/longrun/test_oscap.py -k test_positive_oscap_run_via_ansible[rhel10-ipv4] --external-logging
Test Result : ========== 1 failed, 17 deselected, 30 warnings in 1176.46s (0:19:36) ==========

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Jan 27, 2025
@pondrejk
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/longrun/test_oscap.py -k test_positive_oscap_run_via_ansible[rhel10-ipv4]

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9993
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/longrun/test_oscap.py -k test_positive_oscap_run_via_ansible[rhel10-ipv4] --external-logging
Test Result : ========== 1 failed, 17 deselected, 30 warnings in 1298.02s (0:21:38) ==========

@pondrejk
Copy link
Contributor Author

turns out that rh10 client repo is needed for this one as well

@pondrejk
Copy link
Contributor Author

pondrejk commented Feb 4, 2025

trigger: test-robottelo
pytest: tests/foreman/longrun/test_oscap.py -k test_positive_oscap_run_via_ansible[rhel10-ipv4]

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 10046
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/longrun/test_oscap.py -k test_positive_oscap_run_via_ansible[rhel10-ipv4] --external-logging
Test Result : ========== 1 passed, 17 deselected, 30 warnings in 1128.01s (0:18: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 4, 2025
@pondrejk
Copy link
Contributor Author

pondrejk commented Feb 5, 2025

@lhellebr mind revisiting?

@lhellebr
Copy link
Contributor

lhellebr commented Feb 5, 2025

ACK and FYI: https://issues.redhat.com/browse/SAT-30851

@pondrejk pondrejk merged commit 18af4bc into SatelliteQE:master Feb 6, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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
Development

Successfully merging this pull request may close these issues.

3 participants