Conversation
There was a problem hiding this comment.
Code Review
This pull request removes the AVC_ERROR environment variable from the integration plan and adds an AVC check configuration to the tests file to handle expected SELinux denials. A critical review comment points out that the tests file is auto-generated and should not be edited directly; the changes must instead be implemented within the generator script to prevent them from being overwritten.
|
The test failures here are really strange, I don't understand how this PR could have triggered them. Unless there's some kind of race somehow, like we installed with an older image? |
7516195 to
7f91a8d
Compare
The failure is in the prepare phase - tmt's essential-requires step is trying to install audit, policycoreutils on the VM. These are dependencies of the Tricky thing is even though the check is |
Skip in TMT test to avoid duplicate with downstream Signed-off-by: Xiaofeng Wang <henrywangxf@me.com>
Skip in TMT test to avoid duplicate with downstream