From 4d003c1b1b6dfd4ba94bb006b7ab6451fc62e170 Mon Sep 17 00:00:00 2001 From: Jeff Darcy Date: Mon, 5 Jan 2015 09:39:07 -0500 Subject: tests: let force_umount work on multiple items Some scripts (e.g. features/weighted-rebalance.t) try to unmount multiple mountpoints at once, using UMOUNT_LOOP. This dutifully passes the $* list through to force_umount, which (prior to this fix) would only unmount $1 instead of the whole set. This would leave those devices mounted, which would not only be a resource leak itself but would cause other cleanup actions to fail. Change-Id: I2e3379c85792765025540f10be7cb37b8a4c1bcf Signed-off-by: Jeff Darcy Reviewed-on: http://review.gluster.org/9386 Tested-by: Gluster Build System Reviewed-by: Vijay Bellur --- tests/volume.rc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'tests') diff --git a/tests/volume.rc b/tests/volume.rc index 4712179efd5..daa9c140fd8 100644 --- a/tests/volume.rc +++ b/tests/volume.rc @@ -412,7 +412,7 @@ function path_exists { } function force_umount { - umount -f $1 + umount -f $* if [ $? -eq 0 ]; then echo "Y"; else echo "N"; fi } -- cgit