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
Handle secureboot on&off for ima_evm tests #13673
Conversation
faa7096
to
8863259
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only minor coding style comments.
Others are LGTM.
Looks like there is CI check fail? |
8863259
to
11bd100
Compare
Fixed |
11bd100
to
4f009a0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
4f009a0
to
c3e0fc6
Compare
Based on bsc#1189988, we need disable secureboot if we set kernel parameter "ima_appraise=fix", but we should make sure other tests can still run in enabled state, so enhance the test logic.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for the update.
Based on bsc#1189988, we need disable secureboot
if we set kernel parameter "ima_appraise=fix", but
we should make sure other tests can still run in
enabled state, so enhance the test logic.
Related ticket:
https://progress.opensuse.org/issues/100694
https://progress.opensuse.org/issues/102311
Needles: already uploaded
Verification run:
https://openqa.suse.de/tests/7654325
https://openqa.suse.de/tests/7654326
https://openqa.suse.de/tests/7654327