Autotest: add logs in is_stage.
This CL logs the return result or error information of API is_stage in
devserver.
BUG=None
TEST=unittest
Change-Id: I5b49adb57e6b034a352d8ec1ad9e8620ed49fd7f
Reviewed-on: https://chromium-review.googlesource.com/353652
Reviewed-by: Aviv Keshet <akeshet@chromium.org>
Tested-by: Aviv Keshet <akeshet@chromium.org>
diff --git a/client/common_lib/cros/dev_server.py b/client/common_lib/cros/dev_server.py
index b78f954..1b4f0de 100644
--- a/client/common_lib/cros/dev_server.py
+++ b/client/common_lib/cros/dev_server.py
@@ -836,17 +836,20 @@
"""
try:
- return self.run_call(call) == 'True'
+ result = self.run_call(call)
+ logging.debug('whether artifact is staged: %r', result)
+ return result == 'True'
except urllib2.HTTPError as e:
error_markup = e.read()
raise DevServerException(_strip_http_message(error_markup))
- except urllib2.URLError:
+ except urllib2.URLError as e:
# Could be connection issue, retry it.
# For example: <urlopen error [Errno 111] Connection refused>
+ logging.error('URLError happens in is_stage: %r', e)
return False
- except error.CmdError:
+ except error.CmdError as e:
# Retry if SSH failed to connect to the devserver.
- logging.warning('CmdError: Retrying SSH connection to check is_stage.')
+ logging.warning('CmdError happens in is_stage: %r, will retry', e)
return False
site_utils.poll_for_condition(