commit 4c5b10381f01b36507c862c72a052341b2fc0cc2
author Paul Belanger <pabelanger@redhat.com> Thu Jun 09 17:56:48 2016 -0400
committer Paul Belanger <pabelanger@redhat.com> Mon Jun 13 10:29:09 2016 -0400
tree c97db7da3eff16e471578a9ef9e9b7cfc0a65dc7
parent c7959f219038656477589743e3a85af4e25e6412

Add POST_FAILURE status

First check if we fail to run our post_playbook, if a failure set
POST_FAILURE. This usually means as misconfigured job or
infra problem.

Then check if our job is successful or a failure.

Change-Id: Icc6b8254ac7812c8df733e780a43a44a528f7b78
Signed-off-by: Paul Belanger <pabelanger@redhat.com>

zuul/launcher/ansiblelaunchserver.py

diff --git a/zuul/launcher/ansiblelaunchserver.py b/zuul/launcher/ansiblelaunchserver.py
index f0b695d..33e00b8 100644
--- a/zuul/launcher/ansiblelaunchserver.py
+++ b/zuul/launcher/ansiblelaunchserver.py
@@ -785,7 +785,9 @@
 
             job_status = self.runAnsiblePlaybook(jobdir, timeout)
             post_status = self.runAnsiblePostPlaybook(jobdir, job_status)
-            if job_status and post_status:
+            if not post_status:
+                status = 'POST_FAILURE'
+            elif job_status:
                 status = 'SUCCESS'
             else:
                 status = 'FAILURE'