]> git.baikalelectronics.ru Git - kernel.git/commitdiff
selftests: fib_nexthop_multiprefix: fix cleanup() netns deletion
authorPaolo Pisati <paolo.pisati@canonical.com>
Tue, 14 Jul 2020 15:40:55 +0000 (17:40 +0200)
committerDavid S. Miller <davem@davemloft.net>
Tue, 14 Jul 2020 22:06:12 +0000 (15:06 -0700)
During setup():
...
        for ns in h0 r1 h1 h2 h3
        do
                create_ns ${ns}
        done
...

while in cleanup():
...
        for n in h1 r1 h2 h3 h4
        do
                ip netns del ${n} 2>/dev/null
        done
...

and after removing the stderr redirection in cleanup():

$ sudo ./fib_nexthop_multiprefix.sh
...
TEST: IPv4: host 0 to host 3, mtu 1400                              [ OK ]
TEST: IPv6: host 0 to host 3, mtu 1400                              [ OK ]
Cannot remove namespace file "/run/netns/h4": No such file or directory
$ echo $?
1

and a non-zero return code, make kselftests fail (even if the test
itself is fine):

...
not ok 34 selftests: net: fib_nexthop_multiprefix.sh # exit=1
...

Signed-off-by: Paolo Pisati <paolo.pisati@canonical.com>
Reviewed-by: David Ahern <dsahern@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
tools/testing/selftests/net/fib_nexthop_multiprefix.sh

index 9dc35a16e4159685fb58ec11e3863afc906ab2cf..51df5e305855a7aa3c7ca61782697a3a5d6bb617 100755 (executable)
@@ -144,7 +144,7 @@ setup()
 
 cleanup()
 {
-       for n in h1 r1 h2 h3 h4
+       for n in h0 r1 h1 h2 h3
        do
                ip netns del ${n} 2>/dev/null
        done