From 8b2e4e7e89c0ca2c7236677b1faa87e2e4893f58 Mon Sep 17 00:00:00 2001 From: "John L. Hammond" Date: Fri, 11 Mar 2022 09:14:52 -0600 Subject: [PATCH] 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 Change-Id: I1bf057beffd025a549524e85f02609be9611cccc Reviewed-on: https://review.whamcloud.com/46800 Tested-by: jenkins Tested-by: Maloo Reviewed-by: Jian Yu Reviewed-by: Alexandre Ioffe --- lustre/tests/hot-pools.sh | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) 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 -- 1.8.3.1