From 0216ffecd9d1cca45b03f2f5f20bea6349c3bb18 Mon Sep 17 00:00:00 2001
From: Chris Horn <chris.horn@hpe.com>
Date: Sat, 13 Jan 2024 11:06:10 -0600
Subject: [PATCH] LU-17475 tests: Do not pass IP to do_node in wait_nm_sync

If do_node() resolves to pdsh then the ':' in an IPv6 NID is
misinterpreted as specifying an rcmd module. Avoid the issue by
passing the node hostname instead of IP.

Test-Parameters: trivial
Signed-off-by: Chris Horn <chris.horn@hpe.com>
Change-Id: I511308e3fb5247a85dec7f20a0ff4f3da2de4f3a
Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/53838
Reviewed-by: James Simmons <jsimmons@infradead.org>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
---
 lustre/tests/test-framework.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lustre/tests/test-framework.sh b/lustre/tests/test-framework.sh
index 8317560..43339a8 100755
--- a/lustre/tests/test-framework.sh
+++ b/lustre/tests/test-framework.sh
@@ -11667,7 +11667,7 @@ wait_nm_sync() {
 				[ $node_ip == $mgs_ip ] && continue
 			fi
 
-			out2=$(do_node $node_ip $LCTL get_param $opt \
+			out2=$(do_node $node $LCTL get_param $opt \
 			       nodemap.$proc_param 2>/dev/null)
 			echo "On $node ${node_ip}, ${proc_param} = $out2"
 			[ "$out1" != "$out2" ] && is_sync=false && break
-- 
1.8.3.1