]> git.apps.os.sepia.ceph.com Git - xfstests-dev.git/commitdiff
btrfs/320: skip -O squota runs
authorBoris Burkov <boris@bur.io>
Wed, 13 Mar 2024 19:51:17 +0000 (19:51 +0000)
committerAnand Jain <anand.jain@oracle.com>
Wed, 20 Mar 2024 14:32:52 +0000 (22:32 +0800)
This test makes assumptions about the shared usage under snapshots which
are not valid when using squotas. Skip squotas for this test.

Also, make it use the rescan wrapper, just for uniformity and since it
doesn't hurt.

Reviewed-by: Anand Jain <anand.jain@oracle.com>
Signed-off-by: Boris Burkov <boris@bur.io>
Signed-off-by: Anand Jain <anand.jain@oracle.com>
[ added _require_qgroup_rescan ]

tests/btrfs/320

index 408053457aba61b1f02bc790c236bdd8f9375de5..df7acdbb3deb0cb870acd65dd154e2311fe45fd1 100755 (executable)
@@ -15,7 +15,9 @@ _begin_fstest auto qgroup limit
 
 _supported_fs btrfs
 _require_scratch
+_require_qgroup_rescan
 _require_btrfs_qgroup_report
+_require_scratch_qgroup
 
 # Test to make sure we can actually turn it on and it makes sense
 _basic_test()
@@ -23,7 +25,7 @@ _basic_test()
        echo "=== basic test ===" >> $seqres.full
        _run_btrfs_util_prog subvolume create $SCRATCH_MNT/a
        _run_btrfs_util_prog quota enable $SCRATCH_MNT/a
-       _run_btrfs_util_prog quota rescan -w $SCRATCH_MNT
+       _qgroup_rescan $SCRATCH_MNT
        subvolid=$(_btrfs_get_subvolid $SCRATCH_MNT a)
        $BTRFS_UTIL_PROG qgroup show $units $SCRATCH_MNT | grep $subvolid >> \
                $seqres.full 2>&1
@@ -62,7 +64,7 @@ _rescan_test()
        echo "qgroup values before rescan: $output" >> $seqres.full
        refer=$(echo $output | $AWK_PROG '{ print $2 }')
        excl=$(echo $output | $AWK_PROG '{ print $3 }')
-       _run_btrfs_util_prog quota rescan -w $SCRATCH_MNT
+       _qgroup_rescan $SCRATCH_MNT
        output=$($BTRFS_UTIL_PROG qgroup show $units $SCRATCH_MNT | grep "0/$subvolid")
        echo "qgroup values after rescan: $output" >> $seqres.full
        [ $refer -eq $(echo $output | $AWK_PROG '{ print $2 }') ] || \