summaryrefslogtreecommitdiff
path: root/check
diff options
context:
space:
mode:
authorJosef Bacik <josef@toxicpanda.com>2022-08-24 15:32:10 -0400
committerZorro Lang <zlang@kernel.org>2022-10-16 15:00:05 +0800
commita75c5f50584e03ca7862ad51f48efd2d524d1dc5 (patch)
treed2c23a48d8ae0b797c94ede903e1ea664bff95ec /check
parente9483160803966faa62e039128a2c33ea3e7922d (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-xcheck3
1 files changed, 2 insertions, 1 deletions
diff --git a/check b/check
index a880d52f..d2e51296 100755
--- a/check
+++ b/check
@@ -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"