diff options
author | Petr Machata <petrm@nvidia.com> | 2024-04-12 19:03:06 +0200 |
---|---|---|
committer | Paolo Abeni <pabeni@redhat.com> | 2024-04-16 12:14:41 +0200 |
commit | 492976136bb9fac0ebda28a163561eea8c2896d5 (patch) | |
tree | a60732f6b7081a5697f2eda66544a7c2fcfa5a64 /tools/testing/selftests/net/forwarding/lib.sh | |
parent | selftests: forwarding: lib.sh: Validate NETIFS (diff) | |
download | linux-492976136bb9fac0ebda28a163561eea8c2896d5.tar.xz linux-492976136bb9fac0ebda28a163561eea8c2896d5.zip |
selftests: forwarding: bail_on_lldpad() should SKIP
$ksft_skip is used to mark selftests that have tooling issues. The fact
that LLDPad is running, but shouldn't, is one such issue. Therefore have
bail_on_lldpad() bail with $ksft_skip.
Signed-off-by: Petr Machata <petrm@nvidia.com>
Reviewed-by: Benjamin Poirier <bpoirier@nvidia.com>
Reviewed-by: Hangbin Liu <liuhangbin@gmail.com>
Signed-off-by: Paolo Abeni <pabeni@redhat.com>
Diffstat (limited to 'tools/testing/selftests/net/forwarding/lib.sh')
-rw-r--r-- | tools/testing/selftests/net/forwarding/lib.sh | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/tools/testing/selftests/net/forwarding/lib.sh b/tools/testing/selftests/net/forwarding/lib.sh index 3cbbc2fd4d7d..7913c6ee418d 100644 --- a/tools/testing/selftests/net/forwarding/lib.sh +++ b/tools/testing/selftests/net/forwarding/lib.sh @@ -2138,6 +2138,8 @@ bail_on_lldpad() { local reason1="$1"; shift local reason2="$1"; shift + local caller=${FUNCNAME[1]} + local src=${BASH_SOURCE[1]} if systemctl is-active --quiet lldpad; then @@ -2158,7 +2160,8 @@ bail_on_lldpad() an environment variable ALLOW_LLDPAD to a non-empty string. EOF - exit 1 + log_test_skip $src:$caller + exit $EXIT_STATUS else return fi |