diff options
author | Josef Bacik <josef@toxicpanda.com> | 2022-08-24 15:32:10 -0400 |
---|---|---|
committer | Zorro Lang <zlang@kernel.org> | 2022-10-16 15:00:05 +0800 |
commit | a75c5f50584e03ca7862ad51f48efd2d524d1dc5 (patch) | |
tree | d2c23a48d8ae0b797c94ede903e1ea664bff95ec /check | |
parent | e9483160803966faa62e039128a2c33ea3e7922d (diff) |
fstests: get section config after RUN_SECTION checksv2022.10.16
While trying to do
./check -s <some section>
I was failing because I had a section defined higher than <some section>
that had TEST_DEV=/some/nonexistent/device, since I was using the other
section to test an experimental drive. This appears to be because we
run through all of the sections, and when getting the section config we
check to see if it's valid, and in this case the section wasn't valid.
The section I was actually trying to use was valid however. Fix check
to see if the section we're trying to run is in our list of sections to
run first, and then if it is get the config at that point.
Signed-off-by: Josef Bacik <josef@toxicpanda.com>
Reviewed-by: Zorro Lang <zlang@redhat.com>
Signed-off-by: Zorro Lang <zlang@kernel.org>
Diffstat (limited to 'check')
-rwxr-xr-x | check | 3 |
1 files changed, 2 insertions, 1 deletions
@@ -678,7 +678,6 @@ function run_section() OLD_FSTYP=$FSTYP OLD_TEST_FS_MOUNT_OPTS=$TEST_FS_MOUNT_OPTS - get_next_config $section # Do we need to run only some sections ? if [ ! -z "$RUN_SECTION" ]; then @@ -708,6 +707,8 @@ function run_section() fi fi + get_next_config $section + mkdir -p $RESULT_BASE if [ ! -d $RESULT_BASE ]; then echo "failed to create results directory $RESULT_BASE" |