Treat LOST jobs as failures in didAnyJobFail().
Zuul already treats LOST jobs as failures in much of the code. Do the same in didAnyJobFail so that changes behind a change with a LOST job get their jobs cancelled. Change-Id: I1eb819a6cfb3b3ee0a2c131417b00dab1d83f29d Reviewed-on: https://review.openstack.org/19270 Reviewed-by: James E. Blair <corvus@inaugust.com> Approved: James E. Blair <corvus@inaugust.com> Tested-by: Jenkins
This commit is contained in:
parent
23ec1bac6b
commit
a7c5aa3fa6
|
@ -120,7 +120,8 @@ class Pipeline(object):
|
||||||
if not job.voting:
|
if not job.voting:
|
||||||
continue
|
continue
|
||||||
build = changeish.current_build_set.getBuild(job.name)
|
build = changeish.current_build_set.getBuild(job.name)
|
||||||
if build and build.result == 'FAILURE':
|
# Treat LOST jobs as failures
|
||||||
|
if build and (build.result == 'FAILURE' or build.result == 'LOST'):
|
||||||
return True
|
return True
|
||||||
return False
|
return False
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue