diff mbox series

btrfs/303: use quota rescan wrapper

Message ID 18afc03e67772666190415c742f0793e8207c5e6.1701464909.git.boris@bur.io (mailing list archive)
State New, archived
Headers show
Series btrfs/303: use quota rescan wrapper | expand

Commit Message

Boris Burkov Dec. 1, 2023, 9:09 p.m. UTC
This new test called quota rescan directly rather than with the new
wrapper. As a result, it failed with -O squota in MKFS_OPTIONS. Using
the wrapper, it skips the rescan and passes again.

Signed-off-by: Boris Burkov <boris@bur.io>
---
 tests/btrfs/303 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Filipe Manana Dec. 4, 2023, 1:08 p.m. UTC | #1
On Fri, Dec 1, 2023 at 9:08 PM Boris Burkov <boris@bur.io> wrote:
>
> This new test called quota rescan directly rather than with the new
> wrapper. As a result, it failed with -O squota in MKFS_OPTIONS. Using
> the wrapper, it skips the rescan and passes again.
>
> Signed-off-by: Boris Burkov <boris@bur.io>

Reviewed-by: Filipe Manana <fdmanana@suse.com>

Looks good, thanks.

> ---
>  tests/btrfs/303 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/btrfs/303 b/tests/btrfs/303
> index b9d6c61d1..410460af5 100755
> --- a/tests/btrfs/303
> +++ b/tests/btrfs/303
> @@ -43,7 +43,7 @@ _scratch_mkfs >> $seqres.full 2>&1 || _fail "2nd mkfs failed"
>  _scratch_mount
>
>  $BTRFS_UTIL_PROG quota enable "$SCRATCH_MNT" >> $seqres.full
> -$BTRFS_UTIL_PROG quota rescan -w "$SCRATCH_MNT" >> $seqres.full
> +_qgroup_rescan $SCRATCH_MNT >> $seqres.full
>
>  # Create a qgroup for the first subvolume, this would make the later
>  # subvolume creation to find an existing qgroup, and abort transaction.
> --
> 2.42.0
>
>
diff mbox series

Patch

diff --git a/tests/btrfs/303 b/tests/btrfs/303
index b9d6c61d1..410460af5 100755
--- a/tests/btrfs/303
+++ b/tests/btrfs/303
@@ -43,7 +43,7 @@  _scratch_mkfs >> $seqres.full 2>&1 || _fail "2nd mkfs failed"
 _scratch_mount
 
 $BTRFS_UTIL_PROG quota enable "$SCRATCH_MNT" >> $seqres.full
-$BTRFS_UTIL_PROG quota rescan -w "$SCRATCH_MNT" >> $seqres.full
+_qgroup_rescan $SCRATCH_MNT >> $seqres.full
 
 # Create a qgroup for the first subvolume, this would make the later
 # subvolume creation to find an existing qgroup, and abort transaction.