diff options
author | Naohiro Aota <naohiro.aota@wdc.com> | 2023-12-22 11:56:22 +0900 |
---|---|---|
committer | Zorro Lang <zlang@kernel.org> | 2023-12-24 03:14:21 +0800 |
commit | 17324dbc7743902aec349d12ccad91b3cb6005c5 (patch) | |
tree | 815680222e5988b9d3b2773967ca0d696e3338af | |
parent | d6e9d32578afc1da2dd398cc46018f0a51d7875c (diff) |
fstests: btrfs: use proper filter for subvolume deletionv2023.12.25
Test cases btrfs/208, 233, 276 does not use _filter_btrfs_subvol_delete()
to process "btrfs subvolume delete" command's output. So, the following
diff occurs even with a previous fix.
btrfs/208 - output mismatch (see /host/btrfs/208.out.bad)
--- tests/btrfs/208.out 2023-12-22 02:09:18.000000000 +0000
+++ /host/btrfs/208.out.bad 2023-12-22 02:21:40.697036486 +0000
@@ -6,12 +6,12 @@
subvol1
subvol2
subvol3
-Delete subvolume (no-commit): 'SCRATCH_MNT/subvol1'
+Delete subvolume 256 (no-commit): 'SCRATCH_MNT/subvol1'
After deleting one subvolume:
subvol2
...
Let them use the filter and fix the output accordingly.
Signed-off-by: Naohiro Aota <naohiro.aota@wdc.com>
Reviewed-by: David Disseldorp <ddiss@suse.de>
Signed-off-by: Zorro Lang <zlang@kernel.org>
-rwxr-xr-x | tests/btrfs/208 | 2 | ||||
-rw-r--r-- | tests/btrfs/208.out | 6 | ||||
-rwxr-xr-x | tests/btrfs/233 | 3 | ||||
-rw-r--r-- | tests/btrfs/233.out | 4 | ||||
-rwxr-xr-x | tests/btrfs/276 | 3 | ||||
-rw-r--r-- | tests/btrfs/276.out | 2 |
6 files changed, 11 insertions, 9 deletions
diff --git a/tests/btrfs/208 b/tests/btrfs/208 index 909f9fa4..d58803e2 100755 --- a/tests/btrfs/208 +++ b/tests/btrfs/208 @@ -28,7 +28,7 @@ _delete_and_list() local msg="$2" SUBVOLID=$(_btrfs_get_subvolid $SCRATCH_MNT "$subvol_name") - $BTRFS_UTIL_PROG subvolume delete --subvolid $SUBVOLID $SCRATCH_MNT | _filter_scratch + $BTRFS_UTIL_PROG subvolume delete --subvolid $SUBVOLID $SCRATCH_MNT | _filter_btrfs_subvol_delete echo "$msg" $BTRFS_UTIL_PROG subvolume list $SCRATCH_MNT | $AWK_PROG '{ print $NF }' diff --git a/tests/btrfs/208.out b/tests/btrfs/208.out index 9b660699..dc5761ba 100644 --- a/tests/btrfs/208.out +++ b/tests/btrfs/208.out @@ -6,12 +6,12 @@ Current subvolume ids: subvol1 subvol2 subvol3 -Delete subvolume (no-commit): 'SCRATCH_MNT/subvol1' +Delete subvolume 'SCRATCH_MNT/subvol1' After deleting one subvolume: subvol2 subvol3 -Delete subvolume (no-commit): 'SCRATCH_MNT/subvol3' +Delete subvolume 'SCRATCH_MNT/subvol3' Last remaining subvolume: subvol2 -Delete subvolume (no-commit): 'SCRATCH_MNT/subvol2' +Delete subvolume 'SCRATCH_MNT/subvol2' All subvolumes removed. diff --git a/tests/btrfs/233 b/tests/btrfs/233 index 2b94a9c6..f2c1eba0 100755 --- a/tests/btrfs/233 +++ b/tests/btrfs/233 @@ -21,6 +21,7 @@ _cleanup() # Import common functions. . ./common/filter +. ./common/filter.btrfs . ./common/dmflakey # real QA test starts here @@ -77,7 +78,7 @@ create_subvol_with_orphan() # open, delete the subvolume, then 'sync' to durably persist the orphan # item for the subvolume. exec 73< $SCRATCH_MNT/testsv/foobar - $BTRFS_UTIL_PROG subvolume delete $SCRATCH_MNT/testsv | _filter_scratch + $BTRFS_UTIL_PROG subvolume delete $SCRATCH_MNT/testsv | _filter_btrfs_subvol_delete sync # Now silently drop writes on the device, close the file descriptor and diff --git a/tests/btrfs/233.out b/tests/btrfs/233.out index 492e959d..2754e900 100644 --- a/tests/btrfs/233.out +++ b/tests/btrfs/233.out @@ -1,5 +1,5 @@ QA output created by 233 Create subvolume 'SCRATCH_MNT/testsv' -Delete subvolume (no-commit): 'SCRATCH_MNT/testsv' +Delete subvolume 'SCRATCH_MNT/testsv' Create subvolume 'SCRATCH_MNT/testsv' -Delete subvolume (no-commit): 'SCRATCH_MNT/testsv' +Delete subvolume 'SCRATCH_MNT/testsv' diff --git a/tests/btrfs/276 b/tests/btrfs/276 index 6470a2f6..f15f2082 100755 --- a/tests/btrfs/276 +++ b/tests/btrfs/276 @@ -12,6 +12,7 @@ _begin_fstest auto snapshot fiemap remount . ./common/filter +. ./common/filter.btrfs . ./common/attr _supported_fs btrfs @@ -130,7 +131,7 @@ echo "Number of non-shared extents in range [512K, 512K + 64K): $(count_not_shar echo "Number of non-shared extents in range [249M, 249M + 64K): $(count_not_shared_extents 249M 64K)" # Now delete the snapshot. -$BTRFS_UTIL_PROG subvolume delete -c $SCRATCH_MNT/snap | _filter_scratch +$BTRFS_UTIL_PROG subvolume delete -c $SCRATCH_MNT/snap | _filter_btrfs_subvol_delete # We deleted the snapshot and committed the transaction used to delete it (-c), # but all its extents (both metadata and data) are actually only deleted in the diff --git a/tests/btrfs/276.out b/tests/btrfs/276.out index 197d8edc..352e06b4 100644 --- a/tests/btrfs/276.out +++ b/tests/btrfs/276.out @@ -10,5 +10,5 @@ Number of non-shared extents in the whole file: 2 Number of shared extents in the whole file: 1998 Number of non-shared extents in range [512K, 512K + 64K): 1 Number of non-shared extents in range [249M, 249M + 64K): 1 -Delete subvolume (commit): 'SCRATCH_MNT/snap' +Delete subvolume 'SCRATCH_MNT/snap' Number of non-shared extents in the whole file: 2000 |