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

check_kernel_taint: Implement configurable taint mask #19576

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

mdoucha
Copy link
Contributor

@mdoucha mdoucha commented Jun 20, 2024

Add LTP_TAINT_EXPECTED variable which allows setting the expected list of kernel taint flags. Any flag not in the expected list will be treated as a test failure. Flags on the list will be reported in either case but without impact on the final result.

Add LTP_TAINT_EXPECTED variable which allows setting the expected
list of kernel taint flags. Any flag not in the expected list will
be treated as a test failure. Flags on the list will be reported
in either case but without impact on the final result.
Add a helper function which will return if (and how many) PUBLISH_*
veriables are set on the job. This can be used for deciding whether
minor issues should cause a test to fail, or softfail in order to
allow dependent jobs to run.
Reenable shutdown_ltp failure on unexpected kernel taint unless it's
an install job (any PUBLISH_* variable is set).
Copy link

Great PR! Please pay attention to the following items before merging:

Files matching lib/**.pm:

  • Consider adding or extending unit tests in t/

This is an automatically generated QA checklist based on modified files.

@mdoucha
Copy link
Contributor Author

mdoucha commented Jun 28, 2024

All VRs finished with the expected result. This PR is ready for review and merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant