diff options
author | David Spickett <david.spickett@linaro.org> | 2020-05-13 14:27:20 +0100 |
---|---|---|
committer | David Spickett <david.spickett@linaro.org> | 2020-05-14 14:33:09 +0000 |
commit | b3334e58e24ea4ade3088c620dec315e62042754 (patch) | |
tree | c63901b67c695c61245ce88f8c407567ed3aba70 /round-robin.sh | |
parent | f362d7f03b41898cb77901b01afab5efbf59016c (diff) |
tcwg_bmk/round-robin: Add assert_with_msg function
This acts like assert but prints a message
if the assert fails. This will be visible in
the Jenkins logs and give us a unique string
to look for in the abscence of traceback for
shell scripts.
Two instances of returning false in check_regression
have had echos added to them, instead of asserts.
This is because they are not hard failures,
and may be handled by the caller. (run_step)
I have only converted the tcwg-bmk relevant
scripts to assert_with_msg so far, as a proof
of concept.
Change-Id: I3b1badf09e4856c80b973ca8b23088626ed66b0a
Diffstat (limited to 'round-robin.sh')
-rw-r--r-- | round-robin.sh | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/round-robin.sh b/round-robin.sh index be6f4d8b..3361f0ce 100644 --- a/round-robin.sh +++ b/round-robin.sh @@ -531,8 +531,8 @@ check_regression () ${rr[no_regression_p]} ${rr[top_artifacts]} base-artifacts & if wait $!; then - # Current results are no better then the all-time-best results. - # No reason to rebase base-artifacts. + echo "Current results are no better then the all-time-best results. \ +No reason to rebase base-artifacts." false fi @@ -611,7 +611,7 @@ EOF fi if [ x"${rr[update_baseline]}" = x"update" ]; then - # Fail if detected regression in "update" mode. + echo "Detected a regression in \"update\" mode!" false else # Compare results to generate logs and other artifacts, |