X-Git-Url: https://git.whamcloud.com/?a=blobdiff_plain;f=lustre%2Ftests%2Flarge-lun.sh;h=b184fadb2d93da6ce3d94c80cd3bd538bcecbbca;hb=99d7a8ed43be126b2769ad8bb0b5350cd328ed7f;hp=2a60f902c3d68d89031b80f95ac8e6fc629cbda8;hpb=935d49c9c19edb8a71d281bbbca2c8e801cb81af;p=fs%2Flustre-release.git

diff --git a/lustre/tests/large-lun.sh b/lustre/tests/large-lun.sh
index 2a60f90..b184fad 100644
--- a/lustre/tests/large-lun.sh
+++ b/lustre/tests/large-lun.sh
@@ -11,10 +11,9 @@ init_test_env $@
 . ${CONFIG:=$LUSTRE/tests/cfg/$NAME.sh}
 init_logging
 
-if ! [ "$REFORMAT" ]; then
+if [ "$REFORMAT" != "yes" ]; then
 	skip_env "$0 reformats all devices,\
 		please set REFORMAT to run this test"
-	exit 0
 fi
 
 # Variable to run mdsrate
@@ -150,7 +149,7 @@ trap cleanupall EXIT
 
 test_1 () {
 	[ $(facet_fstype $SINGLEMDS) != ldiskfs ] &&
-		skip "Only applicable to ldiskfs-based nodes" && return
+		skip_env "ldiskfs only test"
 	local dev
 	for num in $(seq $OSTCOUNT); do
 		dev=$(ostdevname $num)
@@ -240,10 +239,10 @@ test_2 () {
 run_test 2 "run llverfs on OST ldiskfs/zfs filesystem"
 
 test_3 () {
-	[ -z "$CLIENTS" ] && skip_env "CLIENTS not defined, skipping" && return
-	[ -z "$MPIRUN" ] && skip_env "MIPRUN not defined, skipping" && return
-	[ -z "$MDSRATE" ] && skip_env "MDSRATE not defined, skipping" && return
-	[ ! -x $MDSRATE ] && skip_env "$MDSRATE not built, skipping" && return
+	[ -z "$CLIENTS" ] && skip_env "CLIENTS not defined, skipping"
+	[ -z "$MPIRUN" ] && skip_env "MIPRUN not defined, skipping"
+	[ -z "$MDSRATE" ] && skip_env "MDSRATE not defined, skipping"
+	[ ! -x $MDSRATE ] && skip_env "$MDSRATE not built, skipping"
 	# Setup the Lustre filesystem.
 	log "setup the lustre filesystem"
 	REFORMAT="yes" check_and_setup_lustre