Thread (2 messages) 2 messages, 2 authors, 2021-11-22

Re: [PATCH] fstests: btrfs/099: use the qgroupid for qgroup limit

From: Qu Wenruo <hidden>
Date: 2021-11-22 23:32:47
Also in: fstests


On 2021/11/23 04:50, Josef Bacik wrote:
A change to btrfs-progs uncovered a problem with btrfs/099, we weren't
specifying the qgroupid with the subvol id.  This technically worked
before but only by accident, and all other tests properly specify the
qgroupid for qgroup limit commands.  Fix this test to specify the
qgroupid, which will work with older versions of btrfs-progs and newer
ones as well.

Signed-off-by: Josef Bacik <josef@toxicpanda.com>
Reviewed-by: Qu Wenruo <redacted>

Thanks,
Qu
quoted hunk ↗ jump to hunk
---
  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
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help