[Devel] [PATCH hci-8.1 v2 1/1] selftests: bonding: do not build on the namespaces of a killed run

[Eva Kurchatova]

A lag test run that is killed, by the suite timeout for one, leaves
lag_node1 and lag_node2 behind, and the next run builds on them: the
veth is renamed into a namespace that already has an eth0, the kernel
warns about a partly applied link change, and the bond never comes
back:

  TEST: mode (2) bond recovery                              [FAIL]
        Bond failed to recover

The failure is intermittent, which makes it look like a timing problem.
Delete the leftover namespaces before creating them.

https://virtuozzo.atlassian.net/browse/VSTOR-139651
Feature: fix selftests
Signed-off-by: Eva Kurchatova <eva.kurchatova at virtuozzo.com>
---
 tools/testing/selftests/drivers/net/bonding/lag_lib.sh | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/tools/testing/selftests/drivers/net/bonding/lag_lib.sh b/tools/testing/selftests/drivers/net/bonding/lag_lib.sh
index bf9bcd1b5ec0..3710acd2f8d9 100644
--- a/tools/testing/selftests/drivers/net/bonding/lag_lib.sh
+++ b/tools/testing/selftests/drivers/net/bonding/lag_lib.sh
@@ -90,8 +90,9 @@ lag_setup2x2()
 	local state=${1:-down}
 	local namespaces="lag_node1 lag_node2"
 
-	# create namespaces
+	# A killed run leaves the namespaces behind, start clean
 	for n in ${namespaces}; do
+		ip netns del ${n} 2>/dev/null
 		ip netns add ${n}
 	done
 
-- 
2.55.0