Whamcloud - gitweb
LU-12639 tests: initialize variable sanity 317 66/36366/2
authorJames Nunez <jnunez@whamcloud.com>
Wed, 7 Aug 2019 15:56:50 +0000 (09:56 -0600)
committerOleg Drokin <green@whamcloud.com>
Thu, 21 Nov 2019 07:31:42 +0000 (07:31 +0000)
sanity test 317 checks the file system type of $facet,
but facet is not initiaized in the test.

Replace the check of the facet file system type with the
variable ost1_FSTYPE.  The call to return after skip is
removed.

This is a backport to Lustre b2_12 of
Lustre-change: https://review.whamcloud.com/35716
Lustre-commit: c00be06a1f8f27eb5bd8bb47086d0f1e5b5f5f50

Fixes: 6115eb7fd55a ("LU-10370 ofd: truncate does not update blocks count on client")
Test-Parameters: trivial
Test-Parameters: fstype=zfs envdefinitions=ONLY=317 testlist=sanity

Signed-off-by: James Nunez <jnunez@whamcloud.com>
Change-Id: If67c4d786e4d23752effd1aaebc82bb1be8aceb5
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>
Reviewed-by: Arshad Hussain <arshad.super@gmail.com>
Reviewed-on: https://review.whamcloud.com/36366
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
lustre/tests/sanity.sh

index 97a4866..8118bec 100755 (executable)
@@ -18879,14 +18879,13 @@ run_test 316 "lfs mv"
 test_317() {
        [ $MDS1_VERSION -lt $(version_code 2.11.53) ] &&
                skip "Need MDS version at least 2.11.53"
+       if [ "$ost1_FSTYPE" == "zfs" ]; then
+               skip "LU-10370: no implementation for ZFS"
+       fi
+
        local trunc_sz
        local grant_blk_size
 
-       if [ "$(facet_fstype $facet)" == "zfs" ]; then
-               skip "LU-10370: no implementation for ZFS" && return
-       fi
-
-       stack_trap "rm -f $DIR/$tfile" EXIT
        grant_blk_size=$($LCTL get_param osc.$FSNAME*.import |
                        awk '/grant_block_size:/ { print $2; exit; }')
        #
@@ -18894,7 +18893,8 @@ test_317() {
        # blocks count.
        #
        dd if=/dev/zero of=$DIR/$tfile bs=5M count=1 conv=fsync ||
-               error "Create file : $DIR/$tfile"
+               error "Create file $DIR/$tfile failed"
+       stack_trap "rm -f $DIR/$tfile" EXIT
 
        for trunc_sz in 2097152 4097 4000 509 0; do
                $TRUNCATE $DIR/$tfile $trunc_sz ||