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 (executable)
--- 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