From 922d3c154e7e8e17b1e0d40e18a675f4862456ab Mon Sep 17 00:00:00 2001 From: Richard Ipsum Date: Wed, 16 Apr 2014 09:56:41 +0000 Subject: 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 --- distbuild/build_controller.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) 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: -- cgit v1.2.1