summaryrefslogtreecommitdiffstats
path: root/tests/volume.rc
diff options
context:
space:
mode:
authorJeff Darcy <jdarcy@redhat.com>2015-01-05 09:39:07 -0500
committerVijay Bellur <vbellur@redhat.com>2015-01-18 22:07:16 -0800
commit4d003c1b1b6dfd4ba94bb006b7ab6451fc62e170 (patch)
treee256e242eb747d0181a24b969598b7c67c5d6840 /tests/volume.rc
parent4f734b04694feabe047d758c2a0a6cd8ce5fc450 (diff)
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 <jdarcy@redhat.com> Reviewed-on: http://review.gluster.org/9386 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Vijay Bellur <vbellur@redhat.com>
Diffstat (limited to 'tests/volume.rc')
-rw-r--r--tests/volume.rc2
1 files changed, 1 insertions, 1 deletions
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
}