| Message ID | bf9b79f45de378f88344d44550f0a5052b386199.1724692132.git.petrm@nvidia.com |
|---|---|
| State | Accepted |
| Commit | 65a3cce43d5b4c53cf16b0be1a03991f665a0806 |
| Headers | show |
| Series | [net-next] selftests: forwarding: local_termination: Down ports on cleanup | expand |
Hello: This patch was applied to netdev/net.git (main) by Jakub Kicinski <kuba@kernel.org>: On Mon, 26 Aug 2024 19:15:11 +0200 you wrote: > This test neglects to put ports down on cleanup. Fix it. > > Fixes: 90b9566aa5cd ("selftests: forwarding: add a test for local_termination.sh") > Signed-off-by: Petr Machata <petrm@nvidia.com> > --- > tools/testing/selftests/net/forwarding/local_termination.sh | 4 ++++ > 1 file changed, 4 insertions(+) Here is the summary with links: - [net-next] selftests: forwarding: local_termination: Down ports on cleanup https://git.kernel.org/netdev/net/c/65a3cce43d5b You are awesome, thank you!
diff --git a/tools/testing/selftests/net/forwarding/local_termination.sh b/tools/testing/selftests/net/forwarding/local_termination.sh index c5b0cbc85b3e..9b5a63519b94 100755 --- a/tools/testing/selftests/net/forwarding/local_termination.sh +++ b/tools/testing/selftests/net/forwarding/local_termination.sh @@ -278,6 +278,10 @@ bridge() cleanup() { pre_cleanup + + ip link set $h2 down + ip link set $h1 down + vrf_cleanup }
This test neglects to put ports down on cleanup. Fix it. Fixes: 90b9566aa5cd ("selftests: forwarding: add a test for local_termination.sh") Signed-off-by: Petr Machata <petrm@nvidia.com> --- tools/testing/selftests/net/forwarding/local_termination.sh | 4 ++++ 1 file changed, 4 insertions(+)