From: Aleksei Oladko <[email protected]>

Some net/forwarding kselftests set NUM_NETIFS, causing lib.sh to create
the requested number of veth peer interfaces.

These interfaces are not removed when the tests finish, leaving stale
veth devices in the system. This can cause subsequent tests to fail,
for example min_max_mtu.sh.

Ensure that veth peers created via NUM_NETIFS are properly removed at
the end of the tests to avoid interference between test runs.

https://virtuozzo.atlassian.net/browse/VSTOR-121804

Signed-off-by: Aleksei Oladko <[email protected]>

Feature: fix selftests

---
changes:
 v2: Added ignoring of ip link del dev errors - just in case those veth
     interfaces will be already removed one day.

---
 tools/testing/selftests/net/forwarding/lib.sh | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/tools/testing/selftests/net/forwarding/lib.sh 
b/tools/testing/selftests/net/forwarding/lib.sh
index 9f7f5b7ab18c8..4ef22f17f1848 100644
--- a/tools/testing/selftests/net/forwarding/lib.sh
+++ b/tools/testing/selftests/net/forwarding/lib.sh
@@ -793,6 +793,10 @@ vrf_cleanup()
        ip -6 rule del pref 32765
        ip -4 rule add pref 0 table local
        ip -4 rule del pref 32765
+
+       for ((i = 1; i <= NUM_NETIFS; i=$i+2)); do
+               ip link delete dev ${NETIFS[p$i]} 2>/dev/null || true
+       done
 }
 
 __last_tb_id=0
-- 
2.43.0

_______________________________________________
Devel mailing list
[email protected]
https://lists.openvz.org/mailman/listinfo/devel

Reply via email to