[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [XEN PATCH 3/9] CI: Fix status check for tools/tests
From: Anthony PERARD <anthony.perard@xxxxxxxxxx> Without "pipefail", $? have the exit value of `tee`, which should always be 0. But instead of using "pipefail", do collect the value of from the test with $PIPESTATUS. Signed-off-by: Anthony PERARD <anthony.perard@xxxxxxxxxx> --- Notes: Without the next patch "CI: Ignore run-tools-test return value", this just mean that if one test fail, the script `qubes-x86-64.sh` will detect a ERROR-Timeout because the expected string is never printed. automation/scripts/run-tools-tests | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/automation/scripts/run-tools-tests b/automation/scripts/run-tools-tests index 2bca1589db..695ed77e46 100755 --- a/automation/scripts/run-tools-tests +++ b/automation/scripts/run-tools-tests @@ -20,7 +20,7 @@ for f in "$1"/*; do echo "Running $f" printf ' <testcase name="%s">\n' "$f" >> "$xml_out" "$f" 2>&1 | tee /tmp/out - ret=$? + ret=${PIPESTATUS[0]} if [ "$ret" -ne 0 ]; then echo "FAILED: $f" failed+=" $f" -- Anthony PERARD
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |