diff options
author | Thomas Haller <thaller@redhat.com> | 2023-09-08 00:07:20 +0200 |
---|---|---|
committer | Florian Westphal <fw@strlen.de> | 2023-09-08 11:50:41 +0200 |
commit | a529b3dcda72a8b60eb2ccc9143faf284b01e49c (patch) | |
tree | dc9d3d7c3323bc1b822638d5437938b3e41327dd /tests/shell/run-tests.sh | |
parent | c9393e2468ee6cc27bbd9311839699deca6e17cc (diff) |
tests/shell: print "kernel is tainted" separate from test result
Once the kernel is tainted, it stays until reboot. It would not be
useful to fail the entire test run based on that (and we don't do that).
But then, it seems odd to print this in the same style as the test
results, because a [FAILED] of a test counts as an overall failure.
Instead, print this warning in a different style.
Previously:
$ ./tests/shell/run-tests.sh -- /usr/bin/true
...
W: [FAILED] kernel is tainted
I: [OK] /usr/bin/true
I: results: [OK] 1 [SKIPPED] 0 [FAILED] 0 [TOTAL] 1
Now:
$ ./tests/shell/run-tests.sh -- /usr/bin/true
...
W: kernel is tainted
I: [OK] /usr/bin/true
I: results: [OK] 1 [SKIPPED] 0 [FAILED] 0 [TOTAL] 1
Signed-off-by: Thomas Haller <thaller@redhat.com>
Signed-off-by: Florian Westphal <fw@strlen.de>
Diffstat (limited to 'tests/shell/run-tests.sh')
-rwxr-xr-x | tests/shell/run-tests.sh | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/tests/shell/run-tests.sh b/tests/shell/run-tests.sh index 4c1ab29b..6abb6c0c 100755 --- a/tests/shell/run-tests.sh +++ b/tests/shell/run-tests.sh @@ -496,7 +496,8 @@ check_kmemleak() read kernel_tainted < /proc/sys/kernel/tainted if [ "$kernel_tainted" -ne 0 ] ; then - msg_warn "[FAILED] kernel is tainted" + msg_warn "kernel is tainted" + echo fi print_test_header() { |