diff options
author | Maxim Kuvyrkov <maxim.kuvyrkov@linaro.org> | 2022-08-11 08:55:55 +0000 |
---|---|---|
committer | Maxim Kuvyrkov <maxim.kuvyrkov@linaro.org> | 2022-08-11 08:56:03 +0000 |
commit | 8d0783f72d656f3b9995f1113f01ce70a8b982f2 (patch) | |
tree | 9d672ce6085db31418dc1cd641fe8e23e4276b55 | |
parent | cab5aeb079026927bbc8990b56ed33d13ba8ee38 (diff) |
round-robin.sh: Fix bisect test builds
... which now use "update_baseline == ignore". Fail on regressions
for "onsuccess" and "ignore" values. Ignore regressions for "force"
value, and assert that there can be no regression for "init" value.
Change-Id: I51219b7f3ab50e19a19e6fe5d514bcb80d1e6e1a
-rw-r--r-- | round-robin.sh | 9 |
1 files changed, 4 insertions, 5 deletions
diff --git a/round-robin.sh b/round-robin.sh index b259f578..810714cb 100644 --- a/round-robin.sh +++ b/round-robin.sh @@ -663,6 +663,9 @@ check_regression () exit $EXTERNAL_FAIL fi + assert_with_msg "no_regression_p should succeed in init baseline mode" \ + [ x"${rr[update_baseline]}" != x"init" ] + if [ -f $run_step_artifacts/results.regressions ]; then # Add regression info generated by no_regression_p to top-level # results file. @@ -770,13 +773,9 @@ EOF ) fi - if [ x"${rr[update_baseline]}" = x"onsuccess" ]; then + if [ x"${rr[update_baseline]}" != x"force" ]; then echo "Detected a regression!" false - else - # Compare results to generate logs and other artifacts, - # but we don't really care about regressions. - : fi ) } |