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

[6.16.z] assert fapolicyd is running after setup finished #16896

Conversation

Satellite-QE
Copy link
Collaborator

Cherrypick of PR: #16895

Problem Statement

There is a bug (https://issues.redhat.com/browse/SAT-29311) in fapolicyd, which freezes the machine and our tests didn't catch it. While this change will probably not surface that bug (we are still unsure how to reliably trigger it), it should at least assert fapolicyd is actually running (and didn't crash, which it sometimes does!) after policy loading.

Solution

assert systemd things fapolicyd is running

Related Issues

https://issues.redhat.com/browse/SAT-29311

@Satellite-QE Satellite-QE added 6.16.z Introduced in or relating directly to Satellite 6.16 Auto_Cherry_Picked Automatically cherrypicked PR using GHA No-CherryPick PR doesnt need CherryPick to previous branches labels Nov 12, 2024
@Satellite-QE Satellite-QE requested a review from a team as a code owner November 12, 2024 10:22
@Satellite-QE
Copy link
Collaborator Author

trigger: test-robottelo
pytest: tests/foreman/installer/test_installer.py -k test_capsule_installation

@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 9332
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/installer/test_installer.py -k test_capsule_installation --external-logging
Test Result : ========== 1 failed, 23 deselected, 48 warnings in 1702.49s (0:28:22) ==========

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Nov 12, 2024
@evgeni
Copy link
Member

evgeni commented Nov 12, 2024

trigger: test-robottelo
pytest: tests/foreman/installer/test_installer.py -k test_capsule_installation

@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 9333
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/installer/test_installer.py -k test_capsule_installation --external-logging
Test Result : ========== 1 failed, 23 deselected, 48 warnings in 1718.02s (0:28:38) ==========

@jameerpathan111
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/installer/test_installer.py -k test_capsule_installation

@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 9363
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/installer/test_installer.py -k test_capsule_installation --external-logging
Test Result : ========== 1 passed, 23 deselected, 48 warnings in 8150.42s (2:15:50) ==========

@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 Nov 14, 2024
@jameerpathan111 jameerpathan111 merged commit e0db434 into 6.16.z Nov 14, 2024
20 checks passed
@jameerpathan111 jameerpathan111 deleted the cherry-pick-6.16.z-04e6b7b3cf61977cec8b754f30c6e3081f651222 branch November 14, 2024 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.16.z Introduced in or relating directly to Satellite 6.16 Auto_Cherry_Picked Automatically cherrypicked PR using GHA No-CherryPick PR doesnt need CherryPick to previous branches PRT-Passed Indicates that latest PRT run is passed for the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants