blob: 3f459e10fcd911267d040b5e25360c5d66477fa1 [file] [log] [blame]
Fang Deng5d518f42013-08-02 14:04:32 -07001# Copyright (c) 2013 The Chromium OS Authors. All rights reserved.
2# Use of this source code is governed by a BSD-style license that can be
3# found in the LICENSE file.
4#
5# Expects to be run in an environment with sudo and no interactive password
6# prompt, such as within the Chromium OS development chroot.
7
8
9"""This file provides core logic for servo verify/repair process."""
10
11
Fang Deng5d518f42013-08-02 14:04:32 -070012import logging
Raul E Rangel52ca2e82018-07-03 14:10:14 -060013import os
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -080014import re
15import tarfile
Garry Wangc1288cf2019-12-17 14:58:00 -080016import time
Gregory Nisbet265a52c2019-12-10 20:38:42 -080017import traceback
Dana Goyette4dc0adc2019-05-06 14:51:53 -070018import xmlrpclib
Fang Deng5d518f42013-08-02 14:04:32 -070019
20from autotest_lib.client.bin import utils
Garry Wang79e9af62019-06-12 15:19:19 -070021from autotest_lib.client.common_lib import error
Richard Barnette9a26ad62016-06-10 12:03:08 -070022from autotest_lib.client.common_lib import hosts
Garry Wang7b0e1b72020-03-25 19:08:59 -070023from autotest_lib.client.common_lib import lsbrelease_utils
Fang Deng5d518f42013-08-02 14:04:32 -070024from autotest_lib.client.common_lib.cros import retry
Christopher Wileycef1f902014-06-19 11:11:23 -070025from autotest_lib.client.common_lib.cros.network import ping_runner
Richard Barnette9a26ad62016-06-10 12:03:08 -070026from autotest_lib.server.cros.servo import servo
Richard Barnetted31580e2018-05-14 19:58:00 +000027from autotest_lib.server.hosts import servo_repair
Garry Wangebc015b2019-06-06 17:45:06 -070028from autotest_lib.server.hosts import base_servohost
Garry Wang11b5e872020-03-11 15:14:08 -070029from autotest_lib.server.hosts import servo_constants
30from autotest_lib.client.common_lib import global_config
Garry Wangd7367482020-02-27 13:52:40 -080031
Dan Shi3b2adf62015-09-02 17:46:54 -070032_CONFIG = global_config.global_config
Fang Deng5d518f42013-08-02 14:04:32 -070033
Otabek Kasimova7ba91a2020-03-09 08:31:01 -070034
Garry Wangebc015b2019-06-06 17:45:06 -070035class ServoHost(base_servohost.BaseServoHost):
36 """Host class for a servo host(e.g. beaglebone, labstation)
Dana Goyette0b6e6402019-10-04 11:09:24 -070037 that with a servo instance for a specific port.
38
39 @type _servo: servo.Servo | None
40 """
Fang Deng5d518f42013-08-02 14:04:32 -070041
Raul E Rangel52ca2e82018-07-03 14:10:14 -060042 DEFAULT_PORT = int(os.getenv('SERVOD_PORT', '9999'))
Richard Barnette9a26ad62016-06-10 12:03:08 -070043
Dan Shie5b3c512014-08-21 12:12:09 -070044 # Timeout for initializing servo signals.
Wai-Hong Tam37b6ed32017-09-19 15:52:39 -070045 INITIALIZE_SERVO_TIMEOUT_SECS = 60
Richard Barnette9a26ad62016-06-10 12:03:08 -070046
xixuan6cf6d2f2016-01-29 15:29:00 -080047 # Ready test function
48 SERVO_READY_METHOD = 'get_version'
Fang Deng5d518f42013-08-02 14:04:32 -070049
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -080050 # Directory prefix on the servo host where the servod logs are stored.
51 SERVOD_LOG_PREFIX = '/var/log/servod'
52
53 # Exit code to use when symlinks for servod logs are not found.
54 NO_SYMLINKS_CODE = 9
55
56 # Directory in the job's results directory to dump the logs into.
57 LOG_DIR = 'servod'
58
59 # Prefix for joint loglevel files in the logs.
60 JOINT_LOG_PREFIX = 'log'
61
62 # Regex group to extract timestamp from logfile name.
63 TS_GROUP = 'ts'
64
65 # This regex is used to extract the timestamp from servod logs.
66 # files always start with log.
67 TS_RE = (r'log.'
68 # The timestamp is of format %Y-%m-%d--%H-%M-%S.MS
69 r'(?P<%s>\d{4}(\-\d{2}){2}\-(-\d{2}){3}.\d{3})'
70 # The loglevel is optional depending on labstation version.
71 r'(.(INFO|DEBUG|WARNING))?' % TS_GROUP)
72 TS_EXTRACTOR = re.compile(TS_RE)
73
74 # Regex group to extract MCU name from logline in servod logs.
75 MCU_GROUP = 'mcu'
76
77 # Regex group to extract logline from MCU logline in servod logs.
78 LINE_GROUP = 'line'
79
80 # This regex is used to extract the mcu and the line content from an
81 # MCU logline in servod logs. e.g. EC or servo_v4 console logs.
82 # Here is an example log-line:
83 #
84 # 2020-01-23 13:15:12,223 - servo_v4 - EC3PO.Console - DEBUG -
85 # console.py:219:LogConsoleOutput - /dev/pts/9 - cc polarity: cc1
86 #
87 # Here is conceptually how they are formatted:
88 #
89 # <time> - <MCU> - EC3PO.Console - <LVL> - <file:line:func> - <pts> -
90 # <output>
91 #
92 # The log format starts with a timestamp
93 MCU_RE = (r'[\d\-]+ [\d:,]+ '
94 # The mcu that is logging this is next.
95 r'- (?P<%s>\w+) - '
96 # Next, we have more log outputs before the actual line.
97 # Information about the file line, logging function etc.
98 # Anchor on EC3PO Console, LogConsoleOutput and dev/pts.
99 # NOTE: if the log format changes, this regex needs to be
100 # adjusted.
101 r'EC3PO\.Console[\s\-\w\d:.]+LogConsoleOutput - /dev/pts/\d+ - '
102 # Lastly, we get the MCU's console line.
103 r'(?P<%s>.+$)' % (MCU_GROUP, LINE_GROUP))
104 MCU_EXTRACTOR = re.compile(MCU_RE)
105
106 # Suffix to identify compressed logfiles.
107 COMPRESSION_SUFFIX = '.tbz2'
108
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -0700109 # A suffix to mark servod log directories that came from instance that
110 # ran during this servo_host, but are not the last one running e.g. when
111 # an instance (on purpose, or due to a bug) restarted in the middle of the
112 # run.
113 OLD_LOG_SUFFIX = 'old'
114
Otabek Kasimovcc9738e2020-02-14 16:17:15 -0800115 def _init_attributes(self):
116 self._servo_state = None
117 self.servo_port = None
118 self.servo_board = None
119 self.servo_model = None
120 self.servo_serial = None
121 self._servo = None
122 self._servod_server_proxy = None
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -0700123 self._initial_instance_ts = None
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800124 # Flag to make sure that multiple calls to close do not result in the
125 # logic executing multiple times.
126 self._closed = False
Fang Deng5d518f42013-08-02 14:04:32 -0700127
Richard Barnette17bfc6c2016-08-04 18:41:43 -0700128 def _initialize(self, servo_host='localhost',
Richard Barnettee519dcd2016-08-15 17:37:17 -0700129 servo_port=DEFAULT_PORT, servo_board=None,
Nick Sanders2f3c9852018-10-24 12:10:24 -0700130 servo_model=None, servo_serial=None, is_in_lab=None,
131 *args, **dargs):
Fang Deng5d518f42013-08-02 14:04:32 -0700132 """Initialize a ServoHost instance.
133
134 A ServoHost instance represents a host that controls a servo.
135
136 @param servo_host: Name of the host where the servod process
137 is running.
Raul E Rangel52ca2e82018-07-03 14:10:14 -0600138 @param servo_port: Port the servod process is listening on. Defaults
139 to the SERVOD_PORT environment variable if set,
140 otherwise 9999.
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700141 @param servo_board: Board that the servo is connected to.
Nick Sanders2f3c9852018-10-24 12:10:24 -0700142 @param servo_model: Model that the servo is connected to.
Dan Shi4d478522014-02-14 13:46:32 -0800143 @param is_in_lab: True if the servo host is in Cros Lab. Default is set
144 to None, for which utils.host_is_in_lab_zone will be
145 called to check if the servo host is in Cros lab.
Fang Deng5d518f42013-08-02 14:04:32 -0700146
147 """
148 super(ServoHost, self)._initialize(hostname=servo_host,
Garry Wangebc015b2019-06-06 17:45:06 -0700149 is_in_lab=is_in_lab, *args, **dargs)
Otabek Kasimovcc9738e2020-02-14 16:17:15 -0800150 self._init_attributes()
Richard Barnette42f4db92018-08-23 15:05:15 -0700151 self.servo_port = int(servo_port)
Richard Barnettee519dcd2016-08-15 17:37:17 -0700152 self.servo_board = servo_board
Nick Sanders2f3c9852018-10-24 12:10:24 -0700153 self.servo_model = servo_model
Kevin Cheng643ce8a2016-09-15 15:42:12 -0700154 self.servo_serial = servo_serial
Wai-Hong Tam3a8a2552019-11-19 14:28:04 +0800155
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800156 # The location of the log files on the servo host for this instance.
157 self.remote_log_dir = '%s_%s' % (self.SERVOD_LOG_PREFIX,
158 self.servo_port)
Garry Wang79e9af62019-06-12 15:19:19 -0700159 # Path of the servo host lock file.
160 self._lock_file = (self.TEMP_FILE_DIR + str(self.servo_port)
161 + self.LOCK_FILE_POSTFIX)
162 # File path to declare a reboot request.
163 self._reboot_file = (self.TEMP_FILE_DIR + str(self.servo_port)
164 + self.REBOOT_FILE_POSTFIX)
165
166 # Lock the servo host if it's an in-lab labstation to prevent other
167 # task to reboot it until current task completes. We also wait and
168 # make sure the labstation is up here, in the case of the labstation is
169 # in the middle of reboot.
Garry Wang7c00b0f2019-06-25 17:28:17 -0700170 self._is_locked = False
Garry Wang42b4d862019-06-25 15:50:49 -0700171 if (self.wait_up(self.REBOOT_TIMEOUT) and self.is_in_lab()
172 and self.is_labstation()):
Garry Wang79e9af62019-06-12 15:19:19 -0700173 self._lock()
Garry Wangebc015b2019-06-06 17:45:06 -0700174
Richard Barnette9a26ad62016-06-10 12:03:08 -0700175 self._repair_strategy = (
176 servo_repair.create_servo_repair_strategy())
Richard Barnettee519dcd2016-08-15 17:37:17 -0700177
Dana Goyetteafa62fd2020-03-16 13:45:27 -0700178 def __str__(self):
179 return "<%s '%s:%s'>" % (
180 type(self).__name__, self.hostname, self.servo_port)
181
Richard Barnette9a26ad62016-06-10 12:03:08 -0700182 def connect_servo(self):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700183 """Establish a connection to the servod server on this host.
Richard Barnette9a26ad62016-06-10 12:03:08 -0700184
185 Initializes `self._servo` and then verifies that all network
186 connections are working. This will create an ssh tunnel if
187 it's required.
188
189 As a side effect of testing the connection, all signals on the
190 target servo are reset to default values, and the USB stick is
191 set to the neutral (off) position.
192 """
Kevin Cheng643ce8a2016-09-15 15:42:12 -0700193 servo_obj = servo.Servo(servo_host=self, servo_serial=self.servo_serial)
Kuang-che Wu05763f52019-08-30 16:48:21 +0800194 self._servo = servo_obj
Richard Barnette9a26ad62016-06-10 12:03:08 -0700195 timeout, _ = retry.timeout(
196 servo_obj.initialize_dut,
197 timeout_sec=self.INITIALIZE_SERVO_TIMEOUT_SECS)
198 if timeout:
199 raise hosts.AutoservVerifyError(
200 'Servo initialize timed out.')
Richard Barnette9a26ad62016-06-10 12:03:08 -0700201
202
203 def disconnect_servo(self):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700204 """Disconnect our servo if it exists.
Richard Barnette9a26ad62016-06-10 12:03:08 -0700205
206 If we've previously successfully connected to our servo,
207 disconnect any established ssh tunnel, and set `self._servo`
208 back to `None`.
209 """
210 if self._servo:
211 # N.B. This call is safe even without a tunnel:
212 # rpc_server_tracker.disconnect() silently ignores
213 # unknown ports.
214 self.rpc_server_tracker.disconnect(self.servo_port)
215 self._servo = None
Fang Deng5d518f42013-08-02 14:04:32 -0700216
Garry Wangc1288cf2019-12-17 14:58:00 -0800217
Wai-Hong Tam3a8a2552019-11-19 14:28:04 +0800218 def _create_servod_server_proxy(self):
219 """Create a proxy that can be used to communicate with servod server.
Fang Deng5d518f42013-08-02 14:04:32 -0700220
221 @returns: An xmlrpclib.ServerProxy that is connected to the servod
222 server on the host.
Fang Deng5d518f42013-08-02 14:04:32 -0700223 """
Garry Wang11b5e872020-03-11 15:14:08 -0700224 if (servo_constants.ENABLE_SSH_TUNNEL_FOR_SERVO
225 and not self.is_localhost()):
Richard Barnette9a26ad62016-06-10 12:03:08 -0700226 return self.rpc_server_tracker.xmlrpc_connect(
227 None, self.servo_port,
228 ready_test_name=self.SERVO_READY_METHOD,
Allen Li2b1a8992018-11-27 14:17:18 -0800229 timeout_seconds=60,
Dana Goyetteafa62fd2020-03-16 13:45:27 -0700230 request_timeout_seconds=3600,
231 server_desc=str(self))
Richard Barnette9a26ad62016-06-10 12:03:08 -0700232 else:
233 remote = 'http://%s:%s' % (self.hostname, self.servo_port)
234 return xmlrpclib.ServerProxy(remote)
Fang Deng5d518f42013-08-02 14:04:32 -0700235
236
Wai-Hong Tam3a8a2552019-11-19 14:28:04 +0800237 def get_servod_server_proxy(self):
238 """Return a cached proxy if exists; otherwise, create a new one.
239
240 @returns: An xmlrpclib.ServerProxy that is connected to the servod
241 server on the host.
242 """
243 # Single-threaded execution, no race
244 if self._servod_server_proxy is None:
245 self._servod_server_proxy = self._create_servod_server_proxy()
246 return self._servod_server_proxy
247
248
Richard Barnette1edbb162016-11-01 11:47:50 -0700249 def verify(self, silent=False):
250 """Update the servo host and verify it's in a good state.
251
252 @param silent If true, suppress logging in `status.log`.
253 """
Richard Barnetteabbdc252018-07-26 16:57:42 -0700254 message = 'Beginning verify for servo host %s port %s serial %s'
255 message %= (self.hostname, self.servo_port, self.servo_serial)
256 self.record('INFO', None, None, message)
Richard Barnette9a26ad62016-06-10 12:03:08 -0700257 try:
Richard Barnette1edbb162016-11-01 11:47:50 -0700258 self._repair_strategy.verify(self, silent)
Garry Wang11b5e872020-03-11 15:14:08 -0700259 self._servo_state = servo_constants.SERVO_STATE_WORKING
260 self.record('INFO', None, None,
261 'ServoHost verify set servo_state as WORKING')
Garry Wang63b8c382020-03-11 22:28:40 -0700262 except Exception as e:
Garry Wang11b5e872020-03-11 15:14:08 -0700263 self._servo_state = servo_constants.SERVO_STATE_BROKEN
264 self.record('INFO', None, None,
265 'ServoHost verify set servo_state as BROKEN')
Garry Wang63b8c382020-03-11 22:28:40 -0700266 if self._is_critical_error(e):
267 raise
Fang Deng5d518f42013-08-02 14:04:32 -0700268
269
Garry Wang7b0e1b72020-03-25 19:08:59 -0700270 def get_image_name_from_usbkey(self, usbkey_dev):
271 """Mount usb drive and check ChromeOS image name on it if there is
272 one. This method assumes the image_usbkey_direction is already set
273 to servo side.
274
275 @param: usbkey_dev usbkey dev path(e.g. /dev/sdb).
276
277 @returns: image_name on the usbkey, e.g. nami-release/R82.10138.0.0,
278 or empty string if no test image detected, or unexpected
279 error occurred.
280 @raises AutoservRepairError if mount usb drive failed with no
281 specific device error, which usually means the usbkey is
282 not existing or in bad shape.
283 """
284 usb_mount_path = '/media/servo_usb/%s' % self.servo_port
285 unmount_cmd = 'umount %s' % usb_mount_path
286 # ChromeOS root fs is in /dev/sdx3
287 mount_cmd = 'mount -o ro %s3 %s' % (usbkey_dev, usb_mount_path)
288 # Unmount if there is an existing stale mount.
289 self.run(unmount_cmd, ignore_status=True)
290 # Create if the mount point is not existing.
291 self.run('mkdir -p %s' % usb_mount_path)
292 try:
293 # Attempt to mount the usb drive.
294 mount_result = self.run(mount_cmd, ignore_status=True)
295 if mount_result.exit_status == 0:
296 release_content = self.run(
297 'cat %s/etc/lsb-release' % usb_mount_path,
298 ignore_status=True).stdout.strip()
299
300 if not re.search(r'RELEASE_TRACK=.*test', release_content):
301 logging.info('The image on usbkey is not a test image')
302 return ''
303
304 return lsbrelease_utils.get_chromeos_release_builder_path(
305 lsb_release_content=release_content)
306 elif (mount_result.exit_status == 32
307 and 'does not exist' in mount_result.stderr):
308 ## probe_host_usb_dev() sometimes return stale record.
309 raise hosts.AutoservRepairError('No usbkey detected on servo,'
310 ' the usbkey may be either'
311 ' missing or broken.',
312 'missing usbkey')
313 else:
314 logging.error('Unexpected error occurred on mount usb'
315 ' drive, skipping usbkey validation.')
316 return ''
317 finally:
318 logging.debug('Usbkey validation compeleted, unmounting the'
319 ' usb drive.')
320 self.run(unmount_cmd, ignore_status=True)
321
322
323 def validate_image_usbkey(self):
324 """This method validate if there is a usbkey on servo that accessible
325 to servohost. It will get the usb disk path, and then mount the usb,
326 if image_name is provided, this method will also check if the image
327 is already on the usb drive, so we can avoid unnecessary download and
328 flash to usb device.
329
330 @returns: image_name on the usbkey, e.g. nami-release/R82.10138.0.0,
331 or empty string if no test image detected, or unexpected
332 error occurred.
333 @raises: AutoservRepairError if the usbkey is not detected on servo.
334 """
335 logging.info('Validating image usbkey on servo.')
336 usbkey_dev = None
337 try:
338 usbkey_dev = self._servo.probe_host_usb_dev()
339 except Exception as e:
340 # We don't want any unexpected or transient servo communicating
341 # failure block usb repair, so capture all errors here.
342 logging.error(e, exc_info=True)
343 logging.error('Unexpected error occurred on get usbkey dev path,'
344 ' skipping usbkey validation.')
345 return ''
346
347 if not usbkey_dev:
348 raise hosts.AutoservRepairError('No usbkey detected on servo, the'
349 ' usbkey may be either missing or'
350 ' broken.', 'missing usbkey')
351 return self.get_image_name_from_usbkey(usbkey_dev)
352
353
Richard Barnette1edbb162016-11-01 11:47:50 -0700354 def repair(self, silent=False):
355 """Attempt to repair servo host.
356
357 @param silent If true, suppress logging in `status.log`.
358 """
Richard Barnetteabbdc252018-07-26 16:57:42 -0700359 message = 'Beginning repair for servo host %s port %s serial %s'
360 message %= (self.hostname, self.servo_port, self.servo_serial)
361 self.record('INFO', None, None, message)
Richard Barnette9a26ad62016-06-10 12:03:08 -0700362 try:
Richard Barnette1edbb162016-11-01 11:47:50 -0700363 self._repair_strategy.repair(self, silent)
Garry Wang11b5e872020-03-11 15:14:08 -0700364 self._servo_state = servo_constants.SERVO_STATE_WORKING
365 self.record('INFO', None, None,
366 'ServoHost repair set servo_state as WORKING')
Garry Wang464ff1e2019-07-18 17:20:34 -0700367 # If target is a labstation then try to withdraw any existing
368 # reboot request created by this servo because it passed repair.
369 if self.is_labstation():
370 self.withdraw_reboot_request()
Garry Wang63b8c382020-03-11 22:28:40 -0700371 except Exception as e:
Garry Wang11b5e872020-03-11 15:14:08 -0700372 self._servo_state = servo_constants.SERVO_STATE_BROKEN
373 self.record('INFO', None, None,
374 'ServoHost repair set servo_state as BROKEN')
Garry Wang63b8c382020-03-11 22:28:40 -0700375 if self._is_critical_error(e):
376 self.disconnect_servo()
377 self.stop_servod()
378 raise
379
380
381 def _is_critical_error(self, error):
382 if (isinstance(error, hosts.AutoservVerifyDependencyError)
383 and not error.is_critical()):
384 logging.warning('Non-critical verify failure(s) detected during'
385 ' verify/repair servo, servo connection will'
386 ' still up but may not fully functional.'
387 ' Some repair actions and servo depended'
388 ' tests may not run.')
389 return False
390 logging.info('Critical verify failure(s) detected during repair/verify'
391 ' servo. Disconnecting servo and stop servod, all repair '
392 'action and tests that depends on servo will not run.')
393 return True
Fang Deng5d518f42013-08-02 14:04:32 -0700394
395
Dan Shi4d478522014-02-14 13:46:32 -0800396 def get_servo(self):
397 """Get the cached servo.Servo object.
Fang Deng5d518f42013-08-02 14:04:32 -0700398
Dan Shi4d478522014-02-14 13:46:32 -0800399 @return: a servo.Servo object.
Dana Goyette353d1d92019-06-27 10:43:59 -0700400 @rtype: autotest_lib.server.cros.servo.servo.Servo
Fang Deng5d518f42013-08-02 14:04:32 -0700401 """
Dan Shi4d478522014-02-14 13:46:32 -0800402 return self._servo
403
404
Garry Wang79e9af62019-06-12 15:19:19 -0700405 def request_reboot(self):
406 """Request servohost to be rebooted when it's safe to by touch a file.
407 """
408 logging.debug('Request to reboot servohost %s has been created by '
Garry Wang464ff1e2019-07-18 17:20:34 -0700409 'servo with port # %s', self.hostname, self.servo_port)
Garry Wang79e9af62019-06-12 15:19:19 -0700410 self.run('touch %s' % self._reboot_file, ignore_status=True)
411
412
Garry Wang464ff1e2019-07-18 17:20:34 -0700413 def withdraw_reboot_request(self):
414 """Withdraw a servohost reboot request if exists by remove the flag
415 file.
416 """
417 logging.debug('Withdrawing request to reboot servohost %s that created'
418 ' by servo with port # %s if exists.',
419 self.hostname, self.servo_port)
420 self.run('rm -f %s' % self._reboot_file, ignore_status=True)
421
422
Garry Wangc1288cf2019-12-17 14:58:00 -0800423 def start_servod(self, quick_startup=False):
424 """Start the servod process on servohost.
425 """
Garry Wang2ac15ee2019-12-30 19:03:02 -0800426 # Skip if running on the localhost.(crbug.com/1038168)
427 if self.is_localhost():
428 logging.debug("Servohost is a localhost, skipping start servod.")
429 return
430
431 cmd = 'start servod'
Garry Wangc1288cf2019-12-17 14:58:00 -0800432 if self.servo_board:
Garry Wang2ac15ee2019-12-30 19:03:02 -0800433 cmd += ' BOARD=%s' % self.servo_board
Garry Wangc1288cf2019-12-17 14:58:00 -0800434 if self.servo_model:
435 cmd += ' MODEL=%s' % self.servo_model
Garry Wangc1288cf2019-12-17 14:58:00 -0800436 else:
Garry Wang2ac15ee2019-12-30 19:03:02 -0800437 logging.warning('Board for DUT is unknown; starting servod'
438 ' assuming a pre-configured board.')
439
440 cmd += ' PORT=%d' % self.servo_port
441 if self.servo_serial:
442 cmd += ' SERIAL=%s' % self.servo_serial
Garry Wangd7367482020-02-27 13:52:40 -0800443
444 # Start servod with dual_v4 if the DUT/servo from designated pools.
445 dut_host_info = self.get_dut_host_info()
446 if dut_host_info:
Garry Wang11b5e872020-03-11 15:14:08 -0700447 if bool(dut_host_info.pools &
448 servo_constants.POOLS_SUPPORT_DUAL_V4):
Garry Wangd7367482020-02-27 13:52:40 -0800449 logging.debug('The DUT is detected in following designated'
450 ' pools %s,starting servod with DUAL_V4 option.',
Garry Wang11b5e872020-03-11 15:14:08 -0700451 servo_constants.POOLS_SUPPORT_DUAL_V4)
Garry Wangd7367482020-02-27 13:52:40 -0800452 cmd += ' DUAL_V4=1'
453
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800454 # Remove the symbolic links from the logs. This helps ensure that
455 # a failed servod instantiation does not cause us to grab old logs
456 # by mistake.
457 self.remove_latest_log_symlinks()
Garry Wangcdd27b22020-01-13 14:59:11 -0800458 self.run(cmd, timeout=60)
Garry Wangc1288cf2019-12-17 14:58:00 -0800459
460 # There's a lag between when `start servod` completes and when
461 # the _ServodConnectionVerifier trigger can actually succeed.
462 # The call to time.sleep() below gives time to make sure that
463 # the trigger won't fail after we return.
464
465 # Normally servod on servo_v3 and labstation take ~10 seconds to ready,
466 # But in the rare case all servo on a labstation are in heavy use they
467 # may take ~30 seconds. So the timeout value will double these value,
468 # and we'll try quick start up when first time initialize servohost,
469 # and use standard start up timeout in repair.
470 if quick_startup:
Garry Wang11b5e872020-03-11 15:14:08 -0700471 timeout = servo_constants.SERVOD_QUICK_STARTUP_TIMEOUT
Garry Wangc1288cf2019-12-17 14:58:00 -0800472 else:
Garry Wang11b5e872020-03-11 15:14:08 -0700473 timeout = servo_constants.SERVOD_STARTUP_TIMEOUT
Garry Wangc1288cf2019-12-17 14:58:00 -0800474 logging.debug('Wait %s seconds for servod process fully up.', timeout)
475 time.sleep(timeout)
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -0700476 # Cache the initial instance timestamp to check against servod restarts
477 self._initial_instance_ts = self.get_instance_logs_ts()
Garry Wangc1288cf2019-12-17 14:58:00 -0800478
479
480 def stop_servod(self):
481 """Stop the servod process on servohost.
482 """
Garry Wang2ac15ee2019-12-30 19:03:02 -0800483 # Skip if running on the localhost.(crbug.com/1038168)
484 if self.is_localhost():
485 logging.debug("Servohost is a localhost, skipping stop servod.")
486 return
487
Garry Wangc1288cf2019-12-17 14:58:00 -0800488 logging.debug('Stopping servod on port %s', self.servo_port)
Garry Wangcdd27b22020-01-13 14:59:11 -0800489 self.run('stop servod PORT=%d' % self.servo_port,
490 timeout=60, ignore_status=True)
Garry Wangc1288cf2019-12-17 14:58:00 -0800491 logging.debug('Wait %s seconds for servod process fully teardown.',
Garry Wang11b5e872020-03-11 15:14:08 -0700492 servo_constants.SERVOD_TEARDOWN_TIMEOUT)
493 time.sleep(servo_constants.SERVOD_TEARDOWN_TIMEOUT)
Garry Wangc1288cf2019-12-17 14:58:00 -0800494
495
496 def restart_servod(self, quick_startup=False):
497 """Restart the servod process on servohost.
498 """
499 self.stop_servod()
500 self.start_servod(quick_startup)
501
Garry Wang11b5e872020-03-11 15:14:08 -0700502
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800503 def _extract_compressed_logs(self, logdir, relevant_files):
504 """Decompress servod logs in |logdir|.
505
506 @param logdir: directory containing compressed servod logs.
507 @param relevant_files: list of files in |logdir| to consider.
508
509 @returns: tuple, (tarfiles, files) where
510 tarfiles: list of the compressed filenames that have been
511 extracted and deleted
512 files: list of the uncompressed files that were generated
513 """
514 # For all tar-files, first extract them to the directory, and
515 # then let the common flow handle them.
516 tarfiles = [cf for cf in relevant_files if
517 cf.endswith(self.COMPRESSION_SUFFIX)]
518 files = []
519 for f in tarfiles:
520 norm_name = os.path.basename(f)[:-len(self.COMPRESSION_SUFFIX)]
521 with tarfile.open(f) as tf:
522 # Each tarfile has only one member, as
523 # that's the compressed log.
524 member = tf.members[0]
525 # Manipulate so that it only extracts the basename, and not
526 # the directories etc.
527 member.name = norm_name
528 files.append(os.path.join(logdir, member.name))
529 tf.extract(member, logdir)
530 # File has been extracted: remove the compressed file.
531 os.remove(f)
532 return tarfiles, files
533
534 def _extract_mcu_logs(self, log_subdir):
535 """Extract MCU (EC, Cr50, etc) console output from servod debug logs.
536
537 Using the MCU_EXTRACTOR regex (above) extract and split out MCU console
538 lines from the logs to generate invidiual console logs e.g. after
539 this method, you can find an ec.txt and servo_v4.txt in |log_dir| if
540 those MCUs had any console input/output.
541
542 @param log_subdir: directory with log.DEBUG.txt main servod debug logs.
543 """
544 # Extract the MCU for each one. The MCU logs are only in the .DEBUG
545 # files
546 mcu_lines_file = os.path.join(log_subdir, 'log.DEBUG.txt')
547 if not os.path.exists(mcu_lines_file):
548 logging.info('No DEBUG logs found to extract MCU logs from.')
549 return
550 mcu_files = {}
551 mcu_file_template = '%s.txt'
552 with open(mcu_lines_file, 'r') as f:
553 for line in f:
554 match = self.MCU_EXTRACTOR.match(line)
555 if match:
556 mcu = match.group(self.MCU_GROUP).lower()
557 line = match.group(self.LINE_GROUP)
558 if mcu not in mcu_files:
559 mcu_file = os.path.join(log_subdir,
560 mcu_file_template % mcu)
561 mcu_files[mcu] = open(mcu_file, 'a')
562 fd = mcu_files[mcu]
563 fd.write(line + '\n')
564 for f in mcu_files:
565 mcu_files[f].close()
566
567
568 def remove_latest_log_symlinks(self):
569 """Remove the conveninence symlinks 'latest' servod logs."""
570 symlink_wildcard = '%s/latest*' % self.remote_log_dir
571 cmd = 'rm ' + symlink_wildcard
572 self.run(cmd, stderr_tee=None, ignore_status=True)
573
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -0700574 def probe_servod_restart(self, instance_ts, outdir):
575 """Grab servod logs from previous instances if part of this session.
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800576
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -0700577 If since the last time this host called start_servod() servod crashed
578 and restarted, this helper finds those logs as well, and stores them
579 with the |OLD_LOG_SUFFIX| to investigate if necessary.
Prasad Vuppalapu5bd9da12020-03-31 01:46:47 +0000580
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -0700581 It also issues a panicinfo command to servo devices after the restart
582 to try and collect reboot information for debugging.
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800583
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -0700584 @param instance_ts: the log timestamp that the current instance uses
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800585 @param outdir: directory to create a subdirectory into to place the
586 servod logs into.
587 """
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -0700588 if self._initial_instance_ts is None:
589 logging.info('No log timestamp grabbed successfully on servod '
590 'startup. Cannot check device restarts. Ignoring.')
591 return
592 if instance_ts == self._initial_instance_ts:
593 logging.debug('Servod appears to have run without restarting')
594 return
595 # Servod seems to have restarted (at least once). |_initial_instance_ts|
596 # is the first timestamp, and instance_ts is the current timestamp. Find
597 # all timestamps in between them, and grab the logs for each.
598 tss = self._find_instance_timestamps_between(self._initial_instance_ts,
599 instance_ts)
600 logging.info('Servod has restarted %d times between the start and the '
601 'end of this servo_host.', len(tss))
602 logging.info('This might be an issue. Will extract all logs from each '
603 'instance.')
604 logging.info('Logs that are not the currently running (about to turn '
605 'down) instance are maked with a .%s in their folder.',
606 self.OLD_LOG_SUFFIX)
607 for ts in tss:
608 self.get_instance_logs(ts, outdir, old=True)
609 # Lastly, servod has restarted due to a potential issue. Try to get
610 # panic information from servo micro and servo v4 for the current logs.
611 # This can only happen if the |_servo| attribute is initialized.
612 if self._servo:
613 for mcu in ['servo_micro', 'servo_v4']:
614 ctrl = '%s_uart_cmd' % mcu
615 if self._servo.has_control(ctrl):
616 logging.info('Trying to retrieve %r panicinfo into logs',
617 mcu)
618 try:
619 self._servo.set_nocheck(ctrl, 'panicinfo')
620 except error.TestFail as e:
621 logging.error('Failed to generate panicinfo for %r '
622 'logs. %s', mcu, str(e))
623
624 def _find_instance_timestamps_between(self, start_ts, end_ts):
625 """Find all log timestamps between [start_ts, end_ts).
626
627 @param start_ts: str, earliest log timestamp of interest
628 @param end_ts: str, latest log timestamp of interest
629
630 @returns: list, all timestamps between start_ts and end_ts, end_ts
631 exclusive, on the servo_host. An empty list on errors
632 """
633 # Simply get all timestamp, and then sort and remove
634 cmd = 'ls %s' % self.remote_log_dir
635 res = self.run(cmd, stderr_tee=None, ignore_status=True)
636 if res.exit_status != 0:
637 # Here we failed to find anything.
638 logging.info('Failed to find remote servod logs. Ignoring.')
639 return []
640 logfiles = res.stdout.strip().split()
641 timestamps = set()
642 for logfile in logfiles:
643 ts_match = self.TS_EXTRACTOR.match(logfile)
644 if not ts_match:
645 # Simply ignore files that fail the check. It might be the
646 # 'latest' symlinks or random files.
647 continue
648 timestamps.add(ts_match.group(self.TS_GROUP))
649 # At this point we have all unique timestamps.
650 timestamps = sorted(timestamps)
651 for ts in [start_ts, end_ts]:
652 if ts not in timestamps:
653 logging.error('Timestamp %r not in servod logs. Cannot query '
654 'for timestamps in between %r and %r', ts,
655 start_ts, end_ts)
656 return []
657 return timestamps[timestamps.index(start_ts):timestamps.index(end_ts)]
658
659 def get_instance_logs_ts(self):
660 """Retrieve the currently running servod instance's log timestamp
661
662 @returns: str, timestamp for current instance, or None on failure
663 """
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800664 # First, extract the timestamp. This cmd gives the real filename of
665 # the latest aka current log file.
666 cmd = ('if [ -f %(dir)s/latest.DEBUG ];'
667 'then realpath %(dir)s/latest.DEBUG;'
668 'elif [ -f %(dir)s/latest ];'
669 'then realpath %(dir)s/latest;'
670 'else exit %(code)d;'
671 'fi' % {'dir': self.remote_log_dir,
672 'code': self.NO_SYMLINKS_CODE})
673 res = self.run(cmd, stderr_tee=None, ignore_status=True)
674 if res.exit_status != 0:
675 if res.exit_status == self.NO_SYMLINKS_CODE:
676 logging.warning('servod log latest symlinks not found. '
677 'This is likely due to an error starting up '
678 'servod. Ignoring..')
679 else:
680 logging.warning('Failed to find servod logs on servo host.')
681 logging.warning(res.stderr.strip())
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -0700682 return None
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800683 fname = os.path.basename(res.stdout.strip())
684 # From the fname, ought to extract the timestamp using the TS_EXTRACTOR
Ruben Rodriguez Buchillone9aa2b02020-03-04 12:14:28 -0800685 ts_match = self.TS_EXTRACTOR.match(fname)
686 if not ts_match:
687 logging.warning('Failed to extract timestamp from servod log file '
688 '%r. Skipping. The servo host is using outdated '
689 'servod logging and needs to be updated.', fname)
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -0700690 return None
691 return ts_match.group(self.TS_GROUP)
692
693 def get_instance_logs(self, instance_ts, outdir, old=False):
694 """Collect all logs with |instance_ts| and dump into a dir in |outdir|
695
696 This method first collects all logs on the servo_host side pertaining
697 to this servod instance (port, instatiation). It glues them together
698 into combined log.[level].txt files and extracts all available MCU
699 console I/O from the logs into individual files e.g. servo_v4.txt
700
701 All the output can be found in a directory inside |outdir| that
702 this generates based on |LOG_DIR|, the servod port, and the instance
703 timestamp on the servo_host side.
704
705 @param instance_ts: log timestamp to grab logfiles for
706 @param outdir: directory to create a subdirectory into to place the
707 servod logs into.
708 @param old: bool, whether to append |OLD_LOG_SUFFIX| to output dir
709 """
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800710 # Create the local results log dir.
711 log_dir = os.path.join(outdir, '%s_%s.%s' % (self.LOG_DIR,
712 str(self.servo_port),
713 instance_ts))
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -0700714 if old:
715 log_dir = '%s.%s' % (log_dir, self.OLD_LOG_SUFFIX)
716 logging.info('Saving servod logs to %r.', log_dir)
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800717 os.mkdir(log_dir)
718 # Now, get all files with that timestamp.
719 cmd = 'find %s -maxdepth 1 -name "log.%s*"' % (self.remote_log_dir,
720 instance_ts)
721 res = self.run(cmd, stderr_tee=None, ignore_status=True)
722 files = res.stdout.strip().split()
723 try:
724 self.get_file(files, log_dir, try_rsync=False)
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -0700725 if not os.listdir(log_dir):
726 logging.info('No servod logs retrieved. Ignoring, and removing '
727 '%r again.', log_dir)
728 os.rmdir(log_dir)
729 return
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800730 except error.AutoservRunError as e:
731 result = e.result_obj
732 if result.exit_status != 0:
733 stderr = result.stderr.strip()
734 logging.warning("Couldn't retrieve servod logs. Ignoring: %s",
735 stderr or '\n%s' % result)
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -0700736 # Remove the log_dir as nothing was added to it.
737 os.rmdir(log_dir)
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800738 return
739 local_files = [os.path.join(log_dir, f) for f in os.listdir(log_dir)]
740 # TODO(crrev.com/c/1793030): remove no-level case once CL is pushed
741 for level_name in ('DEBUG', 'INFO', 'WARNING', ''):
742 # Create the joint files for each loglevel. i.e log.DEBUG
743 joint_file = self.JOINT_LOG_PREFIX
744 if level_name:
745 joint_file = '%s.%s' % (self.JOINT_LOG_PREFIX, level_name)
746 # This helps with some online tools to avoid complaints about an
747 # unknown filetype.
748 joint_file = joint_file + '.txt'
749 joint_path = os.path.join(log_dir, joint_file)
750 files = [f for f in local_files if level_name in f]
751 if not files:
752 # TODO(crrev.com/c/1793030): remove no-level case once CL
753 # is pushed
754 continue
755 # Extract compressed logs if any.
756 compressed, extracted = self._extract_compressed_logs(log_dir,
757 files)
758 files = list(set(files) - set(compressed))
759 files.extend(extracted)
760 # Need to sort. As they all share the same timestamp, and
761 # loglevel, the index itself is sufficient. The highest index
762 # is the oldest file, therefore we need a descending sort.
763 def sortkey(f, level=level_name):
764 """Custom sortkey to sort based on rotation number int."""
765 if f.endswith(level_name): return 0
766 return int(f.split('.')[-1])
767
768 files.sort(reverse=True, key=sortkey)
769 # Just rename the first file rather than building from scratch.
770 os.rename(files[0], joint_path)
771 with open(joint_path, 'a') as joint_f:
772 for logfile in files[1:]:
773 # Transfer the file to the joint file line by line.
774 with open(logfile, 'r') as log_f:
775 for line in log_f:
776 joint_f.write(line)
777 # File has been written over. Delete safely.
778 os.remove(logfile)
779 # Need to remove all files form |local_files| so we don't
780 # analyze them again.
781 local_files = list(set(local_files) - set(files) - set(compressed))
782 # Lastly, extract MCU logs from the joint logs.
783 self._extract_mcu_logs(log_dir)
784
Garry Wang79e9af62019-06-12 15:19:19 -0700785 def _lock(self):
786 """lock servohost by touching a file.
787 """
788 logging.debug('Locking servohost %s by touching %s file',
789 self.hostname, self._lock_file)
790 self.run('touch %s' % self._lock_file, ignore_status=True)
Garry Wang7c00b0f2019-06-25 17:28:17 -0700791 self._is_locked = True
Garry Wang79e9af62019-06-12 15:19:19 -0700792
793
794 def _unlock(self):
795 """Unlock servohost by removing the lock file.
796 """
797 logging.debug('Unlocking servohost by removing %s file',
798 self._lock_file)
799 self.run('rm %s' % self._lock_file, ignore_status=True)
Garry Wang7c00b0f2019-06-25 17:28:17 -0700800 self._is_locked = False
Garry Wang79e9af62019-06-12 15:19:19 -0700801
802
Congbin Guoa1f9cba2018-07-03 11:36:59 -0700803 def close(self):
Congbin Guofc3b8962019-03-22 17:38:46 -0700804 """Close the associated servo and the host object."""
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -0700805 # NOTE: throughout this method there are multiple attempts to catch
806 # all errors. This is WAI as log grabbing should not fail tests.
807 # However, the goal is to catch and handle/process all errors, thus
808 # we print the traceback and ask for a bug.
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800809 if self._closed:
810 logging.debug('ServoHost is already closed.')
811 return
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -0700812 instance_ts = self.get_instance_logs_ts()
813 # TODO(crbug.com/1011516): once enabled, remove the check against
814 # localhost and instead check against log-rotiation enablement.
815 logs_available = (instance_ts is not None and
816 self.job and
817 not self.is_localhost())
818 if logs_available:
819 # Probe whether there was a servod restart, and grab those old
820 # logs as well.
821 try:
822 self.probe_servod_restart(instance_ts, self.job.resultdir)
823 except (error.AutoservRunError, error.TestFail) as e:
824 logging.info('Failed to grab servo logs due to: %s. '
825 'This error is forgiven.', str(e))
826 except Exception as e:
827 logging.error('Unexpected error probing for old logs. %s. '
828 'Forgiven. Please file a bug and fix or catch '
829 'in log probing function', str(e),
830 exc_info=True)
Congbin Guoa1f9cba2018-07-03 11:36:59 -0700831 if self._servo:
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800832 outdir = None if not self.job else self.job.resultdir
Congbin Guo2e5e2a22018-07-27 10:32:48 -0700833 # In some cases when we run as lab-tools, the job object is None.
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800834 self._servo.close(outdir)
835
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -0700836 if logs_available:
837 # Grab current (not old like above) logs after the servo instance
838 # was closed out.
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800839 try:
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -0700840 self.get_instance_logs(instance_ts, self.job.resultdir)
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800841 except error.AutoservRunError as e:
842 logging.info('Failed to grab servo logs due to: %s. '
843 'This error is forgiven.', str(e))
Ruben Rodriguez Buchillon5bac3062020-03-25 21:32:58 -0700844 except Exception as e:
845 logging.error('Unexpected error grabbing servod logs. %s. '
846 'Forgiven. Please file a bug and fix or catch '
847 'in log grabbing function', str(e), exc_info=True)
Congbin Guoa1f9cba2018-07-03 11:36:59 -0700848
Garry Wang7c00b0f2019-06-25 17:28:17 -0700849 if self._is_locked:
850 # Remove the lock if the servohost has been locked.
Garry Wang79e9af62019-06-12 15:19:19 -0700851 try:
852 self._unlock()
853 except error.AutoservSSHTimeout:
854 logging.error('Unlock servohost failed due to ssh timeout.'
855 ' It may caused by servohost went down during'
856 ' the task.')
Garry Wangc1288cf2019-12-17 14:58:00 -0800857 # We want always stop servod after task to minimum the impact of bad
858 # servod process interfere other servods.(see crbug.com/1028665)
Garry Wang4c624bc2020-01-27 16:34:43 -0800859 try:
860 self.stop_servod()
861 except error.AutoservRunError as e:
862 logging.info("Failed to stop servod due to:\n%s\n"
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800863 "This error is forgiven.", str(e))
Garry Wangc1288cf2019-12-17 14:58:00 -0800864
Congbin Guoa1f9cba2018-07-03 11:36:59 -0700865 super(ServoHost, self).close()
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800866 # Mark closed.
867 self._closed = True
Congbin Guoa1f9cba2018-07-03 11:36:59 -0700868
869
Otabek Kasimovcc9738e2020-02-14 16:17:15 -0800870 def get_servo_state(self):
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700871 if self._servo_state is None:
Garry Wang11b5e872020-03-11 15:14:08 -0700872 return servo_constants.SERVO_STATE_UNKNOWN
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700873 return self._servo_state
Otabek Kasimovcc9738e2020-02-14 16:17:15 -0800874
875
Richard Barnetteea3e4602016-06-10 12:36:41 -0700876def make_servo_hostname(dut_hostname):
877 """Given a DUT's hostname, return the hostname of its servo.
878
879 @param dut_hostname: hostname of a DUT.
880
881 @return hostname of the DUT's servo.
882
883 """
884 host_parts = dut_hostname.split('.')
885 host_parts[0] = host_parts[0] + '-servo'
886 return '.'.join(host_parts)
887
888
889def servo_host_is_up(servo_hostname):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700890 """Given a servo host name, return if it's up or not.
Richard Barnetteea3e4602016-06-10 12:36:41 -0700891
892 @param servo_hostname: hostname of the servo host.
893
894 @return True if it's up, False otherwise
895 """
896 # Technically, this duplicates the SSH ping done early in the servo
897 # proxy initialization code. However, this ping ends in a couple
898 # seconds when if fails, rather than the 60 seconds it takes to decide
899 # that an SSH ping has timed out. Specifically, that timeout happens
900 # when our servo DNS name resolves, but there is no host at that IP.
901 logging.info('Pinging servo host at %s', servo_hostname)
902 ping_config = ping_runner.PingConfig(
903 servo_hostname, count=3,
904 ignore_result=True, ignore_status=True)
905 return ping_runner.PingRunner().ping(ping_config).received > 0
906
907
Richard Barnettee519dcd2016-08-15 17:37:17 -0700908def _map_afe_board_to_servo_board(afe_board):
909 """Map a board we get from the AFE to a servo appropriate value.
910
911 Many boards are identical to other boards for servo's purposes.
912 This function makes that mapping.
913
914 @param afe_board string board name received from AFE.
915 @return board we expect servo to have.
916
917 """
918 KNOWN_SUFFIXES = ['-freon', '_freon', '_moblab', '-cheets']
919 BOARD_MAP = {'gizmo': 'panther'}
920 mapped_board = afe_board
921 if afe_board in BOARD_MAP:
922 mapped_board = BOARD_MAP[afe_board]
923 else:
924 for suffix in KNOWN_SUFFIXES:
925 if afe_board.endswith(suffix):
926 mapped_board = afe_board[0:-len(suffix)]
927 break
928 if mapped_board != afe_board:
929 logging.info('Mapping AFE board=%s to %s', afe_board, mapped_board)
930 return mapped_board
931
932
Prathmesh Prabhub4810232018-09-07 13:24:08 -0700933def get_servo_args_for_host(dut_host):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700934 """Return servo data associated with a given DUT.
Richard Barnetteea3e4602016-06-10 12:36:41 -0700935
Richard Barnetteea3e4602016-06-10 12:36:41 -0700936 @param dut_host Instance of `Host` on which to find the servo
937 attributes.
Prathmesh Prabhuf605dd32018-08-28 17:09:04 -0700938 @return `servo_args` dict with host and an optional port.
Richard Barnetteea3e4602016-06-10 12:36:41 -0700939 """
Prathmesh Prabhucba44292018-08-28 17:44:45 -0700940 info = dut_host.host_info_store.get()
941 servo_args = {k: v for k, v in info.attributes.iteritems()
Garry Wang11b5e872020-03-11 15:14:08 -0700942 if k in servo_constants.SERVO_ATTR_KEYS}
Richard Barnetteea3e4602016-06-10 12:36:41 -0700943
Garry Wang11b5e872020-03-11 15:14:08 -0700944 if servo_constants.SERVO_PORT_ATTR in servo_args:
Prathmesh Prabhucba44292018-08-28 17:44:45 -0700945 try:
Garry Wang11b5e872020-03-11 15:14:08 -0700946 servo_args[servo_constants.SERVO_PORT_ATTR] = int(
947 servo_args[servo_constants.SERVO_PORT_ATTR])
Prathmesh Prabhucba44292018-08-28 17:44:45 -0700948 except ValueError:
949 logging.error('servo port is not an int: %s',
Garry Wang11b5e872020-03-11 15:14:08 -0700950 servo_args[servo_constants.SERVO_PORT_ATTR])
Prathmesh Prabhucba44292018-08-28 17:44:45 -0700951 # Reset servo_args because we don't want to use an invalid port.
Garry Wang11b5e872020-03-11 15:14:08 -0700952 servo_args.pop(servo_constants.SERVO_HOST_ATTR, None)
Prathmesh Prabhucba44292018-08-28 17:44:45 -0700953
954 if info.board:
Garry Wang11b5e872020-03-11 15:14:08 -0700955 servo_board = _map_afe_board_to_servo_board(info.board)
956 servo_args[servo_constants.SERVO_BOARD_ATTR] = servo_board
Nick Sanders2f3c9852018-10-24 12:10:24 -0700957 if info.model:
Garry Wang11b5e872020-03-11 15:14:08 -0700958 servo_args[servo_constants.SERVO_MODEL_ATTR] = info.model
959 return servo_args if servo_constants.SERVO_HOST_ATTR in servo_args else None
Richard Barnetteea3e4602016-06-10 12:36:41 -0700960
961
Prathmesh Prabhuefb1b482018-08-28 17:15:05 -0700962def _tweak_args_for_ssp_moblab(servo_args):
Garry Wang11b5e872020-03-11 15:14:08 -0700963 if (servo_args[servo_constants.SERVO_HOST_ATTR]
964 in ['localhost', '127.0.0.1']):
965 servo_args[servo_constants.SERVO_HOST_ATTR] = _CONFIG.get_config_value(
Prathmesh Prabhuefb1b482018-08-28 17:15:05 -0700966 'SSP', 'host_container_ip', type=str, default=None)
967
968
Dan Shi023aae32016-05-25 11:13:01 -0700969def create_servo_host(dut, servo_args, try_lab_servo=False,
Gregory Nisbetde13e2a2019-12-09 22:44:00 -0800970 try_servo_repair=False, dut_host_info=None):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700971 """Create a ServoHost object for a given DUT, if appropriate.
Dan Shi4d478522014-02-14 13:46:32 -0800972
Richard Barnette9a26ad62016-06-10 12:03:08 -0700973 This function attempts to create and verify or repair a `ServoHost`
974 object for a servo connected to the given `dut`, subject to various
975 constraints imposed by the parameters:
976 * When the `servo_args` parameter is not `None`, a servo
977 host must be created, and must be checked with `repair()`.
978 * Otherwise, if a servo exists in the lab and `try_lab_servo` is
979 true:
980 * If `try_servo_repair` is true, then create a servo host and
981 check it with `repair()`.
982 * Otherwise, if the servo responds to `ping` then create a
983 servo host and check it with `verify()`.
Fang Denge545abb2014-12-30 18:43:47 -0800984
Richard Barnette9a26ad62016-06-10 12:03:08 -0700985 In cases where `servo_args` was not `None`, repair failure
986 exceptions are passed back to the caller; otherwise, exceptions
Richard Barnette07c2e1d2016-10-26 14:24:28 -0700987 are logged and then discarded. Note that this only happens in cases
988 where we're called from a test (not special task) control file that
989 has an explicit dependency on servo. In that case, we require that
990 repair not write to `status.log`, so as to avoid polluting test
991 results.
992
993 TODO(jrbarnette): The special handling for servo in test control
994 files is a thorn in my flesh; I dearly hope to see it cut out before
995 my retirement.
Richard Barnette9a26ad62016-06-10 12:03:08 -0700996
997 Parameters for a servo host consist of a host name, port number, and
998 DUT board, and are determined from one of these sources, in order of
999 priority:
Richard Barnetteea3e4602016-06-10 12:36:41 -07001000 * Servo attributes from the `dut` parameter take precedence over
1001 all other sources of information.
1002 * If a DNS entry for the servo based on the DUT hostname exists in
1003 the CrOS lab network, that hostname is used with the default
Richard Barnette9a26ad62016-06-10 12:03:08 -07001004 port and the DUT's board.
Richard Barnetteea3e4602016-06-10 12:36:41 -07001005 * If no other options are found, the parameters will be taken
Richard Barnette9a26ad62016-06-10 12:03:08 -07001006 from the `servo_args` dict passed in from the caller.
Richard Barnetteea3e4602016-06-10 12:36:41 -07001007
1008 @param dut An instance of `Host` from which to take
1009 servo parameters (if available).
1010 @param servo_args A dictionary with servo parameters to use if
1011 they can't be found from `dut`. If this
1012 argument is supplied, unrepaired exceptions
1013 from `verify()` will be passed back to the
1014 caller.
1015 @param try_lab_servo If not true, servo host creation will be
1016 skipped unless otherwise required by the
1017 caller.
Richard Barnette9a26ad62016-06-10 12:03:08 -07001018 @param try_servo_repair If true, check a servo host with
1019 `repair()` instead of `verify()`.
Dan Shi4d478522014-02-14 13:46:32 -08001020
1021 @returns: A ServoHost object or None. See comments above.
1022
1023 """
Richard Barnette07c2e1d2016-10-26 14:24:28 -07001024 servo_dependency = servo_args is not None
Richard Barnette07c2e1d2016-10-26 14:24:28 -07001025 if dut is not None and (try_lab_servo or servo_dependency):
Prathmesh Prabhub4810232018-09-07 13:24:08 -07001026 servo_args_override = get_servo_args_for_host(dut)
Richard Barnetteea3e4602016-06-10 12:36:41 -07001027 if servo_args_override is not None:
Prathmesh Prabhuefb1b482018-08-28 17:15:05 -07001028 if utils.in_moblab_ssp():
1029 _tweak_args_for_ssp_moblab(servo_args_override)
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -07001030 logging.debug(
1031 'Overriding provided servo_args (%s) with arguments'
1032 ' determined from the host (%s)',
1033 servo_args,
1034 servo_args_override,
1035 )
Richard Barnetteea3e4602016-06-10 12:36:41 -07001036 servo_args = servo_args_override
Prathmesh Prabhucba44292018-08-28 17:44:45 -07001037
Richard Barnetteea3e4602016-06-10 12:36:41 -07001038 if servo_args is None:
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -07001039 logging.debug('No servo_args provided, and failed to find overrides.')
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001040 if try_lab_servo or servo_dependency:
Garry Wang11b5e872020-03-11 15:14:08 -07001041 return None, servo_constants.SERVO_STATE_NOT_CONNECTED
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001042 else:
1043 # For regular test case which not required the servo
1044 return None, None
1045
Garry Wang11b5e872020-03-11 15:14:08 -07001046 servo_hostname = servo_args.get(servo_constants.SERVO_HOST_ATTR)
1047 servo_port = servo_args.get(servo_constants.SERVO_PORT_ATTR)
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001048 if not _is_servo_host_information_exist(servo_hostname, servo_port):
1049 logging.debug(
1050 'Servo connection info missed hostname: %s , port: %s',
1051 servo_hostname, servo_port)
Garry Wang11b5e872020-03-11 15:14:08 -07001052 return None, servo_constants.SERVO_STATE_NOT_CONNECTED
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001053 if not is_servo_host_information_valid(servo_hostname, servo_port):
1054 logging.debug(
1055 'Servo connection info is incorrect hostname: %s , port: %s',
1056 servo_hostname, servo_port)
Garry Wang11b5e872020-03-11 15:14:08 -07001057 return None, servo_constants.SERVO_STATE_WRONG_CONFIG
Richard Barnette07c2e1d2016-10-26 14:24:28 -07001058 if (not servo_dependency and not try_servo_repair and
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001059 not servo_host_is_up(servo_hostname)):
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -07001060 logging.debug('ServoHost is not up.')
Garry Wang11b5e872020-03-11 15:14:08 -07001061 return None, servo_constants.SERVO_STATE_BROKEN
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -07001062
Garry Wangebc015b2019-06-06 17:45:06 -07001063 newhost = ServoHost(**servo_args)
Garry Wangcdd27b22020-01-13 14:59:11 -08001064 try:
1065 newhost.restart_servod(quick_startup=True)
1066 except error.AutoservSSHTimeout:
1067 logging.warning("Restart servod failed due ssh connection "
1068 "to servohost timed out. This error is forgiven"
1069 " here, we will retry in servo repair process.")
1070 except error.AutoservRunError as e:
1071 logging.warning("Restart servod failed due to:\n%s\n"
1072 "This error is forgiven here, we will retry"
1073 " in servo repair process.", str(e))
Garry Wangebc015b2019-06-06 17:45:06 -07001074
Gregory Nisbetde13e2a2019-12-09 22:44:00 -08001075 # TODO(gregorynisbet): Clean all of this up.
1076 logging.debug('create_servo_host: attempt to set info store on '
1077 'servo host')
1078 try:
1079 if dut_host_info is None:
1080 logging.debug('create_servo_host: dut_host_info is '
1081 'None, skipping')
1082 else:
1083 newhost.set_dut_host_info(dut_host_info)
1084 logging.debug('create_servo_host: successfully set info '
1085 'store')
1086 except Exception:
1087 logging.error("create_servo_host: (%s)", traceback.format_exc())
1088
Richard Barnette9a26ad62016-06-10 12:03:08 -07001089 # Note that the logic of repair() includes everything done
1090 # by verify(). It's sufficient to call one or the other;
1091 # we don't need both.
Richard Barnette07c2e1d2016-10-26 14:24:28 -07001092 if servo_dependency:
1093 newhost.repair(silent=True)
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001094 return newhost, newhost.get_servo_state()
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -07001095
1096 if try_servo_repair:
1097 try:
1098 newhost.repair()
1099 except Exception:
1100 logging.exception('servo repair failed for %s', newhost.hostname)
Richard Barnette9a26ad62016-06-10 12:03:08 -07001101 else:
1102 try:
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -07001103 newhost.verify()
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -07001104 except Exception:
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -07001105 logging.exception('servo verify failed for %s', newhost.hostname)
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001106 return newhost, newhost.get_servo_state()
Otabek Kasimov7267a7a2020-03-04 11:18:45 -08001107
1108
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001109def _is_servo_host_information_exist(hostname, port):
Otabek Kasimov7267a7a2020-03-04 11:18:45 -08001110 if hostname is None or len(hostname.strip()) == 0:
1111 return False
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001112 if port is None:
Otabek Kasimov7267a7a2020-03-04 11:18:45 -08001113 return False
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001114 if not type(port) is int:
1115 try:
1116 int(port)
1117 except ValueError:
1118 return False
1119
Otabek Kasimov7267a7a2020-03-04 11:18:45 -08001120 return True
1121
1122
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001123def is_servo_host_information_valid(hostname, port):
1124 if not _is_servo_host_information_exist(hostname, port):
Otabek Kasimov7267a7a2020-03-04 11:18:45 -08001125 return False
1126 # checking range and correct of the port
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001127 port_int = int(port)
Otabek Kasimov7267a7a2020-03-04 11:18:45 -08001128 if port_int < 1 or port_int > 65000:
1129 return False
1130 # we expecting host contain only latters, digits and '-' or '_'
Otabek Kasimova7ba91a2020-03-09 08:31:01 -07001131 if not re.match('[a-zA-Z0-9-_\.]*$', hostname) or len(hostname) < 5:
Otabek Kasimov7267a7a2020-03-04 11:18:45 -08001132 return False
1133 return True