servo: Suppress the exeception of getting BaseEC board on old servod
The method of getting the BaseEC board name is introduced recently
in servod. For the old servod, suppress the exception.
BUG=b:66211919
TEST=Ran the firmware_DevMode test on an old servod.
Change-Id: I8cf5c2f1d3dbaca6f4a88d64281bd32a627a8137
Reviewed-on: https://chromium-review.googlesource.com/692576
Commit-Ready: Wai-Hong Tam <waihong@google.com>
Tested-by: Wai-Hong Tam <waihong@google.com>
Reviewed-by: Kevin Shelton <kmshelton@chromium.org>
diff --git a/server/cros/servo/servo.py b/server/cros/servo/servo.py
index e0c268f..314b10e 100644
--- a/server/cros/servo/servo.py
+++ b/server/cros/servo/servo.py
@@ -449,7 +449,16 @@
def get_base_board(self):
"""Get the board of the base connected to servod."""
- return self._server.get_base_board()
+ try:
+ return self._server.get_base_board()
+ except xmlrpclib.Fault as e:
+ # TODO(waihong): Remove the following compatibility check when
+ # the new versions of hdctools are deployed.
+ if 'not supported' in str(e):
+ logging.warning('The servod is too old that get_base_board '
+ 'not supported.')
+ return ''
+ raise
def _get_xmlrpclib_exception(self, xmlexc):