From bcfc3c60b0e6eee1901eefae6056d4edeb3b6638 Mon Sep 17 00:00:00 2001 From: "James E. Blair" Date: Mon, 4 Jun 2012 08:54:22 -0700 Subject: [PATCH] Fix lost build detection. On maven jobs, a result is reported as soon as one target finishes. Switch to using 'building' to determine whether a build has finished, rather than checking for a result. Change-Id: Ic681d88e77abe6af6af59689337586b48af62d80 --- zuul/launcher/jenkins.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/zuul/launcher/jenkins.py b/zuul/launcher/jenkins.py index 1e59a57ffb..705fbdf787 100644 --- a/zuul/launcher/jenkins.py +++ b/zuul/launcher/jenkins.py @@ -285,8 +285,8 @@ for build %s" % (item['id'], build)) info can not be retreived" % build) lostbuilds.append(build) continue - if not info['result']: - # It hasn't finished, continue + if info['building']: + # It has not finished. continue finish_time = (info['timestamp'] + info['duration']) / 1000 if time.time() - finish_time > JENKINS_GRACE_TIME: