diff options
author | Jeff Darcy <jdarcy@redhat.com> | 2015-01-05 09:39:07 -0500 |
---|---|---|
committer | Vijay Bellur <vbellur@redhat.com> | 2015-01-18 22:07:16 -0800 |
commit | 4d003c1b1b6dfd4ba94bb006b7ab6451fc62e170 (patch) | |
tree | e256e242eb747d0181a24b969598b7c67c5d6840 /tests/volume.rc | |
parent | 4f734b04694feabe047d758c2a0a6cd8ce5fc450 (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.rc | 2 |
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 } |