Commit a4f73241 authored by Rémy Coutable's avatar Rémy Coutable

[Review Apps] Don't exit when dependable job fails

The function that retrieves the dependable job is pretty basic and
retrieves the first job found with the matching name, but this job can
be failed but then successfully retried. In that case, we would exit the
depending job even though the dependable job actually succeeded (the
second time). Let's simplify things, be optimistic and continue with the
depending job even if the dependable job fails.

That reverts to the original behavior.
Signed-off-by: default avatarRémy Coutable <remy@rymai.me>
parent daae58d2
...@@ -346,7 +346,6 @@ function wait_for_job_to_be_done() { ...@@ -346,7 +346,6 @@ function wait_for_job_to_be_done() {
if [[ "${job_status}" == "failed" ]]; then if [[ "${job_status}" == "failed" ]]; then
echo "The '${job_name}' failed." echo "The '${job_name}' failed."
exit 1
elif [[ "${job_status}" == "manual" ]]; then elif [[ "${job_status}" == "manual" ]]; then
echo "The '${job_name}' is manual." echo "The '${job_name}' is manual."
else else
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment