Merge lp:~joetalbott/adt-result-checker/make_results_public_on_failure into lp:adt-result-checker

[Joe Talbott]

Make results public when a message is placed in the deadletter queue.

* This allows jenkins to not have to wait to timeout.

Make results public when a message is placed in the deadletter queue.

* This allows jenkins to not have to wait to timeout.

[Celso Providelo (cprov)]

Josepth,

Thanks for working on this, we have done a good session of local tests together and as mentioned in the card, when result-checker has done everything it can to make the request pass but failed, releasing jenkins slaves is the best approach.

review: Approve

=== modified file 'README.rst'
--- README.rst	2015-03-19 23:26:47 +0000
+++ README.rst	2015-03-20 16:19:11 +0000
@@ -16,6 +16,10 @@
 Install the Service
 ========================
 
+Install required packages::
+
+  $ sudo apt-get install python3-dev
+
 Build and activate a virtualenv with python3::
 
   $ virtualenv -p python3 --system-site-packages ve
 
=== modified file 'adt_result_checker/__init__.py'
--- adt_result_checker/__init__.py	2015-03-20 02:23:17 +0000
+++ adt_result_checker/__init__.py	2015-03-20 16:19:11 +0000
@@ -127,6 +127,11 @@
         queue.put(message.payload)
         queue.close()
 
+        # make the results public so jenkins doesn't have to timeout.
+        request_id = message.payload['request_id']
+        if result_tarball_exists_in_swift(request_id):
+            self.post_results(request_id)
+
     def maybe_retry_message(self, message, reason=''):
         """Re-queue this message for retrial, unless it's retry limit has been
         reached, in whch case insert it into the dead letter queue.