blob: 333c5a35ac575f523cfa8893b1881f584fc9b5b6 [file] [log] [blame]
Derek Beckettf73baca2020-08-19 15:08:47 -07001# Lint as: python2, python3
Fang Deng5d518f42013-08-02 14:04:32 -07002# Copyright (c) 2013 The Chromium OS Authors. All rights reserved.
3# Use of this source code is governed by a BSD-style license that can be
4# found in the LICENSE file.
5#
6# Expects to be run in an environment with sudo and no interactive password
7# prompt, such as within the Chromium OS development chroot.
8
9
10"""This file provides core logic for servo verify/repair process."""
11
12
Derek Beckettf73baca2020-08-19 15:08:47 -070013from __future__ import absolute_import
14from __future__ import division
15from __future__ import print_function
16
Fang Deng5d518f42013-08-02 14:04:32 -070017import logging
Raul E Rangel52ca2e82018-07-03 14:10:14 -060018import os
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -080019import re
20import tarfile
Andrew McRaef0679932020-08-13 09:15:23 +100021import threading
Garry Wang2b5eef92020-08-21 16:23:35 -070022import json
Garry Wangc1288cf2019-12-17 14:58:00 -080023import time
Derek Beckettf73baca2020-08-19 15:08:47 -070024import six
25import six.moves.xmlrpc_client
Otabek Kasimov120b6fa2020-07-03 00:15:27 -070026import calendar
Fang Deng5d518f42013-08-02 14:04:32 -070027
28from autotest_lib.client.bin import utils
Garry Wang79e9af62019-06-12 15:19:19 -070029from autotest_lib.client.common_lib import error
Richard Barnette9a26ad62016-06-10 12:03:08 -070030from autotest_lib.client.common_lib import hosts
Garry Wang7b0e1b72020-03-25 19:08:59 -070031from autotest_lib.client.common_lib import lsbrelease_utils
Fang Deng5d518f42013-08-02 14:04:32 -070032from autotest_lib.client.common_lib.cros import retry
Christopher Wileycef1f902014-06-19 11:11:23 -070033from autotest_lib.client.common_lib.cros.network import ping_runner
Richard Barnette9a26ad62016-06-10 12:03:08 -070034from autotest_lib.server.cros.servo import servo
Richard Barnetted31580e2018-05-14 19:58:00 +000035from autotest_lib.server.hosts import servo_repair
Garry Wangebc015b2019-06-06 17:45:06 -070036from autotest_lib.server.hosts import base_servohost
Garry Wang11b5e872020-03-11 15:14:08 -070037from autotest_lib.server.hosts import servo_constants
Otabek Kasimov4ea636e2020-04-14 23:35:06 -070038from autotest_lib.server.cros.faft.utils import config
Garry Wang11b5e872020-03-11 15:14:08 -070039from autotest_lib.client.common_lib import global_config
Otabek Kasimov8475cce2020-07-14 12:11:31 -070040from autotest_lib.site_utils.admin_audit import servo_updater
Garry Wangd7367482020-02-27 13:52:40 -080041
Otabek Kasimov15963492020-06-23 21:10:51 -070042try:
43 from chromite.lib import metrics
44except ImportError:
45 metrics = utils.metrics_mock
46
Dan Shi3b2adf62015-09-02 17:46:54 -070047_CONFIG = global_config.global_config
Fang Deng5d518f42013-08-02 14:04:32 -070048
Otabek Kasimova7ba91a2020-03-09 08:31:01 -070049
Garry Wangebc015b2019-06-06 17:45:06 -070050class ServoHost(base_servohost.BaseServoHost):
51 """Host class for a servo host(e.g. beaglebone, labstation)
Dana Goyette0b6e6402019-10-04 11:09:24 -070052 that with a servo instance for a specific port.
53
54 @type _servo: servo.Servo | None
55 """
Fang Deng5d518f42013-08-02 14:04:32 -070056
Raul E Rangel52ca2e82018-07-03 14:10:14 -060057 DEFAULT_PORT = int(os.getenv('SERVOD_PORT', '9999'))
Richard Barnette9a26ad62016-06-10 12:03:08 -070058
Dan Shie5b3c512014-08-21 12:12:09 -070059 # Timeout for initializing servo signals.
Wai-Hong Tam37b6ed32017-09-19 15:52:39 -070060 INITIALIZE_SERVO_TIMEOUT_SECS = 60
Richard Barnette9a26ad62016-06-10 12:03:08 -070061
Otabek Kasimov545739c2020-08-20 00:24:21 -070062 # Default timeout for run terminal command.
63 DEFAULT_TERMINAL_TIMEOUT = 30
64
xixuan6cf6d2f2016-01-29 15:29:00 -080065 # Ready test function
66 SERVO_READY_METHOD = 'get_version'
Fang Deng5d518f42013-08-02 14:04:32 -070067
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -080068 # Directory prefix on the servo host where the servod logs are stored.
69 SERVOD_LOG_PREFIX = '/var/log/servod'
70
71 # Exit code to use when symlinks for servod logs are not found.
72 NO_SYMLINKS_CODE = 9
73
74 # Directory in the job's results directory to dump the logs into.
75 LOG_DIR = 'servod'
76
77 # Prefix for joint loglevel files in the logs.
78 JOINT_LOG_PREFIX = 'log'
79
80 # Regex group to extract timestamp from logfile name.
81 TS_GROUP = 'ts'
82
83 # This regex is used to extract the timestamp from servod logs.
Garry Wang22f2e842020-09-09 20:19:19 -070084 # files always start with log.
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -080085 TS_RE = (r'log.'
86 # The timestamp is of format %Y-%m-%d--%H-%M-%S.MS
87 r'(?P<%s>\d{4}(\-\d{2}){2}\-(-\d{2}){3}.\d{3})'
88 # The loglevel is optional depending on labstation version.
89 r'(.(INFO|DEBUG|WARNING))?' % TS_GROUP)
90 TS_EXTRACTOR = re.compile(TS_RE)
91
92 # Regex group to extract MCU name from logline in servod logs.
93 MCU_GROUP = 'mcu'
94
95 # Regex group to extract logline from MCU logline in servod logs.
96 LINE_GROUP = 'line'
97
98 # This regex is used to extract the mcu and the line content from an
99 # MCU logline in servod logs. e.g. EC or servo_v4 console logs.
100 # Here is an example log-line:
101 #
102 # 2020-01-23 13:15:12,223 - servo_v4 - EC3PO.Console - DEBUG -
103 # console.py:219:LogConsoleOutput - /dev/pts/9 - cc polarity: cc1
104 #
105 # Here is conceptually how they are formatted:
106 #
107 # <time> - <MCU> - EC3PO.Console - <LVL> - <file:line:func> - <pts> -
108 # <output>
109 #
Garry Wang22f2e842020-09-09 20:19:19 -0700110 # The log format starts with a timestamp
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800111 MCU_RE = (r'[\d\-]+ [\d:,]+ '
112 # The mcu that is logging this is next.
113 r'- (?P<%s>\w+) - '
114 # Next, we have more log outputs before the actual line.
115 # Information about the file line, logging function etc.
116 # Anchor on EC3PO Console, LogConsoleOutput and dev/pts.
117 # NOTE: if the log format changes, this regex needs to be
118 # adjusted.
119 r'EC3PO\.Console[\s\-\w\d:.]+LogConsoleOutput - /dev/pts/\d+ - '
120 # Lastly, we get the MCU's console line.
121 r'(?P<%s>.+$)' % (MCU_GROUP, LINE_GROUP))
122 MCU_EXTRACTOR = re.compile(MCU_RE)
123
Otabek Kasimov545739c2020-08-20 00:24:21 -0700124 # Regex to detect timeout messages when USBC pigtail has timeout issue.
125 # e.g.: [475635.427072 PD TMOUT RX 1/1]
126 USBC_PIGTAIL_TIMEOUT_RE = r'\[[\d \.]{1,20}(PD TMOUT RX 1\/1)\]'
127
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800128 # Suffix to identify compressed logfiles.
129 COMPRESSION_SUFFIX = '.tbz2'
130
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -0700131 # A suffix to mark servod log directories that came from instance that
132 # ran during this servo_host, but are not the last one running e.g. when
133 # an instance (on purpose, or due to a bug) restarted in the middle of the
134 # run.
135 OLD_LOG_SUFFIX = 'old'
136
Otabek Kasimovc6f30412020-06-30 20:08:12 -0700137 # Mapping servo board with their vid-pid
138 SERVO_VID_PID = {
139 'servo_v4':'18d1:501b',
140 'ccd_cr50':'18d1:5014',
141 'servo_micro':'18d1:501a',
142 'servo_v3':['18d1:5004', '0403:6014'],
143 }
144
Otabek Kasimov15963492020-06-23 21:10:51 -0700145 # States of verifiers
146 # True - verifier run and passed
147 # False - verifier run and failed
148 # None - verifier did not run or dependency failed
149 VERIFY_SUCCESS = True
150 VERIFY_FAILED = False
151 VERIFY_NOT_RUN = None
152
Otabek Kasimovcc9738e2020-02-14 16:17:15 -0800153 def _init_attributes(self):
154 self._servo_state = None
155 self.servo_port = None
156 self.servo_board = None
157 self.servo_model = None
158 self.servo_serial = None
Garry Wang000c6c02020-05-11 21:27:23 -0700159 # The flag that indicate if a servo is connected to a smart usbhub.
160 # TODO(xianuowang@) remove this flag once all usbhubs in the lab
161 # get replaced.
162 self.smart_usbhub = None
Otabek Kasimovcc9738e2020-02-14 16:17:15 -0800163 self._servo = None
Andrew McRaef0679932020-08-13 09:15:23 +1000164 self._tunnel_proxy = None
165 self._tunnel_proxy_lock = threading.Lock()
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -0700166 self._initial_instance_ts = None
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800167 # Flag to make sure that multiple calls to close do not result in the
168 # logic executing multiple times.
169 self._closed = False
Andrew McRaef0679932020-08-13 09:15:23 +1000170 # Per-thread local data
171 self._local = threading.local()
Fang Deng5d518f42013-08-02 14:04:32 -0700172
Richard Barnette17bfc6c2016-08-04 18:41:43 -0700173 def _initialize(self, servo_host='localhost',
Richard Barnettee519dcd2016-08-15 17:37:17 -0700174 servo_port=DEFAULT_PORT, servo_board=None,
Nick Sanders2f3c9852018-10-24 12:10:24 -0700175 servo_model=None, servo_serial=None, is_in_lab=None,
176 *args, **dargs):
Fang Deng5d518f42013-08-02 14:04:32 -0700177 """Initialize a ServoHost instance.
178
179 A ServoHost instance represents a host that controls a servo.
180
181 @param servo_host: Name of the host where the servod process
182 is running.
Raul E Rangel52ca2e82018-07-03 14:10:14 -0600183 @param servo_port: Port the servod process is listening on. Defaults
184 to the SERVOD_PORT environment variable if set,
185 otherwise 9999.
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700186 @param servo_board: Board that the servo is connected to.
Nick Sanders2f3c9852018-10-24 12:10:24 -0700187 @param servo_model: Model that the servo is connected to.
Dan Shi4d478522014-02-14 13:46:32 -0800188 @param is_in_lab: True if the servo host is in Cros Lab. Default is set
189 to None, for which utils.host_is_in_lab_zone will be
190 called to check if the servo host is in Cros lab.
Fang Deng5d518f42013-08-02 14:04:32 -0700191
192 """
193 super(ServoHost, self)._initialize(hostname=servo_host,
Garry Wangebc015b2019-06-06 17:45:06 -0700194 is_in_lab=is_in_lab, *args, **dargs)
Otabek Kasimovcc9738e2020-02-14 16:17:15 -0800195 self._init_attributes()
Richard Barnette42f4db92018-08-23 15:05:15 -0700196 self.servo_port = int(servo_port)
Richard Barnettee519dcd2016-08-15 17:37:17 -0700197 self.servo_board = servo_board
Nick Sanders2f3c9852018-10-24 12:10:24 -0700198 self.servo_model = servo_model
Kevin Cheng643ce8a2016-09-15 15:42:12 -0700199 self.servo_serial = servo_serial
Wai-Hong Tam3a8a2552019-11-19 14:28:04 +0800200
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800201 # The location of the log files on the servo host for this instance.
202 self.remote_log_dir = '%s_%s' % (self.SERVOD_LOG_PREFIX,
203 self.servo_port)
Garry Wang79e9af62019-06-12 15:19:19 -0700204 # Path of the servo host lock file.
Derek Beckettf73baca2020-08-19 15:08:47 -0700205 self._lock_file = (self.TEMP_FILE_DIR + str(self.servo_port) +
206 self.LOCK_FILE_POSTFIX)
Garry Wang79e9af62019-06-12 15:19:19 -0700207 # File path to declare a reboot request.
Derek Beckettf73baca2020-08-19 15:08:47 -0700208 self._reboot_file = (self.TEMP_FILE_DIR + str(self.servo_port) +
209 self.REBOOT_FILE_POSTFIX)
Garry Wang79e9af62019-06-12 15:19:19 -0700210
211 # Lock the servo host if it's an in-lab labstation to prevent other
212 # task to reboot it until current task completes. We also wait and
213 # make sure the labstation is up here, in the case of the labstation is
214 # in the middle of reboot.
Garry Wang7c00b0f2019-06-25 17:28:17 -0700215 self._is_locked = False
Garry Wang42b4d862019-06-25 15:50:49 -0700216 if (self.wait_up(self.REBOOT_TIMEOUT) and self.is_in_lab()
217 and self.is_labstation()):
Garry Wang79e9af62019-06-12 15:19:19 -0700218 self._lock()
Garry Wangebc015b2019-06-06 17:45:06 -0700219
Richard Barnette9a26ad62016-06-10 12:03:08 -0700220 self._repair_strategy = (
221 servo_repair.create_servo_repair_strategy())
Richard Barnettee519dcd2016-08-15 17:37:17 -0700222
Dana Goyetteafa62fd2020-03-16 13:45:27 -0700223 def __str__(self):
224 return "<%s '%s:%s'>" % (
225 type(self).__name__, self.hostname, self.servo_port)
226
Richard Barnette9a26ad62016-06-10 12:03:08 -0700227 def connect_servo(self):
Garry Wang8c8dc972020-06-09 13:41:51 -0700228 """ Initialize and setup servo for later use.
229 """
230 self.initilize_servo()
231 self.initialize_dut_for_servo()
232
Garry Wang8c8dc972020-06-09 13:41:51 -0700233 def initilize_servo(self):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700234 """Establish a connection to the servod server on this host.
Richard Barnette9a26ad62016-06-10 12:03:08 -0700235
236 Initializes `self._servo` and then verifies that all network
237 connections are working. This will create an ssh tunnel if
238 it's required.
Garry Wang8c8dc972020-06-09 13:41:51 -0700239 """
240 self._servo = servo.Servo(servo_host=self,
241 servo_serial=self.servo_serial)
Richard Barnette9a26ad62016-06-10 12:03:08 -0700242
Garry Wang8c8dc972020-06-09 13:41:51 -0700243 def initialize_dut_for_servo(self):
244 """This method will do some setup for dut control, e.g. setup
245 main servo_v4 device, and also testing the connection between servo
246 and DUT. As a side effect of testing the connection, all signals on
247 the target servo are reset to default values, and the USB stick is
Richard Barnette9a26ad62016-06-10 12:03:08 -0700248 set to the neutral (off) position.
249 """
Garry Wang8c8dc972020-06-09 13:41:51 -0700250 if not self._servo:
251 raise hosts.AutoservVerifyError('Servo object needs to be'
252 ' initialized before initialize'
253 ' DUT.')
Richard Barnette9a26ad62016-06-10 12:03:08 -0700254 timeout, _ = retry.timeout(
Garry Wang8c8dc972020-06-09 13:41:51 -0700255 self._servo.initialize_dut,
256 timeout_sec=self.INITIALIZE_SERVO_TIMEOUT_SECS)
Richard Barnette9a26ad62016-06-10 12:03:08 -0700257 if timeout:
Garry Wang8c8dc972020-06-09 13:41:51 -0700258 raise hosts.AutoservVerifyError('Initialize dut for servo timed'
259 ' out.')
Richard Barnette9a26ad62016-06-10 12:03:08 -0700260
Richard Barnette9a26ad62016-06-10 12:03:08 -0700261 def disconnect_servo(self):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700262 """Disconnect our servo if it exists.
Richard Barnette9a26ad62016-06-10 12:03:08 -0700263
264 If we've previously successfully connected to our servo,
265 disconnect any established ssh tunnel, and set `self._servo`
266 back to `None`.
267 """
268 if self._servo:
269 # N.B. This call is safe even without a tunnel:
270 # rpc_server_tracker.disconnect() silently ignores
271 # unknown ports.
272 self.rpc_server_tracker.disconnect(self.servo_port)
273 self._servo = None
Fang Deng5d518f42013-08-02 14:04:32 -0700274
Andrew McRaef0679932020-08-13 09:15:23 +1000275 def _maybe_create_servod_ssh_tunnel_proxy(self):
276 """Create a xmlrpc proxy for use with a ssh tunnel.
277 A lock is used to safely create a singleton proxy.
278 """
279 with self._tunnel_proxy_lock:
280 if self._tunnel_proxy is None:
281 self._tunnel_proxy = self.rpc_server_tracker.xmlrpc_connect(
282 None,
283 self.servo_port,
284 ready_test_name=self.SERVO_READY_METHOD,
285 timeout_seconds=60,
286 request_timeout_seconds=3600,
287 server_desc=str(self))
288
Andrew McRaef0679932020-08-13 09:15:23 +1000289 def get_servod_server_proxy(self):
290 """Return a proxy if it exists; otherwise, create a new one.
291 A proxy can either be a ssh tunnel based proxy, or a httplib
292 based proxy.
Fang Deng5d518f42013-08-02 14:04:32 -0700293
294 @returns: An xmlrpclib.ServerProxy that is connected to the servod
295 server on the host.
Fang Deng5d518f42013-08-02 14:04:32 -0700296 """
Garry Wang11b5e872020-03-11 15:14:08 -0700297 if (servo_constants.ENABLE_SSH_TUNNEL_FOR_SERVO
298 and not self.is_localhost()):
Andrew McRaef0679932020-08-13 09:15:23 +1000299 # Check for existing ssh tunnel proxy.
300 if self._tunnel_proxy is None:
301 self._maybe_create_servod_ssh_tunnel_proxy()
302 return self._tunnel_proxy
Richard Barnette9a26ad62016-06-10 12:03:08 -0700303 else:
Andrew McRaef0679932020-08-13 09:15:23 +1000304 # xmlrpc/httplib is not thread-safe, so each thread must have its
305 # own separate proxy connection.
306 if not hasattr(self._local, "_per_thread_proxy"):
307 remote = 'http://%s:%s' % (self.hostname, self.servo_port)
Derek Beckettf73baca2020-08-19 15:08:47 -0700308 self._local._per_thread_proxy = six.moves.xmlrpc_client.ServerProxy(remote)
Andrew McRaef0679932020-08-13 09:15:23 +1000309 return self._local._per_thread_proxy
Wai-Hong Tam3a8a2552019-11-19 14:28:04 +0800310
Richard Barnette1edbb162016-11-01 11:47:50 -0700311 def verify(self, silent=False):
312 """Update the servo host and verify it's in a good state.
313
314 @param silent If true, suppress logging in `status.log`.
315 """
Richard Barnetteabbdc252018-07-26 16:57:42 -0700316 message = 'Beginning verify for servo host %s port %s serial %s'
317 message %= (self.hostname, self.servo_port, self.servo_serial)
318 self.record('INFO', None, None, message)
Richard Barnette9a26ad62016-06-10 12:03:08 -0700319 try:
Richard Barnette1edbb162016-11-01 11:47:50 -0700320 self._repair_strategy.verify(self, silent)
Garry Wang11b5e872020-03-11 15:14:08 -0700321 self._servo_state = servo_constants.SERVO_STATE_WORKING
322 self.record('INFO', None, None,
323 'ServoHost verify set servo_state as WORKING')
Garry Wang63b8c382020-03-11 22:28:40 -0700324 except Exception as e:
Otabek Kasimov120b6fa2020-07-03 00:15:27 -0700325 if not self.is_localhost():
Otabek Kasimovc6f30412020-06-30 20:08:12 -0700326 self._servo_state = self.determine_servo_state()
327 self.record('INFO', None, None,
328 'ServoHost verify set servo_state as %s'
329 % self._servo_state)
Garry Wang63b8c382020-03-11 22:28:40 -0700330 if self._is_critical_error(e):
331 raise
Fang Deng5d518f42013-08-02 14:04:32 -0700332
Garry Wang2b5eef92020-08-21 16:23:35 -0700333 def _get_default_usbkey_mount_path(self):
334 return '/media/servo_usb/%s' % self.servo_port
Fang Deng5d518f42013-08-02 14:04:32 -0700335
Garry Wang7b0e1b72020-03-25 19:08:59 -0700336 def get_image_name_from_usbkey(self, usbkey_dev):
337 """Mount usb drive and check ChromeOS image name on it if there is
338 one. This method assumes the image_usbkey_direction is already set
339 to servo side.
340
341 @param: usbkey_dev usbkey dev path(e.g. /dev/sdb).
342
343 @returns: image_name on the usbkey, e.g. nami-release/R82.10138.0.0,
344 or empty string if no test image detected, or unexpected
345 error occurred.
Garry Wang7b0e1b72020-03-25 19:08:59 -0700346 """
Garry Wang70e5d062020-04-03 18:01:05 -0700347 logging.info('Checking ChromeOS image name on usbkey.')
Garry Wang2b5eef92020-08-21 16:23:35 -0700348 mount_dst = self._get_default_usbkey_mount_path()
Garry Wang7b0e1b72020-03-25 19:08:59 -0700349 # Unmount if there is an existing stale mount.
Garry Wang2b5eef92020-08-21 16:23:35 -0700350 self._unmount_drive(mount_dst)
351 # ChromeOS root fs is in /dev/sdx3
352 mount_src = usbkey_dev + '3'
Garry Wang7b0e1b72020-03-25 19:08:59 -0700353 try:
Garry Wang2b5eef92020-08-21 16:23:35 -0700354 if not self._mount_drive(mount_src, mount_dst):
355 logging.debug('Unexpected error occurred on mount usb drive.')
Garry Wang7b0e1b72020-03-25 19:08:59 -0700356 return ''
Garry Wang70e5d062020-04-03 18:01:05 -0700357
358 release_content = self.run(
Garry Wang2b5eef92020-08-21 16:23:35 -0700359 'cat %s/etc/lsb-release' % mount_dst,
Garry Wang70e5d062020-04-03 18:01:05 -0700360 ignore_status=True).stdout.strip()
361
362 if not re.search(r'RELEASE_TRACK=.*test', release_content):
363 logging.info('The image on usbkey is not a test image')
364 return ''
365
366 return lsbrelease_utils.get_chromeos_release_builder_path(
367 lsb_release_content=release_content)
Garry Wang7b0e1b72020-03-25 19:08:59 -0700368 finally:
Garry Wang70e5d062020-04-03 18:01:05 -0700369 logging.debug('Image check compeleted, unmounting the usb drive.')
Garry Wang2b5eef92020-08-21 16:23:35 -0700370 self._unmount_drive(mount_dst)
Garry Wang7b0e1b72020-03-25 19:08:59 -0700371
Garry Wang2b5eef92020-08-21 16:23:35 -0700372 def _extract_firmware_image_from_usbkey(self, fw_dst):
373 """Extract firmware images from the usbkey on servo, this method
374 assumes there is already a ChromeOS test image staged on servo.
375
376 @param: fw_dst the path that we'll copy firmware images to.
377
378 @returns: a json format string of firmware manifest data.
379 """
380 usbkey_dev = self._probe_and_validate_usb_dev()
381 if not usbkey_dev:
382 raise hosts.AutoservRepairError('Unexpected error occurred when'
383 ' probe usbkey dev path, please check logs for detail.')
384
385 mount_dst = self._get_default_usbkey_mount_path()
386 # Unmount if there is an existing stale mount.
387 self._unmount_drive(mount_dst)
388 # ChromeOS root fs is in /dev/sdx3
389 mount_src = usbkey_dev + '3'
390 try:
391 if not self._mount_drive(mount_src, mount_dst):
392 raise hosts.AutoservRepairError('Failed to extract firmware'
393 ' image; Unable to mount %s.' % usbkey_dev,
394 'unable to mount usbkey')
395 updater_bin = os.path.join(mount_dst,
396 'usr/sbin/chromeos-firmwareupdate')
397 self.run('%s --unpack %s' % (updater_bin, fw_dst))
398 return self.run('%s --manifest' % updater_bin).stdout
399 finally:
400 self._unmount_drive(mount_dst)
401
402 def prepare_repair_firmware_image(self, fw_dst=None):
403 """Prepare firmware image on the servohost for auto repair process
404 to consume.
405
406 @param: fw_dst the path that we want to store firmware image on
407 the servohost.
408
409 @returns: A tuple that containes ec firmware image path and bios
410 firmware image path on the servohost, or None if type of
411 image is not available based on manifest and dut's model.
412 """
413 model = self.servo_model or self._dut_host_info.model
414 if not model:
415 raise hosts.AutoservRepairError(
416 'Could not determine DUT\'s model.',
417 'model infomation unknown')
418
419 if not fw_dst:
420 fw_dst = '/tmp/firmware_image/%s' % self.servo_port
421 # Cleanup and re-create dst path to have a fresh start.
422 self.run('rm -rf %s' % fw_dst)
423 self.run('mkdir -p %s' % fw_dst)
424
425 manifest = json.loads(self._extract_firmware_image_from_usbkey(fw_dst))
426 model_manifest = manifest.get(model)
427 if not model_manifest:
428 raise hosts.AutoservRepairError('Could not find firmware manifest'
429 ' for model:%s' % model, 'model manifest not found')
430 try:
431 ec_image = os.path.join(fw_dst, model_manifest['ec']['image'])
432 except KeyError:
433 ec_image = None
434 try:
435 bios_image = os.path.join(fw_dst, model_manifest['host']['image'])
436 except KeyError:
437 bios_image = None
438 if not ec_image and not bios_image:
439 raise hosts.AutoservRepairError('Could not find any firmware image'
440 ' for model:%s' % model, 'cannot find firmware image')
441 return ec_image, bios_image
Garry Wang7b0e1b72020-03-25 19:08:59 -0700442
Garry Wang70e5d062020-04-03 18:01:05 -0700443 def _probe_and_validate_usb_dev(self):
444 """This method probe the usb dev path by talking to servo, and then
445 validate the dev path is valid block device to servohost.
446 Possible output:
447 1. Encounter error during probe usb dev, returns empty string.
448 2. probe usb dev completed without error but cannot find usb dev,
449 raise AutoservRepairError.
450 3. probe usb dev find a usb dev path, but failed validation in this
451 method, raise AutoservRepairError.
Garry Wang7b0e1b72020-03-25 19:08:59 -0700452
Garry Wang70e5d062020-04-03 18:01:05 -0700453 @returns: A string of usb dev path(e.g. '/dev/sdb'), or empty string
454 if unexpected error occurred during probe.
455 @raises: AutoservRepairError if servo couldn't probe the usb dev path
456 (servo.probe_host_usb_dev() returns empty string), or the dev path is
457 not valid block device to servohost.
Garry Wang7b0e1b72020-03-25 19:08:59 -0700458 """
459 logging.info('Validating image usbkey on servo.')
Garry Wang7b0e1b72020-03-25 19:08:59 -0700460 try:
Garry Wang70e5d062020-04-03 18:01:05 -0700461 usb_dev = self._servo.probe_host_usb_dev()
Garry Wang7b0e1b72020-03-25 19:08:59 -0700462 except Exception as e:
463 # We don't want any unexpected or transient servo communicating
464 # failure block usb repair, so capture all errors here.
465 logging.error(e, exc_info=True)
466 logging.error('Unexpected error occurred on get usbkey dev path,'
467 ' skipping usbkey validation.')
468 return ''
469
Garry Wang70e5d062020-04-03 18:01:05 -0700470 if usb_dev:
471 # probe_host_usb_dev() sometimes return stale record,
472 # so we need to make sure the path exists in fdisk.
473 validate_cmd = 'fdisk -l | grep %s' % usb_dev
Garry Wang11441182020-06-16 18:34:14 -0700474 try:
475 resp = self.run(validate_cmd, ignore_status=True, timeout=60)
476 if resp.exit_status == 0:
477 return usb_dev
Garry Wang70e5d062020-04-03 18:01:05 -0700478
Garry Wang11441182020-06-16 18:34:14 -0700479 logging.error('%s is reported from "image_usbkey_dev" control'
480 ' but not detected by fdisk!', usb_dev)
481 except error.AutoservRunError as e:
482 if 'Timeout encountered' in str(e):
483 logging.warning('Timeout encountered during fdisk run,'
484 ' skipping usbkey validation.')
485 return ''
486 raise
Garry Wang70e5d062020-04-03 18:01:05 -0700487
488 raise hosts.AutoservRepairError(
489 'No usbkey detected on servo, the usbkey may be either missing'
490 ' or broken. Please replace usbkey on the servo and retry.',
491 'missing usbkey')
492
Otabek Kasimov4ea636e2020-04-14 23:35:06 -0700493 def is_ec_supported(self):
Garry Wang9b8f2342020-04-17 16:34:09 -0700494 """Check if ec is supported on the servo_board"""
Otabek Kasimov4ea636e2020-04-14 23:35:06 -0700495 if self.servo_board:
496 try:
497 frm_config = config.Config(self.servo_board, self.servo_model)
498 return frm_config.chrome_ec
499 except Exception as e:
500 logging.error('Unexpected error when read from firmware'
501 ' configs; %s', str(e))
502 return False
503
Garry Wang70e5d062020-04-03 18:01:05 -0700504 def validate_image_usbkey(self):
505 """This method first validate if there is a recover usbkey on servo
506 that accessible to servohost, and second check if a ChromeOS image is
507 already on the usb drive and return the image_name so we can avoid
508 unnecessary download and flash to the recover usbkey on servo.
509
510 Please note that, there is special error handling logic here:
511 1. If unexpected error happens, we return empty string. So repair
512 actions will not get blocked.
513 2. If no working usbkey present on servo, but no errors, we'll raise
514 AutoservRepairError here.
515
516 @returns: image_name on the usbkey, e.g. nami-release/R82.10138.0.0,
517 or empty string if no test image detected, or unexpected
518 error occurred.
519 @raises: AutoservRepairError if the usbkey is not detected on servo.
520 """
521 usb_dev = self._probe_and_validate_usb_dev()
522 if usb_dev:
523 return self.get_image_name_from_usbkey(usb_dev)
524 else:
525 return ''
Garry Wang7b0e1b72020-03-25 19:08:59 -0700526
Richard Barnette1edbb162016-11-01 11:47:50 -0700527 def repair(self, silent=False):
528 """Attempt to repair servo host.
529
530 @param silent If true, suppress logging in `status.log`.
531 """
Richard Barnetteabbdc252018-07-26 16:57:42 -0700532 message = 'Beginning repair for servo host %s port %s serial %s'
533 message %= (self.hostname, self.servo_port, self.servo_serial)
534 self.record('INFO', None, None, message)
Richard Barnette9a26ad62016-06-10 12:03:08 -0700535 try:
Richard Barnette1edbb162016-11-01 11:47:50 -0700536 self._repair_strategy.repair(self, silent)
Garry Wang11b5e872020-03-11 15:14:08 -0700537 self._servo_state = servo_constants.SERVO_STATE_WORKING
538 self.record('INFO', None, None,
539 'ServoHost repair set servo_state as WORKING')
Garry Wang464ff1e2019-07-18 17:20:34 -0700540 # If target is a labstation then try to withdraw any existing
541 # reboot request created by this servo because it passed repair.
542 if self.is_labstation():
543 self.withdraw_reboot_request()
Garry Wang63b8c382020-03-11 22:28:40 -0700544 except Exception as e:
Otabek Kasimov120b6fa2020-07-03 00:15:27 -0700545 if not self.is_localhost():
Otabek Kasimovc6f30412020-06-30 20:08:12 -0700546 self._servo_state = self.determine_servo_state()
547 self.record('INFO', None, None,
548 'ServoHost repair set servo_state as %s'
549 % self._servo_state)
Garry Wang63b8c382020-03-11 22:28:40 -0700550 if self._is_critical_error(e):
551 self.disconnect_servo()
552 self.stop_servod()
553 raise
554
Garry Wang63b8c382020-03-11 22:28:40 -0700555 def _is_critical_error(self, error):
556 if (isinstance(error, hosts.AutoservVerifyDependencyError)
557 and not error.is_critical()):
558 logging.warning('Non-critical verify failure(s) detected during'
559 ' verify/repair servo, servo connection will'
560 ' still up but may not fully functional.'
561 ' Some repair actions and servo depended'
562 ' tests may not run.')
563 return False
564 logging.info('Critical verify failure(s) detected during repair/verify'
565 ' servo. Disconnecting servo and stop servod, all repair '
566 'action and tests that depends on servo will not run.')
567 return True
Fang Deng5d518f42013-08-02 14:04:32 -0700568
Dan Shi4d478522014-02-14 13:46:32 -0800569 def get_servo(self):
570 """Get the cached servo.Servo object.
Fang Deng5d518f42013-08-02 14:04:32 -0700571
Dan Shi4d478522014-02-14 13:46:32 -0800572 @return: a servo.Servo object.
Dana Goyette353d1d92019-06-27 10:43:59 -0700573 @rtype: autotest_lib.server.cros.servo.servo.Servo
Fang Deng5d518f42013-08-02 14:04:32 -0700574 """
Dan Shi4d478522014-02-14 13:46:32 -0800575 return self._servo
576
Garry Wang79e9af62019-06-12 15:19:19 -0700577 def request_reboot(self):
578 """Request servohost to be rebooted when it's safe to by touch a file.
579 """
580 logging.debug('Request to reboot servohost %s has been created by '
Garry Wang464ff1e2019-07-18 17:20:34 -0700581 'servo with port # %s', self.hostname, self.servo_port)
Garry Wang79e9af62019-06-12 15:19:19 -0700582 self.run('touch %s' % self._reboot_file, ignore_status=True)
583
Garry Wang464ff1e2019-07-18 17:20:34 -0700584 def withdraw_reboot_request(self):
585 """Withdraw a servohost reboot request if exists by remove the flag
586 file.
587 """
588 logging.debug('Withdrawing request to reboot servohost %s that created'
589 ' by servo with port # %s if exists.',
590 self.hostname, self.servo_port)
591 self.run('rm -f %s' % self._reboot_file, ignore_status=True)
592
Garry Wangc1288cf2019-12-17 14:58:00 -0800593 def start_servod(self, quick_startup=False):
594 """Start the servod process on servohost.
595 """
Garry Wang2ac15ee2019-12-30 19:03:02 -0800596 # Skip if running on the localhost.(crbug.com/1038168)
597 if self.is_localhost():
598 logging.debug("Servohost is a localhost, skipping start servod.")
599 return
600
601 cmd = 'start servod'
Garry Wangc1288cf2019-12-17 14:58:00 -0800602 if self.servo_board:
Garry Wang2ac15ee2019-12-30 19:03:02 -0800603 cmd += ' BOARD=%s' % self.servo_board
Garry Wangc1288cf2019-12-17 14:58:00 -0800604 if self.servo_model:
605 cmd += ' MODEL=%s' % self.servo_model
Garry Wangc1288cf2019-12-17 14:58:00 -0800606 else:
Garry Wang2ac15ee2019-12-30 19:03:02 -0800607 logging.warning('Board for DUT is unknown; starting servod'
608 ' assuming a pre-configured board.')
609
610 cmd += ' PORT=%d' % self.servo_port
611 if self.servo_serial:
612 cmd += ' SERIAL=%s' % self.servo_serial
Garry Wangd7367482020-02-27 13:52:40 -0800613
614 # Start servod with dual_v4 if the DUT/servo from designated pools.
615 dut_host_info = self.get_dut_host_info()
616 if dut_host_info:
Otabek Kasimovf10a7052020-08-14 03:09:23 -0700617 # DUAL_V4: servo setup includes servo_micro and ccd_cr50
618 # connection to the DUT
619 is_dual_setup = False
Garry Wang11b5e872020-03-11 15:14:08 -0700620 if bool(dut_host_info.pools &
621 servo_constants.POOLS_SUPPORT_DUAL_V4):
Garry Wangd7367482020-02-27 13:52:40 -0800622 logging.debug('The DUT is detected in following designated'
623 ' pools %s,starting servod with DUAL_V4 option.',
Garry Wang11b5e872020-03-11 15:14:08 -0700624 servo_constants.POOLS_SUPPORT_DUAL_V4)
Otabek Kasimovf10a7052020-08-14 03:09:23 -0700625 is_dual_setup = True
626 elif dut_host_info.attributes.get('servo_setup') == 'DUAL_V4':
627 logging.debug('The DUT servo setup specified in config as '
628 ' "DUAL_V4"')
629 is_dual_setup = True
630 if is_dual_setup:
Garry Wangd7367482020-02-27 13:52:40 -0800631 cmd += ' DUAL_V4=1'
632
Garry Wangb5cee3e2020-09-16 14:58:13 -0700633 # Start servod with CONFIG=cr50.xml which required for some pools.
634 if self._require_cr50_servod_config():
635 cmd += ' CONFIG=cr50.xml'
636
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800637 # Remove the symbolic links from the logs. This helps ensure that
638 # a failed servod instantiation does not cause us to grab old logs
639 # by mistake.
640 self.remove_latest_log_symlinks()
Garry Wangcdd27b22020-01-13 14:59:11 -0800641 self.run(cmd, timeout=60)
Garry Wangc1288cf2019-12-17 14:58:00 -0800642
643 # There's a lag between when `start servod` completes and when
644 # the _ServodConnectionVerifier trigger can actually succeed.
645 # The call to time.sleep() below gives time to make sure that
646 # the trigger won't fail after we return.
647
648 # Normally servod on servo_v3 and labstation take ~10 seconds to ready,
649 # But in the rare case all servo on a labstation are in heavy use they
650 # may take ~30 seconds. So the timeout value will double these value,
651 # and we'll try quick start up when first time initialize servohost,
652 # and use standard start up timeout in repair.
653 if quick_startup:
Garry Wang11b5e872020-03-11 15:14:08 -0700654 timeout = servo_constants.SERVOD_QUICK_STARTUP_TIMEOUT
Garry Wangc1288cf2019-12-17 14:58:00 -0800655 else:
Garry Wang11b5e872020-03-11 15:14:08 -0700656 timeout = servo_constants.SERVOD_STARTUP_TIMEOUT
Garry Wangc1288cf2019-12-17 14:58:00 -0800657 logging.debug('Wait %s seconds for servod process fully up.', timeout)
658 time.sleep(timeout)
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -0700659 # Cache the initial instance timestamp to check against servod restarts
660 self._initial_instance_ts = self.get_instance_logs_ts()
Garry Wangc1288cf2019-12-17 14:58:00 -0800661
Garry Wangc1288cf2019-12-17 14:58:00 -0800662 def stop_servod(self):
663 """Stop the servod process on servohost.
664 """
Garry Wang2ac15ee2019-12-30 19:03:02 -0800665 # Skip if running on the localhost.(crbug.com/1038168)
666 if self.is_localhost():
667 logging.debug("Servohost is a localhost, skipping stop servod.")
668 return
669
Garry Wangc1288cf2019-12-17 14:58:00 -0800670 logging.debug('Stopping servod on port %s', self.servo_port)
Garry Wangcdd27b22020-01-13 14:59:11 -0800671 self.run('stop servod PORT=%d' % self.servo_port,
672 timeout=60, ignore_status=True)
Garry Wangc1288cf2019-12-17 14:58:00 -0800673 logging.debug('Wait %s seconds for servod process fully teardown.',
Garry Wang11b5e872020-03-11 15:14:08 -0700674 servo_constants.SERVOD_TEARDOWN_TIMEOUT)
675 time.sleep(servo_constants.SERVOD_TEARDOWN_TIMEOUT)
Garry Wangc1288cf2019-12-17 14:58:00 -0800676
Garry Wangc1288cf2019-12-17 14:58:00 -0800677 def restart_servod(self, quick_startup=False):
678 """Restart the servod process on servohost.
679 """
680 self.stop_servod()
681 self.start_servod(quick_startup)
682
Garry Wangffbd2162020-04-17 16:13:48 -0700683 def _process_servodtool_error(self, response):
684 """Helper function to handle non-zero servodtool response.
685 """
686 if re.search(servo_constants.ERROR_MESSAGE_USB_HUB_NOT_COMPATIBLE,
Garry Wangad245002020-05-15 15:20:23 -0700687 response.stdout):
Garry Wangffbd2162020-04-17 16:13:48 -0700688 logging.error('The servo is not plugged on a usb hub that supports'
689 ' power-cycle!')
Garry Wang000c6c02020-05-11 21:27:23 -0700690 # change the flag so we can update this label in later process.
691 self.smart_usbhub = False
Garry Wangffbd2162020-04-17 16:13:48 -0700692 return
693
694 if re.search(servo_constants.ERROR_MESSAGE_DEVICE_NOT_FOUND %
695 self.servo_serial, response.stdout):
696 logging.error('No servo with serial %s found!', self.servo_serial)
697 return
698
699 logging.error('Unexpected error occurred from usbhub control, please'
700 ' file a bug and inform chrome-fleet-software@ team!')
701
Otabek Kasimov545739c2020-08-20 00:24:21 -0700702 def _is_usbc_pigtail_connection_timeout(self):
703 """Check if servo has issue with USBC pigtail connection timeout.
704
705 The usb_console has to be clean for good servo. If console generate
706 messages like (below) then issue is present:
707 [475635.427072 PD TMOUT RX 1/1]
708 RXERR1 Preamble
709 [475635.476044 PD TMOUT RX 1/1]
710 RXERR1 Preamble
711 """
712 if not self.servo_serial:
713 return False
714 logging.debug('Starting check if USBC pigtail connection timeout.')
715 try:
716 cmd = 'usb_console -d 18d1:501b -s %s' % self.servo_serial
717 resp = self.run(cmd, timeout=self.DEFAULT_TERMINAL_TIMEOUT)
718 result_lines = resp.stdout.splitlines()
719 for line in result_lines:
720 if re.match(self.USBC_PIGTAIL_TIMEOUT_RE, line):
721 return True
722 except Exception as e:
723 logging.debug('(Non-critical) %s.', e)
724 return False
725
726 def _reset_usbc_pigtail_connection(self):
727 """Reset USBC pigtail connection on servo board.
728
729 To reset need to run 'cc off' and then 'cc srcdts' in usb_console.
730 """
731 if not self.servo_serial:
732 return False
733 logging.debug('Starting reset USBC pigtail connection.')
734 def _run_command(cc_command):
735 """Run configuration chanel commands.
736
737 @returns: True if pas successful and False if fail.
738 """
739 try:
740 cmd = (r"echo 'cc %s' | usb_console -d 18d1:501b -s %s"
741 % (cc_command, self.servo_serial))
742 resp = self.run(cmd, timeout=self.DEFAULT_TERMINAL_TIMEOUT)
743 return True
744 except Exception as e:
745 logging.info('(Non-critical) %s.', e)
746 return False
747
748 logging.info('Turn off configuration channel. And wait 5 seconds.')
749 if _run_command('off'):
750 # wait till command will be effected
751 time.sleep(5)
752 logging.info('Turn on configuration channel. '
753 'And wait 15 seconds.')
754 if _run_command('srcdts'):
755 # wait till command will be effected
756 time.sleep(15)
757
758 def reset_usbc_pigtail_connection_on_need(self):
759 """Reset USBC pitgtail issue if it present."""
760 if not self.is_labstation():
761 logging.info('USBC pigtail reset applicable only for labstations')
762 return
763
764 if self._is_usbc_pigtail_connection_timeout():
765 logging.info('USBC pigtail issue detected on servo.')
766 self._reset_usbc_pigtail_connection()
767 fields = self._get_host_metrics_data()
768 fields['success'] = not self._is_usbc_pigtail_connection_timeout()
769 metrics.Counter(
770 'chromeos/autotest/repair/servo_usbc/reset'
771 ).increment(fields=fields)
Garry Wangffbd2162020-04-17 16:13:48 -0700772
773 def _get_servo_usb_devnum(self):
774 """Helper function to collect current usb devnum of servo.
775 """
Otabek Kasimov09192682020-06-01 18:17:44 -0700776 # TODO remove try-except when fix crbug.com/1087964
777 try:
778 cmd = 'servodtool device -s %s usb-path' % self.servo_serial
779 resp = self.run(cmd, ignore_status=True, timeout=30)
780 except Exception as e:
781 # Here we catch only timeout errors.
782 # Other errors is filtered by ignore_status=True
783 logging.debug('Attempt to get servo usb-path failed due to '
784 'timeout; %s', e)
785 return ''
786
Garry Wangffbd2162020-04-17 16:13:48 -0700787 if resp.exit_status != 0:
788 self._process_servodtool_error(resp)
789 return ''
790 usb_path = resp.stdout.strip()
791 logging.info('Usb path of servo %s is %s', self.servo_serial, usb_path)
792
793 resp = self.run('cat %s/devnum' % usb_path,
794 ignore_status=True)
795 if resp.exit_status != 0:
796 self._process_servodtool_error(resp)
797 return ''
798 return resp.stdout.strip()
799
Garry Wang358aad42020-08-02 20:56:04 -0700800 def reboot_servo_v3_on_need(self):
801 """Check and reboot servo_v3 based on below conditions.
802 1. If there is an update pending on reboot.
803 2. Servo_v3 has been up for more than 96 hours.
804 """
805 if self.get_board() != 'beaglebone_servo':
806 logging.info('Servo reboot is only applicable for servo V3.')
Otabek Kasimove6df8102020-07-21 20:15:25 -0700807 return
808
Garry Wang358aad42020-08-02 20:56:04 -0700809 update_pending_reboot = (self._check_update_status() ==
810 self.UPDATE_STATE.PENDING_REBOOT)
811 uptime_hours = float(self.check_uptime())/3600
812 logging.info('Uptime of servo_v3: %s hour(s)', uptime_hours)
813 long_up_time = uptime_hours > 96
814
815 # Skip reboot if neither condition are met.
816 if not (update_pending_reboot or long_up_time):
Otabek Kasimove6df8102020-07-21 20:15:25 -0700817 return
818
Garry Wang358aad42020-08-02 20:56:04 -0700819 if update_pending_reboot:
820 message = 'Starting reboot servo_v3 because an update is pending.'
821 reboot_method = self._post_update_reboot
822 elif long_up_time:
823 message = 'Starting reboot servo_v3 because uptime > 96 hours.'
824 reboot_method = self._servo_host_reboot
825 self.record('INFO', None, None, message)
826 logging.info(message)
Otabek Kasimove6df8102020-07-21 20:15:25 -0700827 try:
Garry Wang358aad42020-08-02 20:56:04 -0700828 reboot_method()
Otabek Kasimove6df8102020-07-21 20:15:25 -0700829 message = 'Servo_v3 reboot completed successfully.'
830 except Exception as e:
831 logging.debug("Fail to reboot servo_v3; %s", e)
832 message = ('Servo_v3 reboot failed, please check debug log '
833 'for details.')
834 logging.info(message)
835 self.record('INFO', None, None, message)
Garry Wangffbd2162020-04-17 16:13:48 -0700836
837 def _reset_servo(self):
838 logging.info('Resetting servo through smart usbhub.')
Otabek Kasimov09192682020-06-01 18:17:44 -0700839 # TODO remove try-except when fix crbug.com/1087964
840 try:
841 resp = self.run('servodtool device -s %s power-cycle' %
842 self.servo_serial, ignore_status=True,
843 timeout=30)
844 if resp.exit_status != 0:
845 self._process_servodtool_error(resp)
846 return False
847 except Exception as e:
848 # Here we catch only timeout errors.
849 # Other errors is filtered by ignore_status=True
850 logging.debug('Attempt to reset servo failed due to timeout;'
851 ' %s', e)
Garry Wangffbd2162020-04-17 16:13:48 -0700852 return False
853
854 logging.debug('Wait %s seconds for servo to come back from reset.',
855 servo_constants.SERVO_RESET_TIMEOUT_SECONDS)
856 time.sleep(servo_constants.SERVO_RESET_TIMEOUT_SECONDS)
Garry Wang000c6c02020-05-11 21:27:23 -0700857 # change the flag so we can update this label in later process.
858 self.smart_usbhub = True
Garry Wangffbd2162020-04-17 16:13:48 -0700859 return True
860
Garry Wangffbd2162020-04-17 16:13:48 -0700861 def reset_servo(self):
862 """Reset(power-cycle) the servo via smart usbhub.
863 """
864 if not self.is_labstation():
865 logging.info('Servo reset is not applicable to servo_v3.')
866 return
867
868 pre_reset_devnum = self._get_servo_usb_devnum()
869 logging.info('Servo usb devnum before reset: %s', pre_reset_devnum)
870 result = self._reset_servo()
871 if not result:
Garry Wangfd5c8b62020-06-08 15:36:54 -0700872 message = ('Failed to reset servo with serial: %s. (Please ignore'
873 ' this error if the DUT is not connected to a smart'
874 ' usbhub).' % self.servo_serial)
Garry Wangffbd2162020-04-17 16:13:48 -0700875 logging.warning(message)
876 self.record('INFO', None, None, message)
877 return
878
879 post_reset_devnum = self._get_servo_usb_devnum()
880 logging.info('Servo usb devnum after reset: %s', post_reset_devnum)
881 if not (pre_reset_devnum and post_reset_devnum):
882 message = ('Servo reset completed but unable to verify'
883 ' devnum change!')
884 elif pre_reset_devnum != post_reset_devnum:
885 message = ('Reset servo with serial %s completed successfully!'
886 % self.servo_serial)
887 else:
888 message = 'Servo reset completed but devnum is still not changed!'
889 logging.info(message)
890 self.record('INFO', None, None, message)
891
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800892 def _extract_compressed_logs(self, logdir, relevant_files):
893 """Decompress servod logs in |logdir|.
894
895 @param logdir: directory containing compressed servod logs.
896 @param relevant_files: list of files in |logdir| to consider.
897
898 @returns: tuple, (tarfiles, files) where
899 tarfiles: list of the compressed filenames that have been
900 extracted and deleted
901 files: list of the uncompressed files that were generated
902 """
903 # For all tar-files, first extract them to the directory, and
904 # then let the common flow handle them.
905 tarfiles = [cf for cf in relevant_files if
906 cf.endswith(self.COMPRESSION_SUFFIX)]
907 files = []
908 for f in tarfiles:
909 norm_name = os.path.basename(f)[:-len(self.COMPRESSION_SUFFIX)]
910 with tarfile.open(f) as tf:
911 # Each tarfile has only one member, as
912 # that's the compressed log.
913 member = tf.members[0]
914 # Manipulate so that it only extracts the basename, and not
915 # the directories etc.
916 member.name = norm_name
917 files.append(os.path.join(logdir, member.name))
918 tf.extract(member, logdir)
919 # File has been extracted: remove the compressed file.
920 os.remove(f)
921 return tarfiles, files
922
923 def _extract_mcu_logs(self, log_subdir):
924 """Extract MCU (EC, Cr50, etc) console output from servod debug logs.
925
926 Using the MCU_EXTRACTOR regex (above) extract and split out MCU console
927 lines from the logs to generate invidiual console logs e.g. after
928 this method, you can find an ec.txt and servo_v4.txt in |log_dir| if
929 those MCUs had any console input/output.
930
931 @param log_subdir: directory with log.DEBUG.txt main servod debug logs.
932 """
933 # Extract the MCU for each one. The MCU logs are only in the .DEBUG
934 # files
935 mcu_lines_file = os.path.join(log_subdir, 'log.DEBUG.txt')
936 if not os.path.exists(mcu_lines_file):
937 logging.info('No DEBUG logs found to extract MCU logs from.')
938 return
939 mcu_files = {}
940 mcu_file_template = '%s.txt'
941 with open(mcu_lines_file, 'r') as f:
942 for line in f:
943 match = self.MCU_EXTRACTOR.match(line)
944 if match:
945 mcu = match.group(self.MCU_GROUP).lower()
946 line = match.group(self.LINE_GROUP)
947 if mcu not in mcu_files:
948 mcu_file = os.path.join(log_subdir,
949 mcu_file_template % mcu)
950 mcu_files[mcu] = open(mcu_file, 'a')
951 fd = mcu_files[mcu]
952 fd.write(line + '\n')
953 for f in mcu_files:
954 mcu_files[f].close()
955
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800956 def remove_latest_log_symlinks(self):
957 """Remove the conveninence symlinks 'latest' servod logs."""
958 symlink_wildcard = '%s/latest*' % self.remote_log_dir
959 cmd = 'rm ' + symlink_wildcard
960 self.run(cmd, stderr_tee=None, ignore_status=True)
961
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -0700962 def probe_servod_restart(self, instance_ts, outdir):
963 """Grab servod logs from previous instances if part of this session.
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800964
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -0700965 If since the last time this host called start_servod() servod crashed
966 and restarted, this helper finds those logs as well, and stores them
967 with the |OLD_LOG_SUFFIX| to investigate if necessary.
Prasad Vuppalapu5bd9da12020-03-31 01:46:47 +0000968
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -0700969 It also issues a panicinfo command to servo devices after the restart
970 to try and collect reboot information for debugging.
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800971
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -0700972 @param instance_ts: the log timestamp that the current instance uses
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800973 @param outdir: directory to create a subdirectory into to place the
974 servod logs into.
975 """
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -0700976 if self._initial_instance_ts is None:
977 logging.info('No log timestamp grabbed successfully on servod '
978 'startup. Cannot check device restarts. Ignoring.')
979 return
980 if instance_ts == self._initial_instance_ts:
981 logging.debug('Servod appears to have run without restarting')
982 return
983 # Servod seems to have restarted (at least once). |_initial_instance_ts|
984 # is the first timestamp, and instance_ts is the current timestamp. Find
985 # all timestamps in between them, and grab the logs for each.
986 tss = self._find_instance_timestamps_between(self._initial_instance_ts,
987 instance_ts)
988 logging.info('Servod has restarted %d times between the start and the '
989 'end of this servo_host.', len(tss))
990 logging.info('This might be an issue. Will extract all logs from each '
991 'instance.')
992 logging.info('Logs that are not the currently running (about to turn '
993 'down) instance are maked with a .%s in their folder.',
994 self.OLD_LOG_SUFFIX)
995 for ts in tss:
996 self.get_instance_logs(ts, outdir, old=True)
997 # Lastly, servod has restarted due to a potential issue. Try to get
998 # panic information from servo micro and servo v4 for the current logs.
999 # This can only happen if the |_servo| attribute is initialized.
1000 if self._servo:
1001 for mcu in ['servo_micro', 'servo_v4']:
1002 ctrl = '%s_uart_cmd' % mcu
1003 if self._servo.has_control(ctrl):
1004 logging.info('Trying to retrieve %r panicinfo into logs',
1005 mcu)
1006 try:
1007 self._servo.set_nocheck(ctrl, 'panicinfo')
1008 except error.TestFail as e:
1009 logging.error('Failed to generate panicinfo for %r '
1010 'logs. %s', mcu, str(e))
1011
1012 def _find_instance_timestamps_between(self, start_ts, end_ts):
1013 """Find all log timestamps between [start_ts, end_ts).
1014
1015 @param start_ts: str, earliest log timestamp of interest
1016 @param end_ts: str, latest log timestamp of interest
1017
1018 @returns: list, all timestamps between start_ts and end_ts, end_ts
1019 exclusive, on the servo_host. An empty list on errors
1020 """
1021 # Simply get all timestamp, and then sort and remove
1022 cmd = 'ls %s' % self.remote_log_dir
1023 res = self.run(cmd, stderr_tee=None, ignore_status=True)
1024 if res.exit_status != 0:
1025 # Here we failed to find anything.
1026 logging.info('Failed to find remote servod logs. Ignoring.')
1027 return []
1028 logfiles = res.stdout.strip().split()
1029 timestamps = set()
1030 for logfile in logfiles:
1031 ts_match = self.TS_EXTRACTOR.match(logfile)
1032 if not ts_match:
1033 # Simply ignore files that fail the check. It might be the
1034 # 'latest' symlinks or random files.
1035 continue
1036 timestamps.add(ts_match.group(self.TS_GROUP))
1037 # At this point we have all unique timestamps.
1038 timestamps = sorted(timestamps)
1039 for ts in [start_ts, end_ts]:
1040 if ts not in timestamps:
1041 logging.error('Timestamp %r not in servod logs. Cannot query '
1042 'for timestamps in between %r and %r', ts,
1043 start_ts, end_ts)
1044 return []
1045 return timestamps[timestamps.index(start_ts):timestamps.index(end_ts)]
1046
1047 def get_instance_logs_ts(self):
1048 """Retrieve the currently running servod instance's log timestamp
1049
1050 @returns: str, timestamp for current instance, or None on failure
1051 """
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -08001052 # First, extract the timestamp. This cmd gives the real filename of
1053 # the latest aka current log file.
1054 cmd = ('if [ -f %(dir)s/latest.DEBUG ];'
1055 'then realpath %(dir)s/latest.DEBUG;'
1056 'elif [ -f %(dir)s/latest ];'
1057 'then realpath %(dir)s/latest;'
1058 'else exit %(code)d;'
1059 'fi' % {'dir': self.remote_log_dir,
1060 'code': self.NO_SYMLINKS_CODE})
1061 res = self.run(cmd, stderr_tee=None, ignore_status=True)
1062 if res.exit_status != 0:
1063 if res.exit_status == self.NO_SYMLINKS_CODE:
1064 logging.warning('servod log latest symlinks not found. '
1065 'This is likely due to an error starting up '
1066 'servod. Ignoring..')
1067 else:
1068 logging.warning('Failed to find servod logs on servo host.')
1069 logging.warning(res.stderr.strip())
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -07001070 return None
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -08001071 fname = os.path.basename(res.stdout.strip())
1072 # From the fname, ought to extract the timestamp using the TS_EXTRACTOR
Ruben Rodriguez Buchillone9aa2b02020-03-04 12:14:28 -08001073 ts_match = self.TS_EXTRACTOR.match(fname)
1074 if not ts_match:
1075 logging.warning('Failed to extract timestamp from servod log file '
1076 '%r. Skipping. The servo host is using outdated '
1077 'servod logging and needs to be updated.', fname)
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -07001078 return None
1079 return ts_match.group(self.TS_GROUP)
1080
1081 def get_instance_logs(self, instance_ts, outdir, old=False):
1082 """Collect all logs with |instance_ts| and dump into a dir in |outdir|
1083
1084 This method first collects all logs on the servo_host side pertaining
1085 to this servod instance (port, instatiation). It glues them together
1086 into combined log.[level].txt files and extracts all available MCU
1087 console I/O from the logs into individual files e.g. servo_v4.txt
1088
1089 All the output can be found in a directory inside |outdir| that
1090 this generates based on |LOG_DIR|, the servod port, and the instance
1091 timestamp on the servo_host side.
1092
1093 @param instance_ts: log timestamp to grab logfiles for
1094 @param outdir: directory to create a subdirectory into to place the
1095 servod logs into.
1096 @param old: bool, whether to append |OLD_LOG_SUFFIX| to output dir
1097 """
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -08001098 # Create the local results log dir.
1099 log_dir = os.path.join(outdir, '%s_%s.%s' % (self.LOG_DIR,
1100 str(self.servo_port),
1101 instance_ts))
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -07001102 if old:
Garry Wang22f2e842020-09-09 20:19:19 -07001103 log_dir = '%s.%s' % (log_dir, self.OLD_LOG_SUFFIX)
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -07001104 logging.info('Saving servod logs to %r.', log_dir)
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -08001105 os.mkdir(log_dir)
1106 # Now, get all files with that timestamp.
1107 cmd = 'find %s -maxdepth 1 -name "log.%s*"' % (self.remote_log_dir,
1108 instance_ts)
1109 res = self.run(cmd, stderr_tee=None, ignore_status=True)
1110 files = res.stdout.strip().split()
1111 try:
1112 self.get_file(files, log_dir, try_rsync=False)
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -07001113 if not os.listdir(log_dir):
1114 logging.info('No servod logs retrieved. Ignoring, and removing '
1115 '%r again.', log_dir)
1116 os.rmdir(log_dir)
1117 return
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -08001118 except error.AutoservRunError as e:
1119 result = e.result_obj
1120 if result.exit_status != 0:
1121 stderr = result.stderr.strip()
1122 logging.warning("Couldn't retrieve servod logs. Ignoring: %s",
1123 stderr or '\n%s' % result)
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -07001124 # Remove the log_dir as nothing was added to it.
1125 os.rmdir(log_dir)
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -08001126 return
1127 local_files = [os.path.join(log_dir, f) for f in os.listdir(log_dir)]
1128 # TODO(crrev.com/c/1793030): remove no-level case once CL is pushed
1129 for level_name in ('DEBUG', 'INFO', 'WARNING', ''):
1130 # Create the joint files for each loglevel. i.e log.DEBUG
1131 joint_file = self.JOINT_LOG_PREFIX
1132 if level_name:
1133 joint_file = '%s.%s' % (self.JOINT_LOG_PREFIX, level_name)
1134 # This helps with some online tools to avoid complaints about an
1135 # unknown filetype.
1136 joint_file = joint_file + '.txt'
1137 joint_path = os.path.join(log_dir, joint_file)
1138 files = [f for f in local_files if level_name in f]
1139 if not files:
1140 # TODO(crrev.com/c/1793030): remove no-level case once CL
1141 # is pushed
1142 continue
1143 # Extract compressed logs if any.
1144 compressed, extracted = self._extract_compressed_logs(log_dir,
1145 files)
1146 files = list(set(files) - set(compressed))
1147 files.extend(extracted)
1148 # Need to sort. As they all share the same timestamp, and
1149 # loglevel, the index itself is sufficient. The highest index
1150 # is the oldest file, therefore we need a descending sort.
1151 def sortkey(f, level=level_name):
1152 """Custom sortkey to sort based on rotation number int."""
1153 if f.endswith(level_name): return 0
1154 return int(f.split('.')[-1])
1155
1156 files.sort(reverse=True, key=sortkey)
1157 # Just rename the first file rather than building from scratch.
1158 os.rename(files[0], joint_path)
1159 with open(joint_path, 'a') as joint_f:
1160 for logfile in files[1:]:
1161 # Transfer the file to the joint file line by line.
1162 with open(logfile, 'r') as log_f:
1163 for line in log_f:
1164 joint_f.write(line)
1165 # File has been written over. Delete safely.
1166 os.remove(logfile)
1167 # Need to remove all files form |local_files| so we don't
1168 # analyze them again.
1169 local_files = list(set(local_files) - set(files) - set(compressed))
1170 # Lastly, extract MCU logs from the joint logs.
1171 self._extract_mcu_logs(log_dir)
1172
Garry Wang79e9af62019-06-12 15:19:19 -07001173 def _lock(self):
1174 """lock servohost by touching a file.
1175 """
1176 logging.debug('Locking servohost %s by touching %s file',
1177 self.hostname, self._lock_file)
1178 self.run('touch %s' % self._lock_file, ignore_status=True)
Garry Wang7c00b0f2019-06-25 17:28:17 -07001179 self._is_locked = True
Garry Wang79e9af62019-06-12 15:19:19 -07001180
Garry Wang79e9af62019-06-12 15:19:19 -07001181 def _unlock(self):
1182 """Unlock servohost by removing the lock file.
1183 """
1184 logging.debug('Unlocking servohost by removing %s file',
1185 self._lock_file)
1186 self.run('rm %s' % self._lock_file, ignore_status=True)
Garry Wang7c00b0f2019-06-25 17:28:17 -07001187 self._is_locked = False
Garry Wang79e9af62019-06-12 15:19:19 -07001188
Congbin Guoa1f9cba2018-07-03 11:36:59 -07001189 def close(self):
Congbin Guofc3b8962019-03-22 17:38:46 -07001190 """Close the associated servo and the host object."""
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -07001191 # NOTE: throughout this method there are multiple attempts to catch
1192 # all errors. This is WAI as log grabbing should not fail tests.
1193 # However, the goal is to catch and handle/process all errors, thus
1194 # we print the traceback and ask for a bug.
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -08001195 if self._closed:
1196 logging.debug('ServoHost is already closed.')
1197 return
Garry Wang22f2e842020-09-09 20:19:19 -07001198
1199 # Only attempt ssh related actions if servohost is sshable. We call
1200 # check_cached_up_status() first because it's lightweighted and return
1201 # much faster in the case servohost is down, however, we still want
1202 # to call is_up() later since check_cached_up_status() is ping based check
1203 # and not guarantee the servohost is sshable.
1204 servo_host_ready = self.check_cached_up_status() and self.is_up()
1205
1206 if servo_host_ready:
1207 instance_ts = self.get_instance_logs_ts()
1208 else:
1209 logging.info('Servohost is down, will skip servod log collecting.')
1210 instance_ts = None
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -07001211 # TODO(crbug.com/1011516): once enabled, remove the check against
1212 # localhost and instead check against log-rotiation enablement.
1213 logs_available = (instance_ts is not None and
1214 self.job and
1215 not self.is_localhost())
1216 if logs_available:
1217 # Probe whether there was a servod restart, and grab those old
1218 # logs as well.
1219 try:
1220 self.probe_servod_restart(instance_ts, self.job.resultdir)
1221 except (error.AutoservRunError, error.TestFail) as e:
1222 logging.info('Failed to grab servo logs due to: %s. '
1223 'This error is forgiven.', str(e))
1224 except Exception as e:
1225 logging.error('Unexpected error probing for old logs. %s. '
1226 'Forgiven. Please file a bug and fix or catch '
1227 'in log probing function', str(e),
1228 exc_info=True)
Congbin Guoa1f9cba2018-07-03 11:36:59 -07001229 if self._servo:
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -08001230 outdir = None if not self.job else self.job.resultdir
Congbin Guo2e5e2a22018-07-27 10:32:48 -07001231 # In some cases when we run as lab-tools, the job object is None.
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -08001232 self._servo.close(outdir)
1233
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -07001234 if logs_available:
1235 # Grab current (not old like above) logs after the servo instance
1236 # was closed out.
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -08001237 try:
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -07001238 self.get_instance_logs(instance_ts, self.job.resultdir)
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -08001239 except error.AutoservRunError as e:
1240 logging.info('Failed to grab servo logs due to: %s. '
1241 'This error is forgiven.', str(e))
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -07001242 except Exception as e:
1243 logging.error('Unexpected error grabbing servod logs. %s. '
1244 'Forgiven. Please file a bug and fix or catch '
1245 'in log grabbing function', str(e), exc_info=True)
Congbin Guoa1f9cba2018-07-03 11:36:59 -07001246
Garry Wang22f2e842020-09-09 20:19:19 -07001247 if self._is_locked and servo_host_ready:
Garry Wang7c00b0f2019-06-25 17:28:17 -07001248 # Remove the lock if the servohost has been locked.
Garry Wang79e9af62019-06-12 15:19:19 -07001249 try:
1250 self._unlock()
1251 except error.AutoservSSHTimeout:
1252 logging.error('Unlock servohost failed due to ssh timeout.'
1253 ' It may caused by servohost went down during'
1254 ' the task.')
Garry Wangc1288cf2019-12-17 14:58:00 -08001255 # We want always stop servod after task to minimum the impact of bad
1256 # servod process interfere other servods.(see crbug.com/1028665)
Garry Wang22f2e842020-09-09 20:19:19 -07001257 if servo_host_ready:
1258 try:
1259 self.stop_servod()
1260 except error.AutoservRunError as e:
1261 logging.info(
1262 "Failed to stop servod due to:\n%s\n"
1263 "This error is forgiven.", str(e))
Garry Wangc1288cf2019-12-17 14:58:00 -08001264
Congbin Guoa1f9cba2018-07-03 11:36:59 -07001265 super(ServoHost, self).close()
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -08001266 # Mark closed.
1267 self._closed = True
Congbin Guoa1f9cba2018-07-03 11:36:59 -07001268
Otabek Kasimovcc9738e2020-02-14 16:17:15 -08001269 def get_servo_state(self):
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001270 return self._servo_state
Otabek Kasimovcc9738e2020-02-14 16:17:15 -08001271
Otabek Kasimovc6f30412020-06-30 20:08:12 -07001272 def _get_host_metrics_data(self):
1273 return {'port': self.servo_port,
Otabek Kasimov0ea47362020-07-11 20:55:09 -07001274 'host': self.get_dut_hostname() or self.hostname,
Otabek Kasimovc6f30412020-06-30 20:08:12 -07001275 'board': self.servo_board or ''}
1276
1277 def _is_servo_device_connected(self, servo_type, serial):
1278 """Check if device is connected to the labstation.
1279
1280 Works for all servo devices connected to the labstation.
1281 For servo_v3 please use 'self._is_servo_board_present_on_servo_v3'
1282
1283 @param servo_type: The type of servo device. Expecting value can be
1284 servo_v4 or servo_micro.
1285 @param serial: The serial number of the device to detect it.
1286 """
1287 vid_pid = self.SERVO_VID_PID.get(servo_type)
1288 if not vid_pid or not serial:
1289 # device cannot detected without VID/PID or serial number
1290 return False
1291 logging.debug('Started to detect %s', servo_type)
1292 try:
1293 cmd = 'lsusb -v -d %s |grep iSerial |grep %s' % (vid_pid, serial)
1294 result = self.run(cmd, ignore_status=True, timeout=30)
1295 if result.exit_status == 0 and result.stdout.strip():
1296 logging.debug('The %s is plugged in to the host.', servo_type)
1297 return True
1298 logging.debug('%s device is not detected; %s', servo_type, result)
1299 return False
1300 except Exception as e:
1301 # can be triggered by timeout issue due running the script
1302 metrics.Counter(
1303 'chromeos/autotest/repair/servo_detection/timeout'
1304 ).increment(fields=self._get_host_metrics_data())
1305 logging.error('%s device is not detected; %s', servo_type, str(e))
1306 return None
1307
1308 def _is_servo_board_present_on_servo_v3(self):
1309 """Check if servo board is detected on servo_v3"""
1310 vid_pids = self.SERVO_VID_PID['servo_v3']
1311 if not vid_pids or len(vid_pids) == 0:
1312 # device cannot detected without VID/PID
1313 return False
1314 logging.debug('Started to detect servo board on servo_v3')
1315 not_detected = 'The servo board is not detected on servo_v3'
1316 try:
1317 cmd = 'lsusb | grep "%s"' % "\|".join(vid_pids)
1318 result = self.run(cmd, ignore_status=True, timeout=30)
1319 if result.exit_status == 0 and result.stdout.strip():
1320 logging.debug('The servo board is detected on servo_v3')
1321 return True
1322 logging.debug('%s; %s', not_detected, result)
1323 return False
1324 except Exception as e:
1325 # can be triggered by timeout issue due running the script
1326 metrics.Counter(
1327 'chromeos/autotest/repair/servo_detection/timeout'
1328 ).increment(fields=self._get_host_metrics_data())
1329 logging.error('%s; %s', not_detected, str(e))
1330 return None
1331
Otabek Kasimov120b6fa2020-07-03 00:15:27 -07001332 def _is_main_device_not_detected_on_servo_v4(self):
1333 """Check if servod cannot find main device on servo.
1334
1335 The check based on reading servod logs for servo_v4.
1336 """
1337 if not self._initial_instance_ts:
1338 # latest log not found
1339 return False
1340 logging.debug('latest log for servod created at %s',
1341 self._initial_instance_ts)
1342 try:
1343 log_created = calendar.timegm(time.strptime(
1344 self._initial_instance_ts,
1345 "%Y-%m-%d--%H-%M-%S.%f"))
1346 except ValueError as e:
1347 logging.debug('Cannot read time from log file name: %s',
1348 self._initial_instance_ts)
1349 return False
1350 min_time_created = calendar.timegm(time.gmtime())
1351 if min_time_created > log_created + 3600:
1352 # the log file is old we cannot use it
1353 logging.debug('log file was created more than hour ago, too old')
1354 return False
1355 logging.debug('latest log was created not longer then 1 hour ago')
1356
1357 # check if servod can detect main device by servo_v4
1358 message = 'ERROR - No servo micro or CCD detected for board'
1359 cmd = ('cat /var/log/servod_%s/log.%s.INFO |grep "%s"'
1360 % (self.servo_port, self._initial_instance_ts, message))
1361 result = self.run(cmd, ignore_status=True)
1362 if result.stdout.strip():
1363 logging.info('Servod cannot detect main device on the servo; '
1364 'Can be caused by bad hardware of servo or '
1365 'issue on the DUT side.')
1366 return True
1367 logging.debug('The main device is detected')
1368 return False
1369
Garry Wangb5cee3e2020-09-16 14:58:13 -07001370 def _require_cr50_servod_config(self):
1371 """Check whether we need start servod with CONFIG=cr50.xml"""
1372 dut_host_info = self.get_dut_host_info()
1373 if not dut_host_info:
1374 return False
1375 for pool in dut_host_info.pools:
1376 if pool.startswith(servo_constants.CR50_CONFIG_POOL_PREFIX):
1377 return True
1378 return False
1379
Otabek Kasimov15963492020-06-23 21:10:51 -07001380 def get_verify_state(self, tag):
1381 """Return the state of servo verifier.
1382
1383 @returns: bool or None
1384 """
1385 return self._repair_strategy.verifier_is_good(tag)
1386
1387 def determine_servo_state(self):
1388 """Determine servo state based on the failed verifier.
1389
1390 @returns: servo state value
1391 The state detecting based on first fail verifier or collecting of
1392 them.
1393 """
1394 ssh = self.get_verify_state('servo_ssh')
1395 disk_space = self.get_verify_state('disk_space')
1396 start_servod = self.get_verify_state('servod_job')
1397 create_servo = self.get_verify_state('servod_connection')
1398 init_servo = self.get_verify_state('servod_control')
Otabek Kasimova7eb4dc2020-09-16 10:25:17 -07001399 dut_connected = self.get_verify_state('dut_connected')
Otabek Kasimov15963492020-06-23 21:10:51 -07001400 pwr_button = self.get_verify_state('pwr_button')
1401 lid_open = self.get_verify_state('lid_open')
1402 ec_board = self.get_verify_state('ec_board')
1403 ccd_testlab = self.get_verify_state('ccd_testlab')
1404
1405 if not ssh:
1406 return servo_constants.SERVO_STATE_NO_SSH
1407
Otabek Kasimov066bdb82020-08-12 15:57:44 -07001408 if (start_servod == self.VERIFY_FAILED
1409 or create_servo == self.VERIFY_FAILED):
1410 # sometimes servo can start with out present servo
Otabek Kasimovc6f30412020-06-30 20:08:12 -07001411 if self.is_labstation():
1412 if not self.servo_serial:
1413 return servo_constants.SERVO_STATE_WRONG_CONFIG
1414 if self._is_servo_device_connected(
1415 'servo_v4',
1416 self.servo_serial) == False:
1417 return servo_constants.SERVO_STATE_NOT_CONNECTED
1418 elif self._is_servo_board_present_on_servo_v3() == False:
1419 return servo_constants.SERVO_STATE_NOT_CONNECTED
Otabek Kasimov066bdb82020-08-12 15:57:44 -07001420
Otabek Kasimova7eb4dc2020-09-16 10:25:17 -07001421 if dut_connected == self.VERIFY_FAILED:
1422 if pwr_button == self.VERIFY_SUCCESS:
1423 # unexpected case
1424 metrics.Counter(
1425 'chromeos/autotest/repair/servo_unexpected/pwr_button'
1426 ).increment(fields=self._get_host_metrics_data())
1427 return servo_constants.SERVO_STATE_DUT_NOT_CONNECTED
1428
Otabek Kasimov066bdb82020-08-12 15:57:44 -07001429 if start_servod == self.VERIFY_FAILED:
Otabek Kasimovc6f30412020-06-30 20:08:12 -07001430 return servo_constants.SERVO_STATE_SERVOD_ISSUE
1431
Otabek Kasimov120b6fa2020-07-03 00:15:27 -07001432 if create_servo == self.VERIFY_FAILED:
1433 if (self.is_labstation()
1434 and self._is_main_device_not_detected_on_servo_v4()):
1435 servo_type = None
1436 if self.get_dut_host_info():
1437 servo_type = self.get_dut_host_info().get_label_value(
1438 servo_constants.SERVO_TYPE_LABEL_PREFIX)
1439 if servo_type and 'servo_micro' in servo_type:
1440 serial = self.get_servo_micro_serial_number()
1441 logging.debug('servo_micro serial: %s', serial)
1442 if self._is_servo_device_detected('servo_micro',
1443 serial):
1444 return servo_constants.SERVO_STATE_BAD_RIBBON_CABLE
1445 # Device can be not detected because of DUT
1446 # TODO (otabek) update after b/159755652 and b/159754985
1447 metrics.Counter(
1448 'chromeos/autotest/repair/servo_state/needs_replacement'
1449 ).increment(fields=self._get_host_metrics_data())
1450 elif not self.is_labstation():
1451 # Here need logic to check if flex cable is connected
1452 pass
1453
Otabek Kasimov15963492020-06-23 21:10:51 -07001454 # one of the reason why servo can not initialized
1455 if ccd_testlab == self.VERIFY_FAILED:
1456 return servo_constants.SERVO_STATE_CCD_TESTLAB_ISSUE
1457
Otabek Kasimov120b6fa2020-07-03 00:15:27 -07001458 if (create_servo == self.VERIFY_FAILED
1459 or init_servo == self.VERIFY_FAILED):
Otabek Kasimov15963492020-06-23 21:10:51 -07001460 return servo_constants.SERVO_STATE_SERVOD_ISSUE
1461
Otabek Kasimov015c15c2020-08-20 00:40:42 -07001462 if ec_board == self.VERIFY_FAILED:
1463 return servo_constants.SERVO_STATE_EC_BROKEN
Otabek Kasimov15963492020-06-23 21:10:51 -07001464 if pwr_button == self.VERIFY_FAILED:
1465 return servo_constants.SERVO_STATE_BAD_RIBBON_CABLE
1466 if lid_open == self.VERIFY_FAILED:
1467 return servo_constants.SERVO_STATE_LID_OPEN_FAILED
Otabek Kasimov15963492020-06-23 21:10:51 -07001468
Otabek Kasimov15963492020-06-23 21:10:51 -07001469 metrics.Counter(
1470 'chromeos/autotest/repair/unknown_servo_state'
Otabek Kasimovc6f30412020-06-30 20:08:12 -07001471 ).increment(fields=self._get_host_metrics_data())
Otabek Kasimov15963492020-06-23 21:10:51 -07001472 logging.info('We do not have special state for this failure yet :)')
1473 return servo_constants.SERVO_STATE_BROKEN
1474
Otabek Kasimovcc9738e2020-02-14 16:17:15 -08001475
Richard Barnetteea3e4602016-06-10 12:36:41 -07001476def make_servo_hostname(dut_hostname):
1477 """Given a DUT's hostname, return the hostname of its servo.
1478
1479 @param dut_hostname: hostname of a DUT.
1480
1481 @return hostname of the DUT's servo.
1482
1483 """
1484 host_parts = dut_hostname.split('.')
1485 host_parts[0] = host_parts[0] + '-servo'
1486 return '.'.join(host_parts)
1487
1488
1489def servo_host_is_up(servo_hostname):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -07001490 """Given a servo host name, return if it's up or not.
Richard Barnetteea3e4602016-06-10 12:36:41 -07001491
1492 @param servo_hostname: hostname of the servo host.
1493
1494 @return True if it's up, False otherwise
1495 """
1496 # Technically, this duplicates the SSH ping done early in the servo
1497 # proxy initialization code. However, this ping ends in a couple
1498 # seconds when if fails, rather than the 60 seconds it takes to decide
1499 # that an SSH ping has timed out. Specifically, that timeout happens
1500 # when our servo DNS name resolves, but there is no host at that IP.
1501 logging.info('Pinging servo host at %s', servo_hostname)
1502 ping_config = ping_runner.PingConfig(
1503 servo_hostname, count=3,
1504 ignore_result=True, ignore_status=True)
1505 return ping_runner.PingRunner().ping(ping_config).received > 0
1506
1507
Richard Barnettee519dcd2016-08-15 17:37:17 -07001508def _map_afe_board_to_servo_board(afe_board):
1509 """Map a board we get from the AFE to a servo appropriate value.
1510
1511 Many boards are identical to other boards for servo's purposes.
1512 This function makes that mapping.
1513
1514 @param afe_board string board name received from AFE.
1515 @return board we expect servo to have.
1516
1517 """
1518 KNOWN_SUFFIXES = ['-freon', '_freon', '_moblab', '-cheets']
1519 BOARD_MAP = {'gizmo': 'panther'}
1520 mapped_board = afe_board
1521 if afe_board in BOARD_MAP:
1522 mapped_board = BOARD_MAP[afe_board]
1523 else:
1524 for suffix in KNOWN_SUFFIXES:
1525 if afe_board.endswith(suffix):
1526 mapped_board = afe_board[0:-len(suffix)]
1527 break
1528 if mapped_board != afe_board:
1529 logging.info('Mapping AFE board=%s to %s', afe_board, mapped_board)
1530 return mapped_board
1531
1532
Prathmesh Prabhub4810232018-09-07 13:24:08 -07001533def get_servo_args_for_host(dut_host):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -07001534 """Return servo data associated with a given DUT.
Richard Barnetteea3e4602016-06-10 12:36:41 -07001535
Richard Barnetteea3e4602016-06-10 12:36:41 -07001536 @param dut_host Instance of `Host` on which to find the servo
1537 attributes.
Prathmesh Prabhuf605dd32018-08-28 17:09:04 -07001538 @return `servo_args` dict with host and an optional port.
Richard Barnetteea3e4602016-06-10 12:36:41 -07001539 """
Prathmesh Prabhucba44292018-08-28 17:44:45 -07001540 info = dut_host.host_info_store.get()
Derek Beckettf73baca2020-08-19 15:08:47 -07001541 servo_args = {k: v for k, v in six.iteritems(info.attributes)
Garry Wang11b5e872020-03-11 15:14:08 -07001542 if k in servo_constants.SERVO_ATTR_KEYS}
Richard Barnetteea3e4602016-06-10 12:36:41 -07001543
Garry Wang11b5e872020-03-11 15:14:08 -07001544 if servo_constants.SERVO_PORT_ATTR in servo_args:
Prathmesh Prabhucba44292018-08-28 17:44:45 -07001545 try:
Garry Wang11b5e872020-03-11 15:14:08 -07001546 servo_args[servo_constants.SERVO_PORT_ATTR] = int(
1547 servo_args[servo_constants.SERVO_PORT_ATTR])
Prathmesh Prabhucba44292018-08-28 17:44:45 -07001548 except ValueError:
1549 logging.error('servo port is not an int: %s',
Garry Wang11b5e872020-03-11 15:14:08 -07001550 servo_args[servo_constants.SERVO_PORT_ATTR])
Prathmesh Prabhucba44292018-08-28 17:44:45 -07001551 # Reset servo_args because we don't want to use an invalid port.
Garry Wang11b5e872020-03-11 15:14:08 -07001552 servo_args.pop(servo_constants.SERVO_HOST_ATTR, None)
Prathmesh Prabhucba44292018-08-28 17:44:45 -07001553
1554 if info.board:
Garry Wang11b5e872020-03-11 15:14:08 -07001555 servo_board = _map_afe_board_to_servo_board(info.board)
1556 servo_args[servo_constants.SERVO_BOARD_ATTR] = servo_board
Nick Sanders2f3c9852018-10-24 12:10:24 -07001557 if info.model:
Garry Wang11b5e872020-03-11 15:14:08 -07001558 servo_args[servo_constants.SERVO_MODEL_ATTR] = info.model
1559 return servo_args if servo_constants.SERVO_HOST_ATTR in servo_args else None
Richard Barnetteea3e4602016-06-10 12:36:41 -07001560
1561
Prathmesh Prabhuefb1b482018-08-28 17:15:05 -07001562def _tweak_args_for_ssp_moblab(servo_args):
Garry Wang11b5e872020-03-11 15:14:08 -07001563 if (servo_args[servo_constants.SERVO_HOST_ATTR]
1564 in ['localhost', '127.0.0.1']):
1565 servo_args[servo_constants.SERVO_HOST_ATTR] = _CONFIG.get_config_value(
Prathmesh Prabhuefb1b482018-08-28 17:15:05 -07001566 'SSP', 'host_container_ip', type=str, default=None)
1567
1568
Dan Shi023aae32016-05-25 11:13:01 -07001569def create_servo_host(dut, servo_args, try_lab_servo=False,
Gregory Nisbetde13e2a2019-12-09 22:44:00 -08001570 try_servo_repair=False, dut_host_info=None):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -07001571 """Create a ServoHost object for a given DUT, if appropriate.
Dan Shi4d478522014-02-14 13:46:32 -08001572
Richard Barnette9a26ad62016-06-10 12:03:08 -07001573 This function attempts to create and verify or repair a `ServoHost`
1574 object for a servo connected to the given `dut`, subject to various
1575 constraints imposed by the parameters:
1576 * When the `servo_args` parameter is not `None`, a servo
1577 host must be created, and must be checked with `repair()`.
1578 * Otherwise, if a servo exists in the lab and `try_lab_servo` is
1579 true:
1580 * If `try_servo_repair` is true, then create a servo host and
1581 check it with `repair()`.
1582 * Otherwise, if the servo responds to `ping` then create a
1583 servo host and check it with `verify()`.
Fang Denge545abb2014-12-30 18:43:47 -08001584
Richard Barnette9a26ad62016-06-10 12:03:08 -07001585 In cases where `servo_args` was not `None`, repair failure
1586 exceptions are passed back to the caller; otherwise, exceptions
Richard Barnette07c2e1d2016-10-26 14:24:28 -07001587 are logged and then discarded. Note that this only happens in cases
1588 where we're called from a test (not special task) control file that
1589 has an explicit dependency on servo. In that case, we require that
1590 repair not write to `status.log`, so as to avoid polluting test
1591 results.
1592
1593 TODO(jrbarnette): The special handling for servo in test control
1594 files is a thorn in my flesh; I dearly hope to see it cut out before
1595 my retirement.
Richard Barnette9a26ad62016-06-10 12:03:08 -07001596
1597 Parameters for a servo host consist of a host name, port number, and
1598 DUT board, and are determined from one of these sources, in order of
1599 priority:
Richard Barnetteea3e4602016-06-10 12:36:41 -07001600 * Servo attributes from the `dut` parameter take precedence over
1601 all other sources of information.
1602 * If a DNS entry for the servo based on the DUT hostname exists in
1603 the CrOS lab network, that hostname is used with the default
Richard Barnette9a26ad62016-06-10 12:03:08 -07001604 port and the DUT's board.
Richard Barnetteea3e4602016-06-10 12:36:41 -07001605 * If no other options are found, the parameters will be taken
Richard Barnette9a26ad62016-06-10 12:03:08 -07001606 from the `servo_args` dict passed in from the caller.
Richard Barnetteea3e4602016-06-10 12:36:41 -07001607
1608 @param dut An instance of `Host` from which to take
1609 servo parameters (if available).
1610 @param servo_args A dictionary with servo parameters to use if
1611 they can't be found from `dut`. If this
1612 argument is supplied, unrepaired exceptions
1613 from `verify()` will be passed back to the
1614 caller.
1615 @param try_lab_servo If not true, servo host creation will be
1616 skipped unless otherwise required by the
1617 caller.
Richard Barnette9a26ad62016-06-10 12:03:08 -07001618 @param try_servo_repair If true, check a servo host with
1619 `repair()` instead of `verify()`.
Otabek Kasimov8475cce2020-07-14 12:11:31 -07001620 @param dut_host_info: A HostInfo object of the DUT that connected
1621 to this servo.
Dan Shi4d478522014-02-14 13:46:32 -08001622
1623 @returns: A ServoHost object or None. See comments above.
1624
1625 """
Richard Barnette07c2e1d2016-10-26 14:24:28 -07001626 servo_dependency = servo_args is not None
Richard Barnette07c2e1d2016-10-26 14:24:28 -07001627 if dut is not None and (try_lab_servo or servo_dependency):
Prathmesh Prabhub4810232018-09-07 13:24:08 -07001628 servo_args_override = get_servo_args_for_host(dut)
Richard Barnetteea3e4602016-06-10 12:36:41 -07001629 if servo_args_override is not None:
Prathmesh Prabhuefb1b482018-08-28 17:15:05 -07001630 if utils.in_moblab_ssp():
1631 _tweak_args_for_ssp_moblab(servo_args_override)
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -07001632 logging.debug(
1633 'Overriding provided servo_args (%s) with arguments'
1634 ' determined from the host (%s)',
1635 servo_args,
1636 servo_args_override,
1637 )
Richard Barnetteea3e4602016-06-10 12:36:41 -07001638 servo_args = servo_args_override
Prathmesh Prabhucba44292018-08-28 17:44:45 -07001639
Richard Barnetteea3e4602016-06-10 12:36:41 -07001640 if servo_args is None:
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -07001641 logging.debug('No servo_args provided, and failed to find overrides.')
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001642 if try_lab_servo or servo_dependency:
Otabek Kasimov646812c2020-06-23 20:01:36 -07001643 return None, servo_constants.SERVO_STATE_MISSING_CONFIG
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001644 else:
1645 # For regular test case which not required the servo
1646 return None, None
1647
Garry Wang11b5e872020-03-11 15:14:08 -07001648 servo_hostname = servo_args.get(servo_constants.SERVO_HOST_ATTR)
1649 servo_port = servo_args.get(servo_constants.SERVO_PORT_ATTR)
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001650 if not _is_servo_host_information_exist(servo_hostname, servo_port):
1651 logging.debug(
1652 'Servo connection info missed hostname: %s , port: %s',
1653 servo_hostname, servo_port)
Otabek Kasimov646812c2020-06-23 20:01:36 -07001654 return None, servo_constants.SERVO_STATE_MISSING_CONFIG
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001655 if not is_servo_host_information_valid(servo_hostname, servo_port):
1656 logging.debug(
1657 'Servo connection info is incorrect hostname: %s , port: %s',
1658 servo_hostname, servo_port)
Garry Wang11b5e872020-03-11 15:14:08 -07001659 return None, servo_constants.SERVO_STATE_WRONG_CONFIG
Richard Barnette07c2e1d2016-10-26 14:24:28 -07001660 if (not servo_dependency and not try_servo_repair and
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001661 not servo_host_is_up(servo_hostname)):
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -07001662 logging.debug('ServoHost is not up.')
Otabek Kasimov646812c2020-06-23 20:01:36 -07001663 return None, servo_constants.SERVO_STATE_NO_SSH
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -07001664
Garry Wangebc015b2019-06-06 17:45:06 -07001665 newhost = ServoHost(**servo_args)
Garry Wangffbd2162020-04-17 16:13:48 -07001666
Otabek Kasimove6df8102020-07-21 20:15:25 -07001667 # Reset or reboot servo device only during AdminRepair tasks.
1668 if try_servo_repair:
1669 if newhost._is_locked:
1670 # Reset servo if the servo is locked, as we check if the servohost
1671 # is up, if the servohost is labstation and if the servohost is in
1672 # lab inside the locking logic.
Otabek Kasimov545739c2020-08-20 00:24:21 -07001673 newhost.reset_usbc_pigtail_connection_on_need()
Otabek Kasimove6df8102020-07-21 20:15:25 -07001674 newhost.reset_servo()
1675 else:
Garry Wang358aad42020-08-02 20:56:04 -07001676 try:
1677 newhost.reboot_servo_v3_on_need()
Garry Wang1f0d5332020-08-10 19:32:32 -07001678 except Exception as e:
1679 logging.info('[Non-critical] Unexpected error while trying to'
1680 ' reboot servo_v3, skipping the reboot; %s', e)
Otabek Kasimove6df8102020-07-21 20:15:25 -07001681
Otabek Kasimov2b50cdb2020-07-06 19:16:06 -07001682 if dut:
1683 newhost.set_dut_hostname(dut.hostname)
Otabek Kasimov9e90ae12020-08-14 03:01:19 -07001684 if dut_host_info:
1685 newhost.set_dut_host_info(dut_host_info)
Garry Wangffbd2162020-04-17 16:13:48 -07001686
Otabek Kasimov8475cce2020-07-14 12:11:31 -07001687 if try_lab_servo or try_servo_repair:
1688 try:
1689 logging.info("Check and update servo firmware.")
1690 servo_updater.update_servo_firmware(
1691 newhost,
1692 force_update=False)
1693 except Exception as e:
1694 logging.error("Servo device update error: %s", e)
1695
Garry Wangcdd27b22020-01-13 14:59:11 -08001696 try:
1697 newhost.restart_servod(quick_startup=True)
1698 except error.AutoservSSHTimeout:
1699 logging.warning("Restart servod failed due ssh connection "
1700 "to servohost timed out. This error is forgiven"
1701 " here, we will retry in servo repair process.")
1702 except error.AutoservRunError as e:
1703 logging.warning("Restart servod failed due to:\n%s\n"
1704 "This error is forgiven here, we will retry"
1705 " in servo repair process.", str(e))
Garry Wangebc015b2019-06-06 17:45:06 -07001706
Richard Barnette9a26ad62016-06-10 12:03:08 -07001707 # Note that the logic of repair() includes everything done
1708 # by verify(). It's sufficient to call one or the other;
1709 # we don't need both.
Richard Barnette07c2e1d2016-10-26 14:24:28 -07001710 if servo_dependency:
1711 newhost.repair(silent=True)
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001712 return newhost, newhost.get_servo_state()
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -07001713
1714 if try_servo_repair:
1715 try:
1716 newhost.repair()
1717 except Exception:
1718 logging.exception('servo repair failed for %s', newhost.hostname)
Richard Barnette9a26ad62016-06-10 12:03:08 -07001719 else:
1720 try:
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -07001721 newhost.verify()
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -07001722 except Exception:
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -07001723 logging.exception('servo verify failed for %s', newhost.hostname)
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001724 return newhost, newhost.get_servo_state()
Otabek Kasimov7267a7a2020-03-04 11:18:45 -08001725
1726
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001727def _is_servo_host_information_exist(hostname, port):
Otabek Kasimov7267a7a2020-03-04 11:18:45 -08001728 if hostname is None or len(hostname.strip()) == 0:
1729 return False
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001730 if port is None:
Otabek Kasimov7267a7a2020-03-04 11:18:45 -08001731 return False
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001732 if not type(port) is int:
1733 try:
1734 int(port)
1735 except ValueError:
1736 return False
1737
Otabek Kasimov7267a7a2020-03-04 11:18:45 -08001738 return True
1739
1740
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001741def is_servo_host_information_valid(hostname, port):
Garry Wang9b8f2342020-04-17 16:34:09 -07001742 """Check if provided servo attributes are valid.
1743
1744 @param hostname Hostname of the servohost.
1745 @param port servo port number.
1746
1747 @returns: A bool value to indicate if provided servo attribute valid.
1748 """
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001749 if not _is_servo_host_information_exist(hostname, port):
Otabek Kasimov7267a7a2020-03-04 11:18:45 -08001750 return False
1751 # checking range and correct of the port
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001752 port_int = int(port)
Otabek Kasimov7267a7a2020-03-04 11:18:45 -08001753 if port_int < 1 or port_int > 65000:
1754 return False
1755 # we expecting host contain only latters, digits and '-' or '_'
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001756 if not re.match('[a-zA-Z0-9-_\.]*$', hostname) or len(hostname) < 5:
Otabek Kasimov7267a7a2020-03-04 11:18:45 -08001757 return False
1758 return True