commit 2d30758b4e3f011ff18157e0992fe9254e5bc79b
author Jamie Lennox <jamielennox@gmail.com> Mon Mar 06 17:46:02 2017 +1100
committer Jesse Keating <omgjlk@us.ibm.com> Thu May 25 10:16:29 2017 -0700
tree 27e2fb4920be320bcf19650f3316ad6dc6c8f231
parent 62198cb76152692a055d83e2453b6c956298920c

Include exc_info in reporter failure

If a reporter fails to push up a status that's a bad and unusual
situation. Include the exception information in this log message so we
know exactly what went wrong and can hopefully fix it.

Change-Id: I309409350f84299b6761507a9c452a0617022339
Signed-off-by: Jamie Lennox <jamielennox@gmail.com>

zuul/driver/github/githubreporter.py

diff --git a/zuul/driver/github/githubreporter.py b/zuul/driver/github/githubreporter.py
index 9975cbc..6808bdd 100644
--- a/zuul/driver/github/githubreporter.py
+++ b/zuul/driver/github/githubreporter.py
@@ -102,12 +102,12 @@
                 item.change.is_merged = True
                 return
             except MergeFailure:
-                self.log.debug(
+                self.log.exception(
                     'Merge attempt of change %s  %s/2 failed.' %
-                    (i, item.change))
+                    (item.change, i), exc_info=True)
                 if i == 1:
                     time.sleep(2)
-        self.log.debug(
+        self.log.warning(
             'Merge of change %s failed after 2 attempts, giving up' %
             item.change)