diff options
author | Richard Ipsum <richard.ipsum@codethink.co.uk> | 2014-04-16 09:56:41 +0000 |
---|---|---|
committer | Richard Ipsum <richard.ipsum@codethink.co.uk> | 2014-04-16 09:56:57 +0000 |
commit | 922d3c154e7e8e17b1e0d40e18a675f4862456ab (patch) | |
tree | 14b2dcb71d4ac922b1da151271be114a80b0e8af /distbuild/build_controller.py | |
parent | f0a0d8ae00b91cba6117a61509229eb4c1f51803 (diff) | |
download | morph-922d3c154e7e8e17b1e0d40e18a675f4862456ab.tar.gz |
Don't log a cache response till we're sure
The controller should check the response event is
actually the response it was waiting for before
logging that there was a cache response
Diffstat (limited to 'distbuild/build_controller.py')
-rw-r--r-- | distbuild/build_controller.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/distbuild/build_controller.py b/distbuild/build_controller.py index ed6c424e..58f0f429 100644 --- a/distbuild/build_controller.py +++ b/distbuild/build_controller.py @@ -341,7 +341,6 @@ class BuildController(distbuild.StateMachine): '(helper id: %s)' % self._helper_id) def _maybe_handle_cache_response(self, event_source, event): - logging.debug('Got cache response: %s' % repr(event.msg)) def set_status(artifact): is_in_cache = cache_state[self._artifact_filename(artifact)] @@ -350,6 +349,8 @@ class BuildController(distbuild.StateMachine): if self._helper_id != event.msg['id']: return # this event is not for us + logging.debug('Got cache response: %s' % repr(event.msg)) + http_status_code = event.msg['status'] if http_status_code != httplib.OK: |