From: Dave Chinner <david@fromorbit.com>
To: fstests@vger.kernel.org
Subject: [PATCH 3/3] check: add exclude sections CLI parameter
Date: Thu, 21 Jan 2016 12:43:13 +1100 [thread overview]
Message-ID: <1453340593-10236-4-git-send-email-david@fromorbit.com> (raw)
In-Reply-To: <1453340593-10236-1-git-send-email-david@fromorbit.com>
From: Dave Chinner <dchinner@redhat.com>
When we have a config file with multiple sections, we might want to
exclude certain config sections from running. Rather than specifying
all the section we want to run, add a "-S <section>" option to build
up a list of sections to exclude.
This is useful if a given section config is known to cause a fatal
failure,but you still want to run all the other config sections.
Also add support to the setup program that emits the currently
configured setup for each section in the config file.
Signed-off-by: Dave Chinner <dchinner@redhat.com>
---
check | 15 +++++++++++++++
setup | 17 ++++++++++++++++-
2 files changed, 31 insertions(+), 1 deletion(-)
diff --git a/check b/check
index 135a9fb..3ef79fc 100755
--- a/check
+++ b/check
@@ -73,6 +73,7 @@ check options
-d dump test output to stdout
--large-fs optimise scratch device for large filesystems
-s section run only specified section from config file
+ -S section exclude the specified section from the config file
testlist options
-g group[,group...] include tests from these groups
@@ -228,6 +229,7 @@ while [ $# -gt 0 ]; do
fi
;;
-s) RUN_SECTION="$RUN_SECTION $2"; shift ;;
+ -S) EXCLUDE_SECTION="$EXCLUDE_SECTION $2"; shift ;;
-l) diff="diff" ;;
-udiff) diff="$diff -u" ;;
@@ -428,6 +430,19 @@ for section in $HOST_OPTIONS_SECTIONS; do
fi
fi
+ # Did this section get excluded?
+ if [ ! -z "$EXCLUDE_SECTION" ]; then
+ skip=false
+ for s in $EXCLUDE_SECTION; do
+ if [ $section == $s ]; then
+ skip=true
+ fi
+ done
+ if $skip; then
+ continue
+ fi
+ fi
+
mkdir -p $RESULT_BASE
if [ ! -d $RESULT_BASE ]; then
echo "failed to create results directory $RESULT_BASE"
diff --git a/setup b/setup
index 1a41d81..6ef2293 100755
--- a/setup
+++ b/setup
@@ -20,7 +20,8 @@ usage()
echo "Usage: $0 [options]"'
- -s section run only specified section(s) from config file
+ -s section run only specified section(s) from config file
+ -S section exclude the specified section from the config file
'
exit 0
}
@@ -29,6 +30,7 @@ while [ $# -gt 0 ]; do
case "$1" in
-\? | -h | --help) usage ;;
-s) RUN_SECTION="$RUN_SECTION $2"; shift ;;
+ -S) EXCLUDE_SECTION="$EXCLUDE_SECTION $2"; shift ;;
*) usage ;;
esac
shift
@@ -58,6 +60,19 @@ for section in $HOST_OPTIONS_SECTIONS; do
fi
fi
+ # Did this section get excluded?
+ if [ ! -z "$EXCLUDE_SECTION" ]; then
+ skip=false
+ for s in $EXCLUDE_SECTION; do
+ if [ $section == $s ]; then
+ skip=true
+ fi
+ done
+ if $skip; then
+ continue
+ fi
+ fi
+
[ "$USE_EXTERNAL" = yes ] || USE_EXTERNAL=no
[ "$USE_LBD_PATCH" = yes ] || USE_LBD_PATCH=no
[ "$LARGE_SCRATCH_DEV" = yes ] || LARGE_SCRATCH_DEV=no
--
2.5.0
next prev parent reply other threads:[~2016-01-21 1:44 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-21 1:43 [PATCH 0/3] fstests: improved config section support Dave Chinner
2016-01-21 1:43 ` [PATCH 1/3] xfs: support realtime/log device setup changes in config sections Dave Chinner
2016-02-02 4:36 ` Eryu Guan
2016-02-04 21:04 ` Dave Chinner
2016-02-05 2:46 ` Eryu Guan
2016-01-21 1:43 ` [PATCH 2/3] setup: add section support Dave Chinner
2016-02-05 3:33 ` Eryu Guan
2016-01-21 1:43 ` Dave Chinner [this message]
2016-02-02 3:34 ` [PATCH 0/3] fstests: improved config " Dave Chinner
2016-02-04 21:06 ` Dave Chinner
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1453340593-10236-4-git-send-email-david@fromorbit.com \
--to=david@fromorbit.com \
--cc=fstests@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox