Whamcloud - gitweb
LU-7301 tests: delete old lfsck tests
[fs/lustre-release.git] / lustre / tests / sanity.sh
index a66f35f..8a689af 100755 (executable)
@@ -556,11 +556,7 @@ run_test 17l "Ensure lgetxattr's returned xattr size is consistent ========"
 test_17m() {
        local short_sym="0123456789"
        local WDIR=$DIR/${tdir}m
-       local mds_index
-       local devname
-       local cmd
        local i
-       local rc=0
 
        remote_mds_nodsh && skip "remote MDS with nodsh" && return
        [ $(lustre_version_code $SINGLEMDS) -ge $(version_code 2.2.0) ] &&
@@ -593,49 +589,42 @@ test_17m() {
        echo "recreate the 512 symlink files with a shorter string"
        for ((i = 0; i < 512; ++i)); do
                # rewrite the symlink file with a shorter string
-               ln -sf ${long_sym} $WDIR/long-$i
-               ln -sf ${short_sym} $WDIR/short-$i
+               ln -sf ${long_sym} $WDIR/long-$i || error "long_sym failed"
+               ln -sf ${short_sym} $WDIR/short-$i || error "short_sym failed"
        done
 
-       mds_index=$($LFS getstripe -M $WDIR)
-       mds_index=$((mds_index+1))
-       devname=$(mdsdevname $mds_index)
-       cmd="$E2FSCK -fnvd $devname"
+       local mds_index=$(($($LFS getstripe -M $WDIR) + 1))
+       local devname=$(mdsdevname $mds_index)
 
-       echo "stop and checking mds${mds_index}: $cmd"
+       echo "stop and checking mds${mds_index}:"
        # e2fsck should not return error
        stop mds${mds_index}
-       do_facet mds${mds_index} $cmd || rc=$?
+       run_e2fsck $(facet_active_host mds${mds_index}) $devname -n
+       rc=$?
 
        start mds${mds_index} $devname $MDS_MOUNT_OPTS || error "start failed"
        df $MOUNT > /dev/null 2>&1
-       [ $rc -ne 0 ] && error "e2fsck should not report error upon "\
-               "short/long symlink MDT: rc=$rc"
-       return $rc
+       [ $rc -eq 0 ] ||
+               error "e2fsck detected error for short/long symlink: rc=$rc"
 }
 run_test 17m "run e2fsck against MDT which contains short/long symlink"
 
 check_fs_consistency_17n() {
        local mdt_index
-       local devname
-       local cmd
        local rc=0
 
        # create/unlink in 17n only change 2 MDTs(MDT1/MDT2),
        # so it only check MDT1/MDT2 instead of all of MDTs.
-       for mdt_index in $(seq 1 2); do
-               devname=$(mdsdevname $mdt_index)
-               cmd="$E2FSCK -fnvd $devname"
-
-               echo "stop and checking mds${mdt_index}: $cmd"
+       for mdt_index in 1 2; do
+               local devname=$(mdsdevname $mdt_index)
                # e2fsck should not return error
                stop mds${mdt_index}
-               do_facet mds${mdt_index} $cmd || rc=$?
+               run_e2fsck $(facet_active_host mds$mdt_index) $devname -n ||
+                       rc=$((rc + $?))
 
                start mds${mdt_index} $devname $MDS_MOUNT_OPTS ||
-                       error "mount mds${mdt_index} failed"
+                       error "mount mds$mdt_index failed"
                df $MOUNT > /dev/null 2>&1
-               [ $rc -ne 0 ] && break
        done
        return $rc
 }