| Message ID | 161896457076.776366.1740320523459442249.stgit@magnolia (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | fstests: add inode btree blocks counters to the AGI header | expand |
On Tue, Apr 20, 2021 at 05:22:50PM -0700, Darrick J. Wong wrote: > From: Darrick J. Wong <djwong@kernel.org> > > Currently, the only functional testing for xfs_admin is xfs/287, which > checks that one can add 32-bit project ids to a V4 filesystem. This > obviously isn't an exhaustive test of all the CLI arguments, and > historically there have been xfs configurations that don't even work. > > Therefore, introduce a couple of new tests -- one that will test the > simple options with the default configuration, and a second test that > steps a bit outside of the test run configuration to make sure that we > do the right thing for external devices. The second test already caught > a nasty bug in xfsprogs 5.11. > > Signed-off-by: Darrick J. Wong <djwong@kernel.org> > --- > common/xfs | 21 ++++++++++ > tests/xfs/764 | 93 +++++++++++++++++++++++++++++++++++++++++++ > tests/xfs/764.out | 17 ++++++++ > tests/xfs/773 | 114 +++++++++++++++++++++++++++++++++++++++++++++++++++++ > tests/xfs/773.out | 19 +++++++++ > tests/xfs/group | 2 + > 6 files changed, 266 insertions(+) > create mode 100755 tests/xfs/764 > create mode 100644 tests/xfs/764.out > create mode 100755 tests/xfs/773 > create mode 100644 tests/xfs/773.out > > ... > diff --git a/tests/xfs/773 b/tests/xfs/773 > new file mode 100755 > index 00000000..f184962a > --- /dev/null > +++ b/tests/xfs/773 > @@ -0,0 +1,114 @@ > +#! /bin/bash > +# SPDX-License-Identifier: GPL-2.0-or-later > +# Copyright (c) 2021 Oracle. All Rights Reserved. > +# > +# FS QA Test No. 773 > +# > +# Functional testing for xfs_admin to ensure that it parses arguments correctly > +# with regards to data devices that are files, external logs, and realtime > +# devices. > +# > +# Because this test synthesizes log and rt devices (by modifying the test run > +# configuration), it does /not/ require the ability to mount the scratch > +# filesystem. This increases test coverage while isolating the weird bits to a > +# single test. > +# > +# This is partially a regression test for "xfs_admin: pick up log arguments > +# correctly", insofar as the issue fixed by that patch was discovered with an > +# earlier revision of this test. > + > +seq=`basename $0` > +seqres=$RESULT_DIR/$seq > +echo "QA output created by $seq" > + > +here=`pwd` > +tmp=/tmp/$$ > +status=1 # failure is the default! > +trap "_cleanup; exit \$status" 0 1 2 3 15 > + > +_cleanup() > +{ > + cd / > + rm -f $tmp.* $fake_logfile $fake_rtfile $fake_datafile > +} > + > +# get standard environment, filters and checks > +. ./common/rc > +. ./common/filter > + > +# real QA test starts here > +_supported_fs xfs > +_require_test > +_require_scratch_nocheck > +_require_command "$XFS_ADMIN_PROG" "xfs_admin" > + > +rm -f $seqres.full > + > +# Create some fake sparse files for testing external devices and whatnot > +fake_datafile=$TEST_DIR/scratch.data > +rm -f $fake_datafile > +truncate -s 500m $fake_datafile > + > +fake_logfile=$TEST_DIR/scratch.log > +rm -f $fake_logfile > +truncate -s 500m $fake_logfile > + > +fake_rtfile=$TEST_DIR/scratch.rt > +rm -f $fake_rtfile > +truncate -s 500m $fake_rtfile > + I think it's usually good practice to incorporate $seq into filenames created on the test device. > +# Save the original variables > +orig_ddev=$SCRATCH_DEV > +orig_external=$USE_EXTERNAL > +orig_logdev=$SCRATCH_LOGDEV > +orig_rtdev=$SCRATCH_RTDEV > + > +scenario() { > + echo "$@" | tee -a $seqres.full > + > + SCRATCH_DEV=$orig_ddev > + USE_EXTERNAL=$orig_external > + SCRATCH_LOGDEV=$orig_logdev > + SCRATCH_RTDEV=$orig_rtdev > +} > + > +check_label() { > + _scratch_mkfs -L oldlabel >> $seqres.full > + _scratch_xfs_db -c label > + _scratch_xfs_admin -L newlabel "$@" >> $seqres.full > + _scratch_xfs_db -c label > + _scratch_xfs_repair -n &>> $seqres.full || echo "Check failed?" > +} > + > +scenario "S1: Check that label setting with file image" > +SCRATCH_DEV=$fake_datafile > +check_label -f > + > +scenario "S2: Check that setting with logdev works" > +USE_EXTERNAL=yes > +SCRATCH_LOGDEV=$fake_logfile > +check_label > + > +scenario "S3: Check that setting with rtdev works" > +USE_EXTERNAL=yes > +SCRATCH_RTDEV=$fake_rtfile > +check_label > + > +scenario "S4: Check that setting with rtdev + logdev works" > +USE_EXTERNAL=yes > +SCRATCH_LOGDEV=$fake_logfile > +SCRATCH_RTDEV=$fake_rtfile > +check_label > + > +scenario "S5: Check that setting with nortdev + nologdev works" > +USE_EXTERNAL= > +SCRATCH_LOGDEV= > +SCRATCH_RTDEV= > +check_label > + > +scenario "S6: Check that setting with bdev incorrectly flagged as file works" > +check_label -f > + > +# success, all done > +status=0 > +exit > diff --git a/tests/xfs/773.out b/tests/xfs/773.out > new file mode 100644 > index 00000000..954bfb85 > --- /dev/null > +++ b/tests/xfs/773.out > @@ -0,0 +1,19 @@ > +QA output created by 773 > +S1: Check that label setting with file image > +label = "oldlabel" > +label = "newlabel" > +S2: Check that setting with logdev works > +label = "oldlabel" > +label = "newlabel" > +S3: Check that setting with rtdev works > +label = "oldlabel" > +label = "newlabel" > +S4: Check that setting with rtdev + logdev works > +label = "oldlabel" > +label = "newlabel" > +S5: Check that setting with nortdev + nologdev works > +label = "oldlabel" > +label = "newlabel" > +S6: Check that setting with bdev incorrectly flagged as file works > +label = "oldlabel" > +label = "newlabel" > diff --git a/tests/xfs/group b/tests/xfs/group > index 461ae2b2..a2309465 100644 > --- a/tests/xfs/group > +++ b/tests/xfs/group > @@ -522,5 +522,7 @@ > 537 auto quick > 538 auto stress > 539 auto quick mount > +764 auto quick repair > 768 auto quick repair > 770 auto repair > +773 auto quick repair > Both of these tests are primarily targeted at xfs_admin, right? If so, I'm not sure the repair group makes much sense. With the nits fixed up: Reviewed-by: Brian Foster <bfoster@redhat.com>
On Wed, Apr 21, 2021 at 02:00:38PM -0400, Brian Foster wrote: > On Tue, Apr 20, 2021 at 05:22:50PM -0700, Darrick J. Wong wrote: > > From: Darrick J. Wong <djwong@kernel.org> > > > > Currently, the only functional testing for xfs_admin is xfs/287, which > > checks that one can add 32-bit project ids to a V4 filesystem. This > > obviously isn't an exhaustive test of all the CLI arguments, and > > historically there have been xfs configurations that don't even work. > > > > Therefore, introduce a couple of new tests -- one that will test the > > simple options with the default configuration, and a second test that > > steps a bit outside of the test run configuration to make sure that we > > do the right thing for external devices. The second test already caught > > a nasty bug in xfsprogs 5.11. > > > > Signed-off-by: Darrick J. Wong <djwong@kernel.org> > > --- > > common/xfs | 21 ++++++++++ > > tests/xfs/764 | 93 +++++++++++++++++++++++++++++++++++++++++++ > > tests/xfs/764.out | 17 ++++++++ > > tests/xfs/773 | 114 +++++++++++++++++++++++++++++++++++++++++++++++++++++ > > tests/xfs/773.out | 19 +++++++++ > > tests/xfs/group | 2 + > > 6 files changed, 266 insertions(+) > > create mode 100755 tests/xfs/764 > > create mode 100644 tests/xfs/764.out > > create mode 100755 tests/xfs/773 > > create mode 100644 tests/xfs/773.out > > > > > ... > > diff --git a/tests/xfs/773 b/tests/xfs/773 > > new file mode 100755 > > index 00000000..f184962a > > --- /dev/null > > +++ b/tests/xfs/773 > > @@ -0,0 +1,114 @@ > > +#! /bin/bash > > +# SPDX-License-Identifier: GPL-2.0-or-later > > +# Copyright (c) 2021 Oracle. All Rights Reserved. > > +# > > +# FS QA Test No. 773 > > +# > > +# Functional testing for xfs_admin to ensure that it parses arguments correctly > > +# with regards to data devices that are files, external logs, and realtime > > +# devices. > > +# > > +# Because this test synthesizes log and rt devices (by modifying the test run > > +# configuration), it does /not/ require the ability to mount the scratch > > +# filesystem. This increases test coverage while isolating the weird bits to a > > +# single test. > > +# > > +# This is partially a regression test for "xfs_admin: pick up log arguments > > +# correctly", insofar as the issue fixed by that patch was discovered with an > > +# earlier revision of this test. > > + > > +seq=`basename $0` > > +seqres=$RESULT_DIR/$seq > > +echo "QA output created by $seq" > > + > > +here=`pwd` > > +tmp=/tmp/$$ > > +status=1 # failure is the default! > > +trap "_cleanup; exit \$status" 0 1 2 3 15 > > + > > +_cleanup() > > +{ > > + cd / > > + rm -f $tmp.* $fake_logfile $fake_rtfile $fake_datafile > > +} > > + > > +# get standard environment, filters and checks > > +. ./common/rc > > +. ./common/filter > > + > > +# real QA test starts here > > +_supported_fs xfs > > +_require_test > > +_require_scratch_nocheck > > +_require_command "$XFS_ADMIN_PROG" "xfs_admin" > > + > > +rm -f $seqres.full > > + > > +# Create some fake sparse files for testing external devices and whatnot > > +fake_datafile=$TEST_DIR/scratch.data > > +rm -f $fake_datafile > > +truncate -s 500m $fake_datafile > > + > > +fake_logfile=$TEST_DIR/scratch.log > > +rm -f $fake_logfile > > +truncate -s 500m $fake_logfile > > + > > +fake_rtfile=$TEST_DIR/scratch.rt > > +rm -f $fake_rtfile > > +truncate -s 500m $fake_rtfile > > + > > I think it's usually good practice to incorporate $seq into filenames > created on the test device. Ok fixed. > > +# Save the original variables > > +orig_ddev=$SCRATCH_DEV > > +orig_external=$USE_EXTERNAL > > +orig_logdev=$SCRATCH_LOGDEV > > +orig_rtdev=$SCRATCH_RTDEV > > + > > +scenario() { > > + echo "$@" | tee -a $seqres.full > > + > > + SCRATCH_DEV=$orig_ddev > > + USE_EXTERNAL=$orig_external > > + SCRATCH_LOGDEV=$orig_logdev > > + SCRATCH_RTDEV=$orig_rtdev > > +} > > + > > +check_label() { > > + _scratch_mkfs -L oldlabel >> $seqres.full > > + _scratch_xfs_db -c label > > + _scratch_xfs_admin -L newlabel "$@" >> $seqres.full > > + _scratch_xfs_db -c label > > + _scratch_xfs_repair -n &>> $seqres.full || echo "Check failed?" > > +} > > + > > +scenario "S1: Check that label setting with file image" > > +SCRATCH_DEV=$fake_datafile > > +check_label -f > > + > > +scenario "S2: Check that setting with logdev works" > > +USE_EXTERNAL=yes > > +SCRATCH_LOGDEV=$fake_logfile > > +check_label > > + > > +scenario "S3: Check that setting with rtdev works" > > +USE_EXTERNAL=yes > > +SCRATCH_RTDEV=$fake_rtfile > > +check_label > > + > > +scenario "S4: Check that setting with rtdev + logdev works" > > +USE_EXTERNAL=yes > > +SCRATCH_LOGDEV=$fake_logfile > > +SCRATCH_RTDEV=$fake_rtfile > > +check_label > > + > > +scenario "S5: Check that setting with nortdev + nologdev works" > > +USE_EXTERNAL= > > +SCRATCH_LOGDEV= > > +SCRATCH_RTDEV= > > +check_label > > + > > +scenario "S6: Check that setting with bdev incorrectly flagged as file works" > > +check_label -f > > + > > +# success, all done > > +status=0 > > +exit > > diff --git a/tests/xfs/773.out b/tests/xfs/773.out > > new file mode 100644 > > index 00000000..954bfb85 > > --- /dev/null > > +++ b/tests/xfs/773.out > > @@ -0,0 +1,19 @@ > > +QA output created by 773 > > +S1: Check that label setting with file image > > +label = "oldlabel" > > +label = "newlabel" > > +S2: Check that setting with logdev works > > +label = "oldlabel" > > +label = "newlabel" > > +S3: Check that setting with rtdev works > > +label = "oldlabel" > > +label = "newlabel" > > +S4: Check that setting with rtdev + logdev works > > +label = "oldlabel" > > +label = "newlabel" > > +S5: Check that setting with nortdev + nologdev works > > +label = "oldlabel" > > +label = "newlabel" > > +S6: Check that setting with bdev incorrectly flagged as file works > > +label = "oldlabel" > > +label = "newlabel" > > diff --git a/tests/xfs/group b/tests/xfs/group > > index 461ae2b2..a2309465 100644 > > --- a/tests/xfs/group > > +++ b/tests/xfs/group > > @@ -522,5 +522,7 @@ > > 537 auto quick > > 538 auto stress > > 539 auto quick mount > > +764 auto quick repair > > 768 auto quick repair > > 770 auto repair > > +773 auto quick repair > > > > Both of these tests are primarily targeted at xfs_admin, right? If so, > I'm not sure the repair group makes much sense. With the nits fixed up: > > Reviewed-by: Brian Foster <bfoster@redhat.com> Will fix the groups; thanks for the review! --D
On 4/20/21 5:22 PM, Darrick J. Wong wrote: > From: Darrick J. Wong <djwong@kernel.org> > > Currently, the only functional testing for xfs_admin is xfs/287, which > checks that one can add 32-bit project ids to a V4 filesystem. This > obviously isn't an exhaustive test of all the CLI arguments, and > historically there have been xfs configurations that don't even work. > > Therefore, introduce a couple of new tests -- one that will test the > simple options with the default configuration, and a second test that > steps a bit outside of the test run configuration to make sure that we > do the right thing for external devices. The second test already caught > a nasty bug in xfsprogs 5.11. > > Signed-off-by: Darrick J. Wong <djwong@kernel.org> Ok, I followed it through and I think it makes sense. With Brians comments fixed: Reviewed-by: Allison Henderson <allison.henderson@oracle.com> > --- > common/xfs | 21 ++++++++++ > tests/xfs/764 | 93 +++++++++++++++++++++++++++++++++++++++++++ > tests/xfs/764.out | 17 ++++++++ > tests/xfs/773 | 114 +++++++++++++++++++++++++++++++++++++++++++++++++++++ > tests/xfs/773.out | 19 +++++++++ > tests/xfs/group | 2 + > 6 files changed, 266 insertions(+) > create mode 100755 tests/xfs/764 > create mode 100644 tests/xfs/764.out > create mode 100755 tests/xfs/773 > create mode 100644 tests/xfs/773.out > > > diff --git a/common/xfs b/common/xfs > index c2384146..5abc7034 100644 > --- a/common/xfs > +++ b/common/xfs > @@ -1142,3 +1142,24 @@ _check_scratch_xfs_features() > echo "${output[@]}" > test "${found}" -eq "$#" > } > + > +# Decide if xfs_repair knows how to set (or clear) a filesystem feature. > +_require_xfs_repair_upgrade() > +{ > + local type="$1" > + > + $XFS_REPAIR_PROG -c "$type=garbagevalue" 2>&1 | \ > + grep -q 'unknown option' && \ > + _notrun "xfs_repair does not support upgrading fs with $type" > +} > + > +_require_xfs_scratch_inobtcount() > +{ > + _require_scratch > + > + _scratch_mkfs -m inobtcount=1 &> /dev/null || \ > + _notrun "mkfs.xfs doesn't have inobtcount feature" > + _try_scratch_mount || \ > + _notrun "inobtcount not supported by scratch filesystem type: $FSTYP" > + _scratch_unmount > +} > diff --git a/tests/xfs/764 b/tests/xfs/764 > new file mode 100755 > index 00000000..ebdf8883 > --- /dev/null > +++ b/tests/xfs/764 > @@ -0,0 +1,93 @@ > +#! /bin/bash > +# SPDX-License-Identifier: GPL-2.0-or-later > +# Copyright (c) 2021 Oracle. All Rights Reserved. > +# > +# FS QA Test No. 764 > +# > +# Functional testing for xfs_admin to make sure that it handles option parsing > +# correctly for functionality that's relevant to V5 filesystems. It doesn't > +# test the options that apply only to V4 filesystems because that disk format > +# is deprecated. > + > +seq=`basename $0` > +seqres=$RESULT_DIR/$seq > +echo "QA output created by $seq" > + > +here=`pwd` > +tmp=/tmp/$$ > +status=1 # failure is the default! > +trap "_cleanup; exit \$status" 0 1 2 3 15 > + > +_cleanup() > +{ > + cd / > + rm -f $tmp.* > +} > + > +# get standard environment, filters and checks > +. ./common/rc > +. ./common/filter > + > +# real QA test starts here > +_supported_fs xfs > +_require_scratch > +_require_command "$XFS_ADMIN_PROG" "xfs_admin" > + > +rm -f $seqres.full > + > +note() { > + echo "$@" | tee -a $seqres.full > +} > + > +note "S0: Initialize filesystem" > +_scratch_mkfs -L origlabel -m uuid=babababa-baba-baba-baba-babababababa >> $seqres.full > +_scratch_xfs_db -c label -c uuid > +_scratch_xfs_repair -n &>> $seqres.full || echo "Check failed?" > + > +note "S1: Set a filesystem label" > +_scratch_xfs_admin -L newlabel >> $seqres.full > +_scratch_xfs_db -c label > +_scratch_xfs_repair -n &>> $seqres.full || echo "Check failed?" > + > +note "S2: Clear filesystem label" > +_scratch_xfs_admin -L -- >> $seqres.full > +_scratch_xfs_db -c label > +_scratch_xfs_repair -n &>> $seqres.full || echo "Check failed?" > + > +note "S3: Try to set oversized label" > +_scratch_xfs_admin -L thisismuchtoolongforxfstohandle >> $seqres.full > +_scratch_xfs_db -c label > +_scratch_xfs_repair -n &>> $seqres.full || echo "Check failed?" > + > +note "S4: Set filesystem UUID" > +_scratch_xfs_admin -U deaddead-dead-dead-dead-deaddeaddead >> $seqres.full > +_scratch_xfs_db -c uuid > +_scratch_xfs_repair -n &>> $seqres.full || echo "Check failed?" > + > +note "S5: Zero out filesystem UUID" > +_scratch_xfs_admin -U nil >> $seqres.full > +_scratch_xfs_db -c uuid > +_scratch_xfs_repair -n &>> $seqres.full || echo "Check failed?" > + > +note "S6: Randomize filesystem UUID" > +old_uuid="$(_scratch_xfs_db -c uuid)" > +_scratch_xfs_admin -U generate >> $seqres.full > +new_uuid="$(_scratch_xfs_db -c uuid)" > +if [ "$new_uuid" = "$old_uuid" ]; then > + echo "UUID randomization failed? $old_uuid == $new_uuid" > +fi > +_scratch_xfs_repair -n &>> $seqres.full || echo "Check failed?" > + > +note "S7: Restore original filesystem UUID" > +if _check_scratch_xfs_features V5 >/dev/null; then > + # Only V5 supports the metauuid feature that enables us to restore the > + # original UUID after a change. > + _scratch_xfs_admin -U restore >> $seqres.full > + _scratch_xfs_db -c uuid > +else > + echo "UUID = babababa-baba-baba-baba-babababababa" > +fi > + > +# success, all done > +status=0 > +exit > diff --git a/tests/xfs/764.out b/tests/xfs/764.out > new file mode 100644 > index 00000000..8da929ec > --- /dev/null > +++ b/tests/xfs/764.out > @@ -0,0 +1,17 @@ > +QA output created by 764 > +S0: Initialize filesystem > +label = "origlabel" > +UUID = babababa-baba-baba-baba-babababababa > +S1: Set a filesystem label > +label = "newlabel" > +S2: Clear filesystem label > +label = "" > +S3: Try to set oversized label > +label = "thisismuchto" > +S4: Set filesystem UUID > +UUID = deaddead-dead-dead-dead-deaddeaddead > +S5: Zero out filesystem UUID > +UUID = 00000000-0000-0000-0000-000000000000 > +S6: Randomize filesystem UUID > +S7: Restore original filesystem UUID > +UUID = babababa-baba-baba-baba-babababababa > diff --git a/tests/xfs/773 b/tests/xfs/773 > new file mode 100755 > index 00000000..f184962a > --- /dev/null > +++ b/tests/xfs/773 > @@ -0,0 +1,114 @@ > +#! /bin/bash > +# SPDX-License-Identifier: GPL-2.0-or-later > +# Copyright (c) 2021 Oracle. All Rights Reserved. > +# > +# FS QA Test No. 773 > +# > +# Functional testing for xfs_admin to ensure that it parses arguments correctly > +# with regards to data devices that are files, external logs, and realtime > +# devices. > +# > +# Because this test synthesizes log and rt devices (by modifying the test run > +# configuration), it does /not/ require the ability to mount the scratch > +# filesystem. This increases test coverage while isolating the weird bits to a > +# single test. > +# > +# This is partially a regression test for "xfs_admin: pick up log arguments > +# correctly", insofar as the issue fixed by that patch was discovered with an > +# earlier revision of this test. > + > +seq=`basename $0` > +seqres=$RESULT_DIR/$seq > +echo "QA output created by $seq" > + > +here=`pwd` > +tmp=/tmp/$$ > +status=1 # failure is the default! > +trap "_cleanup; exit \$status" 0 1 2 3 15 > + > +_cleanup() > +{ > + cd / > + rm -f $tmp.* $fake_logfile $fake_rtfile $fake_datafile > +} > + > +# get standard environment, filters and checks > +. ./common/rc > +. ./common/filter > + > +# real QA test starts here > +_supported_fs xfs > +_require_test > +_require_scratch_nocheck > +_require_command "$XFS_ADMIN_PROG" "xfs_admin" > + > +rm -f $seqres.full > + > +# Create some fake sparse files for testing external devices and whatnot > +fake_datafile=$TEST_DIR/scratch.data > +rm -f $fake_datafile > +truncate -s 500m $fake_datafile > + > +fake_logfile=$TEST_DIR/scratch.log > +rm -f $fake_logfile > +truncate -s 500m $fake_logfile > + > +fake_rtfile=$TEST_DIR/scratch.rt > +rm -f $fake_rtfile > +truncate -s 500m $fake_rtfile > + > +# Save the original variables > +orig_ddev=$SCRATCH_DEV > +orig_external=$USE_EXTERNAL > +orig_logdev=$SCRATCH_LOGDEV > +orig_rtdev=$SCRATCH_RTDEV > + > +scenario() { > + echo "$@" | tee -a $seqres.full > + > + SCRATCH_DEV=$orig_ddev > + USE_EXTERNAL=$orig_external > + SCRATCH_LOGDEV=$orig_logdev > + SCRATCH_RTDEV=$orig_rtdev > +} > + > +check_label() { > + _scratch_mkfs -L oldlabel >> $seqres.full > + _scratch_xfs_db -c label > + _scratch_xfs_admin -L newlabel "$@" >> $seqres.full > + _scratch_xfs_db -c label > + _scratch_xfs_repair -n &>> $seqres.full || echo "Check failed?" > +} > + > +scenario "S1: Check that label setting with file image" > +SCRATCH_DEV=$fake_datafile > +check_label -f > + > +scenario "S2: Check that setting with logdev works" > +USE_EXTERNAL=yes > +SCRATCH_LOGDEV=$fake_logfile > +check_label > + > +scenario "S3: Check that setting with rtdev works" > +USE_EXTERNAL=yes > +SCRATCH_RTDEV=$fake_rtfile > +check_label > + > +scenario "S4: Check that setting with rtdev + logdev works" > +USE_EXTERNAL=yes > +SCRATCH_LOGDEV=$fake_logfile > +SCRATCH_RTDEV=$fake_rtfile > +check_label > + > +scenario "S5: Check that setting with nortdev + nologdev works" > +USE_EXTERNAL= > +SCRATCH_LOGDEV= > +SCRATCH_RTDEV= > +check_label > + > +scenario "S6: Check that setting with bdev incorrectly flagged as file works" > +check_label -f > + > +# success, all done > +status=0 > +exit > diff --git a/tests/xfs/773.out b/tests/xfs/773.out > new file mode 100644 > index 00000000..954bfb85 > --- /dev/null > +++ b/tests/xfs/773.out > @@ -0,0 +1,19 @@ > +QA output created by 773 > +S1: Check that label setting with file image > +label = "oldlabel" > +label = "newlabel" > +S2: Check that setting with logdev works > +label = "oldlabel" > +label = "newlabel" > +S3: Check that setting with rtdev works > +label = "oldlabel" > +label = "newlabel" > +S4: Check that setting with rtdev + logdev works > +label = "oldlabel" > +label = "newlabel" > +S5: Check that setting with nortdev + nologdev works > +label = "oldlabel" > +label = "newlabel" > +S6: Check that setting with bdev incorrectly flagged as file works > +label = "oldlabel" > +label = "newlabel" > diff --git a/tests/xfs/group b/tests/xfs/group > index 461ae2b2..a2309465 100644 > --- a/tests/xfs/group > +++ b/tests/xfs/group > @@ -522,5 +522,7 @@ > 537 auto quick > 538 auto stress > 539 auto quick mount > +764 auto quick repair > 768 auto quick repair > 770 auto repair > +773 auto quick repair >
diff --git a/common/xfs b/common/xfs index c2384146..5abc7034 100644 --- a/common/xfs +++ b/common/xfs @@ -1142,3 +1142,24 @@ _check_scratch_xfs_features() echo "${output[@]}" test "${found}" -eq "$#" } + +# Decide if xfs_repair knows how to set (or clear) a filesystem feature. +_require_xfs_repair_upgrade() +{ + local type="$1" + + $XFS_REPAIR_PROG -c "$type=garbagevalue" 2>&1 | \ + grep -q 'unknown option' && \ + _notrun "xfs_repair does not support upgrading fs with $type" +} + +_require_xfs_scratch_inobtcount() +{ + _require_scratch + + _scratch_mkfs -m inobtcount=1 &> /dev/null || \ + _notrun "mkfs.xfs doesn't have inobtcount feature" + _try_scratch_mount || \ + _notrun "inobtcount not supported by scratch filesystem type: $FSTYP" + _scratch_unmount +} diff --git a/tests/xfs/764 b/tests/xfs/764 new file mode 100755 index 00000000..ebdf8883 --- /dev/null +++ b/tests/xfs/764 @@ -0,0 +1,93 @@ +#! /bin/bash +# SPDX-License-Identifier: GPL-2.0-or-later +# Copyright (c) 2021 Oracle. All Rights Reserved. +# +# FS QA Test No. 764 +# +# Functional testing for xfs_admin to make sure that it handles option parsing +# correctly for functionality that's relevant to V5 filesystems. It doesn't +# test the options that apply only to V4 filesystems because that disk format +# is deprecated. + +seq=`basename $0` +seqres=$RESULT_DIR/$seq +echo "QA output created by $seq" + +here=`pwd` +tmp=/tmp/$$ +status=1 # failure is the default! +trap "_cleanup; exit \$status" 0 1 2 3 15 + +_cleanup() +{ + cd / + rm -f $tmp.* +} + +# get standard environment, filters and checks +. ./common/rc +. ./common/filter + +# real QA test starts here +_supported_fs xfs +_require_scratch +_require_command "$XFS_ADMIN_PROG" "xfs_admin" + +rm -f $seqres.full + +note() { + echo "$@" | tee -a $seqres.full +} + +note "S0: Initialize filesystem" +_scratch_mkfs -L origlabel -m uuid=babababa-baba-baba-baba-babababababa >> $seqres.full +_scratch_xfs_db -c label -c uuid +_scratch_xfs_repair -n &>> $seqres.full || echo "Check failed?" + +note "S1: Set a filesystem label" +_scratch_xfs_admin -L newlabel >> $seqres.full +_scratch_xfs_db -c label +_scratch_xfs_repair -n &>> $seqres.full || echo "Check failed?" + +note "S2: Clear filesystem label" +_scratch_xfs_admin -L -- >> $seqres.full +_scratch_xfs_db -c label +_scratch_xfs_repair -n &>> $seqres.full || echo "Check failed?" + +note "S3: Try to set oversized label" +_scratch_xfs_admin -L thisismuchtoolongforxfstohandle >> $seqres.full +_scratch_xfs_db -c label +_scratch_xfs_repair -n &>> $seqres.full || echo "Check failed?" + +note "S4: Set filesystem UUID" +_scratch_xfs_admin -U deaddead-dead-dead-dead-deaddeaddead >> $seqres.full +_scratch_xfs_db -c uuid +_scratch_xfs_repair -n &>> $seqres.full || echo "Check failed?" + +note "S5: Zero out filesystem UUID" +_scratch_xfs_admin -U nil >> $seqres.full +_scratch_xfs_db -c uuid +_scratch_xfs_repair -n &>> $seqres.full || echo "Check failed?" + +note "S6: Randomize filesystem UUID" +old_uuid="$(_scratch_xfs_db -c uuid)" +_scratch_xfs_admin -U generate >> $seqres.full +new_uuid="$(_scratch_xfs_db -c uuid)" +if [ "$new_uuid" = "$old_uuid" ]; then + echo "UUID randomization failed? $old_uuid == $new_uuid" +fi +_scratch_xfs_repair -n &>> $seqres.full || echo "Check failed?" + +note "S7: Restore original filesystem UUID" +if _check_scratch_xfs_features V5 >/dev/null; then + # Only V5 supports the metauuid feature that enables us to restore the + # original UUID after a change. + _scratch_xfs_admin -U restore >> $seqres.full + _scratch_xfs_db -c uuid +else + echo "UUID = babababa-baba-baba-baba-babababababa" +fi + +# success, all done +status=0 +exit diff --git a/tests/xfs/764.out b/tests/xfs/764.out new file mode 100644 index 00000000..8da929ec --- /dev/null +++ b/tests/xfs/764.out @@ -0,0 +1,17 @@ +QA output created by 764 +S0: Initialize filesystem +label = "origlabel" +UUID = babababa-baba-baba-baba-babababababa +S1: Set a filesystem label +label = "newlabel" +S2: Clear filesystem label +label = "" +S3: Try to set oversized label +label = "thisismuchto" +S4: Set filesystem UUID +UUID = deaddead-dead-dead-dead-deaddeaddead +S5: Zero out filesystem UUID +UUID = 00000000-0000-0000-0000-000000000000 +S6: Randomize filesystem UUID +S7: Restore original filesystem UUID +UUID = babababa-baba-baba-baba-babababababa diff --git a/tests/xfs/773 b/tests/xfs/773 new file mode 100755 index 00000000..f184962a --- /dev/null +++ b/tests/xfs/773 @@ -0,0 +1,114 @@ +#! /bin/bash +# SPDX-License-Identifier: GPL-2.0-or-later +# Copyright (c) 2021 Oracle. All Rights Reserved. +# +# FS QA Test No. 773 +# +# Functional testing for xfs_admin to ensure that it parses arguments correctly +# with regards to data devices that are files, external logs, and realtime +# devices. +# +# Because this test synthesizes log and rt devices (by modifying the test run +# configuration), it does /not/ require the ability to mount the scratch +# filesystem. This increases test coverage while isolating the weird bits to a +# single test. +# +# This is partially a regression test for "xfs_admin: pick up log arguments +# correctly", insofar as the issue fixed by that patch was discovered with an +# earlier revision of this test. + +seq=`basename $0` +seqres=$RESULT_DIR/$seq +echo "QA output created by $seq" + +here=`pwd` +tmp=/tmp/$$ +status=1 # failure is the default! +trap "_cleanup; exit \$status" 0 1 2 3 15 + +_cleanup() +{ + cd / + rm -f $tmp.* $fake_logfile $fake_rtfile $fake_datafile +} + +# get standard environment, filters and checks +. ./common/rc +. ./common/filter + +# real QA test starts here +_supported_fs xfs +_require_test +_require_scratch_nocheck +_require_command "$XFS_ADMIN_PROG" "xfs_admin" + +rm -f $seqres.full + +# Create some fake sparse files for testing external devices and whatnot +fake_datafile=$TEST_DIR/scratch.data +rm -f $fake_datafile +truncate -s 500m $fake_datafile + +fake_logfile=$TEST_DIR/scratch.log +rm -f $fake_logfile +truncate -s 500m $fake_logfile + +fake_rtfile=$TEST_DIR/scratch.rt +rm -f $fake_rtfile +truncate -s 500m $fake_rtfile + +# Save the original variables +orig_ddev=$SCRATCH_DEV +orig_external=$USE_EXTERNAL +orig_logdev=$SCRATCH_LOGDEV +orig_rtdev=$SCRATCH_RTDEV + +scenario() { + echo "$@" | tee -a $seqres.full + + SCRATCH_DEV=$orig_ddev + USE_EXTERNAL=$orig_external + SCRATCH_LOGDEV=$orig_logdev + SCRATCH_RTDEV=$orig_rtdev +} + +check_label() { + _scratch_mkfs -L oldlabel >> $seqres.full + _scratch_xfs_db -c label + _scratch_xfs_admin -L newlabel "$@" >> $seqres.full + _scratch_xfs_db -c label + _scratch_xfs_repair -n &>> $seqres.full || echo "Check failed?" +} + +scenario "S1: Check that label setting with file image" +SCRATCH_DEV=$fake_datafile +check_label -f + +scenario "S2: Check that setting with logdev works" +USE_EXTERNAL=yes +SCRATCH_LOGDEV=$fake_logfile +check_label + +scenario "S3: Check that setting with rtdev works" +USE_EXTERNAL=yes +SCRATCH_RTDEV=$fake_rtfile +check_label + +scenario "S4: Check that setting with rtdev + logdev works" +USE_EXTERNAL=yes +SCRATCH_LOGDEV=$fake_logfile +SCRATCH_RTDEV=$fake_rtfile +check_label + +scenario "S5: Check that setting with nortdev + nologdev works" +USE_EXTERNAL= +SCRATCH_LOGDEV= +SCRATCH_RTDEV= +check_label + +scenario "S6: Check that setting with bdev incorrectly flagged as file works" +check_label -f + +# success, all done +status=0 +exit diff --git a/tests/xfs/773.out b/tests/xfs/773.out new file mode 100644 index 00000000..954bfb85 --- /dev/null +++ b/tests/xfs/773.out @@ -0,0 +1,19 @@ +QA output created by 773 +S1: Check that label setting with file image +label = "oldlabel" +label = "newlabel" +S2: Check that setting with logdev works +label = "oldlabel" +label = "newlabel" +S3: Check that setting with rtdev works +label = "oldlabel" +label = "newlabel" +S4: Check that setting with rtdev + logdev works +label = "oldlabel" +label = "newlabel" +S5: Check that setting with nortdev + nologdev works +label = "oldlabel" +label = "newlabel" +S6: Check that setting with bdev incorrectly flagged as file works +label = "oldlabel" +label = "newlabel" diff --git a/tests/xfs/group b/tests/xfs/group index 461ae2b2..a2309465 100644 --- a/tests/xfs/group +++ b/tests/xfs/group @@ -522,5 +522,7 @@ 537 auto quick 538 auto stress 539 auto quick mount +764 auto quick repair 768 auto quick repair 770 auto repair +773 auto quick repair