diff mbox

[Branch,~linaro-validation/lava-dispatcher/trunk] Rev 668: Better logging of error messages for the scheduler to consume

Message ID 20130905174928.10879.84647.launchpad@ackee.canonical.com
State Accepted
Headers show

Commit Message

Antonio Terceiro Sept. 5, 2013, 5:49 p.m. UTC 
Merge authors:
  Antonio Terceiro (terceiro)
Related merge proposals:
  https://code.launchpad.net/~terceiro/lava-dispatcher/better-logging/+merge/183993
  proposed by: Antonio Terceiro (terceiro)
  review: Approve - Tyler Baker (tyler-baker)
------------------------------------------------------------
revno: 668 [merge]
committer: Antonio Terceiro <antonio.terceiro@linaro.org>
branch nick: trunk
timestamp: Thu 2013-09-05 14:48:39 -0300
message:
  Better logging of error messages for the scheduler to consume
modified:
  lava_dispatcher/job.py


--
lp:lava-dispatcher
https://code.launchpad.net/~linaro-validation/lava-dispatcher/trunk

You are subscribed to branch lp:lava-dispatcher.
To unsubscribe from this branch go to https://code.launchpad.net/~linaro-validation/lava-dispatcher/trunk/+edit-subscription
diff mbox

Patch

=== modified file 'lava_dispatcher/job.py'
--- lava_dispatcher/job.py	2013-08-28 14:55:50 +0000
+++ lava_dispatcher/job.py	2013-09-05 17:05:29 +0000
@@ -302,7 +302,7 @@ 
                     logging.warn("pexpect timed out, pass with status %s" % status)
                     pass
                 except Exception as err:
-                    logging.info("General Exception")
+                    logging.info("General Exception: %s" % unicode(str(err)))
                     raise
                 else:
                     logging.info("setting status pass")
@@ -324,6 +324,7 @@ 
                             err_msg += "Lava failed on test: %s" % \
                                        params.get('test_name', "Unknown")
                         err_msg = err_msg + traceback.format_exc()
+                        print("ErrorMessage: %s" % unicode(str(err)))
                         print err_msg
                     else:
                         logging.info(