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

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

[Eva Kurchatova]

Patchset for task https://virtuozzo.atlassian.net/browse/VSTOR-139651

Changes in v2:

- Drop the slowwait change, it is a separate matter.
- One-line comment, no measured numbers in the message.
- Drop the unused "|| return 1".
- Sent alone, 1/3 and 2/3 are merged.
- Link to v1: https://lists.openvz.org/pipermail/devel/2026-September/084966.html

Link to v1: http://lore.virtuozzo.com/kernel/20260831234036.1649006-3-eva.kurchatova@virtuozzo.com/

Eva Kurchatova (1):
  selftests: bonding: do not build on the namespaces of a killed run

 tools/testing/selftests/drivers/net/bonding/lag_lib.sh | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

-- 
2.55.0

_______________________________________________
Devel mailing list
Devel@openvz.org
https://lists.openvz.org/mailman/listinfo/devel

[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@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

_______________________________________________
Devel mailing list
Devel@openvz.org
https://lists.openvz.org/mailman/listinfo/devel

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

[Konstantin Khorenko]

On Mon, 28 Sep 2026 22:46:45 +0300, Eva Kurchatova wrote:
> selftests: bonding: do not build on the namespaces of a killed run
> 
> Patchset for task https://virtuozzo.atlassian.net/browse/VSTOR-139651
> 
> Changes in v2:
> 
> - Drop the slowwait change, it is a separate matter.
> - One-line comment, no measured numbers in the message.
> - Drop the unused "|| return 1".
> - Sent alone, 1/3 and 2/3 are merged.
> - Link to v1: https://lists.openvz.org/pipermail/devel/2026-September/084966.html
> 
> [...]

Applied to branch-rh10-6.12.0-211.52.1.22.x.vz10-ovz, will be released in rh10-6.12.0-211.52.1.22.5.vz10, thanks!

[1/1] selftests: bonding: do not build on the namespaces of a killed run
      https://bitbucket.org/virtuozzocore/vzkernel.vzs/commits/4687be708287

-- 
Konstantin Khorenko <khorenko@virtuozzo.com>

_______________________________________________
Devel mailing list
Devel@openvz.org
https://lists.openvz.org/mailman/listinfo/devel