diff options
| author | Alex Elder <elder@doink.(none)> | 2012-04-14 11:26:21 -0500 |
|---|---|---|
| committer | Alex Elder <elder@dreamhost.com> | 2012-04-14 11:26:21 -0500 |
| commit | 3d187db50130d048bc9c6e980c5c64fbd36d8cb4 (patch) | |
| tree | effb01808811f1f71b63c640a60d884a376c44a7 /qa | |
| parent | bf8847e7c14a52467950c8f3bab88e43f660e3c0 (diff) | |
| download | ceph-3d187db50130d048bc9c6e980c5c64fbd36d8cb4.tar.gz | |
run_xfstests.sh: pass test result via exit status
It turns out that xfstests *does* exit with non-zero status
when a test fails. Its exit status is the number of tests
that failed (which, now that we have over 255 tests could be
an issue...)
Save the exit status and make it be the result of the run.
Signed-off-by: Alex Elder <elder@dreamhost.com
Diffstat (limited to 'qa')
| -rw-r--r-- | qa/run_xfstests.sh | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/qa/run_xfstests.sh b/qa/run_xfstests.sh index f906c97a7c0..92b449aab64 100644 --- a/qa/run_xfstests.sh +++ b/qa/run_xfstests.sh @@ -407,7 +407,7 @@ setup pushd "${XFSTESTS_DIR}" ./check ${TESTS} -# sadly, check doesn't return a meaningful exit status +status=$? popd cleanup @@ -415,4 +415,4 @@ cleanup echo "This xfstests run started at: ${start_date}" echo "xfstests run completed at: $(date)" -exit 0 +exit "${status}" |