From: Chris Horn <chris.horn@hpe.com>
Date: Sat, 13 Jan 2024 17:06:10 +0000 (-0600)
Subject: LU-17475 tests: Do not pass IP to do_node in wait_nm_sync
X-Git-Tag: 2.15.61~8
X-Git-Url: https://git.whamcloud.com/?a=commitdiff_plain;h=refs%2Fchanges%2F38%2F53838%2F2;p=fs%2Flustre-release.git

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

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