[PATCH] btrfs/099: Fix qgroupid issue

[PATCH] btrfs/099: Fix qgroupid issue

[An Long]

The qgroup commands now accepts the level/id format. Use 0/subvolid to
avoid invalid qgroupid error.

Signed-off-by: An Long <lan@suse.com>
---
 tests/btrfs/099 | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/btrfs/099 b/tests/btrfs/099
index 375cc2b9..f3a2002a 100755
--- a/tests/btrfs/099
+++ b/tests/btrfs/099
@@ -29,7 +29,7 @@ _scratch_mount
 _require_fs_space $SCRATCH_MNT $(($FILESIZE * 2 / 1024))
 
 _run_btrfs_util_prog quota enable $SCRATCH_MNT
-_run_btrfs_util_prog qgroup limit $FILESIZE 5 $SCRATCH_MNT
+_run_btrfs_util_prog qgroup limit $FILESIZE 0/5 $SCRATCH_MNT
 
 # loop 5 times without sync to ensure reserved space leak will happen
 for i in `seq 1 5`; do
-- 
2.31.1

Re: [PATCH] btrfs/099: Fix qgroupid issue

[Eryu Guan]

On Wed, Dec 01, 2021 at 10:46:10PM +0800, An Long wrote:
> The qgroup commands now accepts the level/id format. Use 0/subvolid to
> avoid invalid qgroupid error.
> 
> Signed-off-by: An Long <lan@suse.com>

Thanks for the fix! But this has been fixed by commit e4746f0dcd1e
("btrfs/099: use the qgroupid for qgroup limit"), would you please pull
the latest code and check if that fixes the issue for you?

Thanks,
Eryu

P.S.

Please also cc btrfs list in future for btrfs-specific patches.
Similarly for patches for other filesystems.

> ---
>  tests/btrfs/099 | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/btrfs/099 b/tests/btrfs/099
> index 375cc2b9..f3a2002a 100755
> --- a/tests/btrfs/099
> +++ b/tests/btrfs/099
> @@ -29,7 +29,7 @@ _scratch_mount
>  _require_fs_space $SCRATCH_MNT $(($FILESIZE * 2 / 1024))
>  
>  _run_btrfs_util_prog quota enable $SCRATCH_MNT
> -_run_btrfs_util_prog qgroup limit $FILESIZE 5 $SCRATCH_MNT
> +_run_btrfs_util_prog qgroup limit $FILESIZE 0/5 $SCRATCH_MNT
>  
>  # loop 5 times without sync to ensure reserved space leak will happen
>  for i in `seq 1 5`; do
> -- 
> 2.31.1