| Message ID | 595829c06353ab12280513ab880bb742a2389bbe.1637614211.git.josef@toxicpanda.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | fstests: btrfs/099: use the qgroupid for qgroup limit | expand |
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 <wqu@suse.com> Thanks, Qu > --- > 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 >
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
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> --- tests/btrfs/099 | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)