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