From: John L. Hammond <jhammond@whamcloud.com>
Date: Fri, 11 Mar 2022 15:14:52 +0000 (-0600)
Subject: EX-4866 lipe: don't unmount an empty client list
X-Git-Url: https://git.whamcloud.com/gitweb?a=commitdiff_plain;h=8b2e4e7e89c0ca2c7236677b1faa87e2e4893f58;p=fs%2Flustre-release.git

EX-4866 lipe: don't unmount an empty client list

In hot-pools.sh, if the node list is empty then don't try to unmount
it since that will only confuse things.

Fixes: e1da905b3884 EX-4866 lipe: don't unmount the local client
Test-Parameters: trivial testlist=hot-pools
Signed-off-by: John L. Hammond <jhammond@whamcloud.com>
Change-Id: I1bf057beffd025a549524e85f02609be9611cccc
Reviewed-on: https://review.whamcloud.com/46800
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Jian Yu <yujian@whamcloud.com>
Reviewed-by: Alexandre Ioffe <aioffe@ddn.com>
---

diff --git a/lustre/tests/hot-pools.sh b/lustre/tests/hot-pools.sh
index 628c3b0..62fff2e 100755
--- a/lustre/tests/hot-pools.sh
+++ b/lustre/tests/hot-pools.sh
@@ -173,7 +173,9 @@ init_hot_pools_env() {
 	# mount Lustre clients on server nodes
 	zconf_mount_clients $(comma_list $(all_server_nodes)) $MOUNT ||
 		error "failed to mount Lustre clients on server nodes"
-	stack_trap "zconf_umount_clients $umount_nodes $MOUNT"
+	if [[ -n "$umount_nodes" ]]; then
+		stack_trap "zconf_umount_clients $umount_nodes $MOUNT"
+	fi
 
 	# enable changelog on MDT(s)
 	if ${INIT_HOT_POOLS_CHANGELOG:-true}; then