blob: 749e1043bb62347631606ac3fdc503b30bcca7f9 [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
Fang Deng5d518f42013-08-02 14:04:32 -070023from autotest_lib.client.common_lib.cros import retry
Christopher Wileycef1f902014-06-19 11:11:23 -070024from autotest_lib.client.common_lib.cros.network import ping_runner
Richard Barnette9a26ad62016-06-10 12:03:08 -070025from autotest_lib.server.cros.servo import servo
Richard Barnetted31580e2018-05-14 19:58:00 +000026from autotest_lib.server.hosts import servo_repair
Garry Wangebc015b2019-06-06 17:45:06 -070027from autotest_lib.server.hosts import base_servohost
Garry Wang11b5e872020-03-11 15:14:08 -070028from autotest_lib.server.hosts import servo_constants
29from autotest_lib.client.common_lib import global_config
Garry Wangd7367482020-02-27 13:52:40 -080030
Dan Shi3b2adf62015-09-02 17:46:54 -070031_CONFIG = global_config.global_config
Fang Deng5d518f42013-08-02 14:04:32 -070032
Otabek Kasimova7ba91a2020-03-09 08:31:01 -070033
Garry Wangebc015b2019-06-06 17:45:06 -070034class ServoHost(base_servohost.BaseServoHost):
35 """Host class for a servo host(e.g. beaglebone, labstation)
Dana Goyette0b6e6402019-10-04 11:09:24 -070036 that with a servo instance for a specific port.
37
38 @type _servo: servo.Servo | None
39 """
Fang Deng5d518f42013-08-02 14:04:32 -070040
Raul E Rangel52ca2e82018-07-03 14:10:14 -060041 DEFAULT_PORT = int(os.getenv('SERVOD_PORT', '9999'))
Richard Barnette9a26ad62016-06-10 12:03:08 -070042
Dan Shie5b3c512014-08-21 12:12:09 -070043 # Timeout for initializing servo signals.
Wai-Hong Tam37b6ed32017-09-19 15:52:39 -070044 INITIALIZE_SERVO_TIMEOUT_SECS = 60
Richard Barnette9a26ad62016-06-10 12:03:08 -070045
xixuan6cf6d2f2016-01-29 15:29:00 -080046 # Ready test function
47 SERVO_READY_METHOD = 'get_version'
Fang Deng5d518f42013-08-02 14:04:32 -070048
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -080049 # Directory prefix on the servo host where the servod logs are stored.
50 SERVOD_LOG_PREFIX = '/var/log/servod'
51
52 # Exit code to use when symlinks for servod logs are not found.
53 NO_SYMLINKS_CODE = 9
54
55 # Directory in the job's results directory to dump the logs into.
56 LOG_DIR = 'servod'
57
58 # Prefix for joint loglevel files in the logs.
59 JOINT_LOG_PREFIX = 'log'
60
61 # Regex group to extract timestamp from logfile name.
62 TS_GROUP = 'ts'
63
64 # This regex is used to extract the timestamp from servod logs.
65 # files always start with log.
66 TS_RE = (r'log.'
67 # The timestamp is of format %Y-%m-%d--%H-%M-%S.MS
68 r'(?P<%s>\d{4}(\-\d{2}){2}\-(-\d{2}){3}.\d{3})'
69 # The loglevel is optional depending on labstation version.
70 r'(.(INFO|DEBUG|WARNING))?' % TS_GROUP)
71 TS_EXTRACTOR = re.compile(TS_RE)
72
73 # Regex group to extract MCU name from logline in servod logs.
74 MCU_GROUP = 'mcu'
75
76 # Regex group to extract logline from MCU logline in servod logs.
77 LINE_GROUP = 'line'
78
79 # This regex is used to extract the mcu and the line content from an
80 # MCU logline in servod logs. e.g. EC or servo_v4 console logs.
81 # Here is an example log-line:
82 #
83 # 2020-01-23 13:15:12,223 - servo_v4 - EC3PO.Console - DEBUG -
84 # console.py:219:LogConsoleOutput - /dev/pts/9 - cc polarity: cc1
85 #
86 # Here is conceptually how they are formatted:
87 #
88 # <time> - <MCU> - EC3PO.Console - <LVL> - <file:line:func> - <pts> -
89 # <output>
90 #
91 # The log format starts with a timestamp
92 MCU_RE = (r'[\d\-]+ [\d:,]+ '
93 # The mcu that is logging this is next.
94 r'- (?P<%s>\w+) - '
95 # Next, we have more log outputs before the actual line.
96 # Information about the file line, logging function etc.
97 # Anchor on EC3PO Console, LogConsoleOutput and dev/pts.
98 # NOTE: if the log format changes, this regex needs to be
99 # adjusted.
100 r'EC3PO\.Console[\s\-\w\d:.]+LogConsoleOutput - /dev/pts/\d+ - '
101 # Lastly, we get the MCU's console line.
102 r'(?P<%s>.+$)' % (MCU_GROUP, LINE_GROUP))
103 MCU_EXTRACTOR = re.compile(MCU_RE)
104
105 # Suffix to identify compressed logfiles.
106 COMPRESSION_SUFFIX = '.tbz2'
107
Otabek Kasimovcc9738e2020-02-14 16:17:15 -0800108 def _init_attributes(self):
109 self._servo_state = None
110 self.servo_port = None
111 self.servo_board = None
112 self.servo_model = None
113 self.servo_serial = None
114 self._servo = None
115 self._servod_server_proxy = None
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800116 # Flag to make sure that multiple calls to close do not result in the
117 # logic executing multiple times.
118 self._closed = False
Fang Deng5d518f42013-08-02 14:04:32 -0700119
Richard Barnette17bfc6c2016-08-04 18:41:43 -0700120 def _initialize(self, servo_host='localhost',
Richard Barnettee519dcd2016-08-15 17:37:17 -0700121 servo_port=DEFAULT_PORT, servo_board=None,
Nick Sanders2f3c9852018-10-24 12:10:24 -0700122 servo_model=None, servo_serial=None, is_in_lab=None,
123 *args, **dargs):
Fang Deng5d518f42013-08-02 14:04:32 -0700124 """Initialize a ServoHost instance.
125
126 A ServoHost instance represents a host that controls a servo.
127
128 @param servo_host: Name of the host where the servod process
129 is running.
Raul E Rangel52ca2e82018-07-03 14:10:14 -0600130 @param servo_port: Port the servod process is listening on. Defaults
131 to the SERVOD_PORT environment variable if set,
132 otherwise 9999.
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700133 @param servo_board: Board that the servo is connected to.
Nick Sanders2f3c9852018-10-24 12:10:24 -0700134 @param servo_model: Model that the servo is connected to.
Dan Shi4d478522014-02-14 13:46:32 -0800135 @param is_in_lab: True if the servo host is in Cros Lab. Default is set
136 to None, for which utils.host_is_in_lab_zone will be
137 called to check if the servo host is in Cros lab.
Fang Deng5d518f42013-08-02 14:04:32 -0700138
139 """
140 super(ServoHost, self)._initialize(hostname=servo_host,
Garry Wangebc015b2019-06-06 17:45:06 -0700141 is_in_lab=is_in_lab, *args, **dargs)
Otabek Kasimovcc9738e2020-02-14 16:17:15 -0800142 self._init_attributes()
Richard Barnette42f4db92018-08-23 15:05:15 -0700143 self.servo_port = int(servo_port)
Richard Barnettee519dcd2016-08-15 17:37:17 -0700144 self.servo_board = servo_board
Nick Sanders2f3c9852018-10-24 12:10:24 -0700145 self.servo_model = servo_model
Kevin Cheng643ce8a2016-09-15 15:42:12 -0700146 self.servo_serial = servo_serial
Wai-Hong Tam3a8a2552019-11-19 14:28:04 +0800147
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800148 # The location of the log files on the servo host for this instance.
149 self.remote_log_dir = '%s_%s' % (self.SERVOD_LOG_PREFIX,
150 self.servo_port)
Garry Wang79e9af62019-06-12 15:19:19 -0700151 # Path of the servo host lock file.
152 self._lock_file = (self.TEMP_FILE_DIR + str(self.servo_port)
153 + self.LOCK_FILE_POSTFIX)
154 # File path to declare a reboot request.
155 self._reboot_file = (self.TEMP_FILE_DIR + str(self.servo_port)
156 + self.REBOOT_FILE_POSTFIX)
157
158 # Lock the servo host if it's an in-lab labstation to prevent other
159 # task to reboot it until current task completes. We also wait and
160 # make sure the labstation is up here, in the case of the labstation is
161 # in the middle of reboot.
Garry Wang7c00b0f2019-06-25 17:28:17 -0700162 self._is_locked = False
Garry Wang42b4d862019-06-25 15:50:49 -0700163 if (self.wait_up(self.REBOOT_TIMEOUT) and self.is_in_lab()
164 and self.is_labstation()):
Garry Wang79e9af62019-06-12 15:19:19 -0700165 self._lock()
Garry Wangebc015b2019-06-06 17:45:06 -0700166
Richard Barnette9a26ad62016-06-10 12:03:08 -0700167 self._repair_strategy = (
168 servo_repair.create_servo_repair_strategy())
Richard Barnettee519dcd2016-08-15 17:37:17 -0700169
Dana Goyetteafa62fd2020-03-16 13:45:27 -0700170 def __str__(self):
171 return "<%s '%s:%s'>" % (
172 type(self).__name__, self.hostname, self.servo_port)
173
Richard Barnette9a26ad62016-06-10 12:03:08 -0700174 def connect_servo(self):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700175 """Establish a connection to the servod server on this host.
Richard Barnette9a26ad62016-06-10 12:03:08 -0700176
177 Initializes `self._servo` and then verifies that all network
178 connections are working. This will create an ssh tunnel if
179 it's required.
180
181 As a side effect of testing the connection, all signals on the
182 target servo are reset to default values, and the USB stick is
183 set to the neutral (off) position.
184 """
Kevin Cheng643ce8a2016-09-15 15:42:12 -0700185 servo_obj = servo.Servo(servo_host=self, servo_serial=self.servo_serial)
Kuang-che Wu05763f52019-08-30 16:48:21 +0800186 self._servo = servo_obj
Richard Barnette9a26ad62016-06-10 12:03:08 -0700187 timeout, _ = retry.timeout(
188 servo_obj.initialize_dut,
189 timeout_sec=self.INITIALIZE_SERVO_TIMEOUT_SECS)
190 if timeout:
191 raise hosts.AutoservVerifyError(
192 'Servo initialize timed out.')
Richard Barnette9a26ad62016-06-10 12:03:08 -0700193
194
195 def disconnect_servo(self):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700196 """Disconnect our servo if it exists.
Richard Barnette9a26ad62016-06-10 12:03:08 -0700197
198 If we've previously successfully connected to our servo,
199 disconnect any established ssh tunnel, and set `self._servo`
200 back to `None`.
201 """
202 if self._servo:
203 # N.B. This call is safe even without a tunnel:
204 # rpc_server_tracker.disconnect() silently ignores
205 # unknown ports.
206 self.rpc_server_tracker.disconnect(self.servo_port)
207 self._servo = None
Fang Deng5d518f42013-08-02 14:04:32 -0700208
Garry Wangc1288cf2019-12-17 14:58:00 -0800209
Wai-Hong Tam3a8a2552019-11-19 14:28:04 +0800210 def _create_servod_server_proxy(self):
211 """Create a proxy that can be used to communicate with servod server.
Fang Deng5d518f42013-08-02 14:04:32 -0700212
213 @returns: An xmlrpclib.ServerProxy that is connected to the servod
214 server on the host.
Fang Deng5d518f42013-08-02 14:04:32 -0700215 """
Garry Wang11b5e872020-03-11 15:14:08 -0700216 if (servo_constants.ENABLE_SSH_TUNNEL_FOR_SERVO
217 and not self.is_localhost()):
Richard Barnette9a26ad62016-06-10 12:03:08 -0700218 return self.rpc_server_tracker.xmlrpc_connect(
219 None, self.servo_port,
220 ready_test_name=self.SERVO_READY_METHOD,
Allen Li2b1a8992018-11-27 14:17:18 -0800221 timeout_seconds=60,
Dana Goyetteafa62fd2020-03-16 13:45:27 -0700222 request_timeout_seconds=3600,
223 server_desc=str(self))
Richard Barnette9a26ad62016-06-10 12:03:08 -0700224 else:
225 remote = 'http://%s:%s' % (self.hostname, self.servo_port)
226 return xmlrpclib.ServerProxy(remote)
Fang Deng5d518f42013-08-02 14:04:32 -0700227
228
Wai-Hong Tam3a8a2552019-11-19 14:28:04 +0800229 def get_servod_server_proxy(self):
230 """Return a cached proxy if exists; otherwise, create a new one.
231
232 @returns: An xmlrpclib.ServerProxy that is connected to the servod
233 server on the host.
234 """
235 # Single-threaded execution, no race
236 if self._servod_server_proxy is None:
237 self._servod_server_proxy = self._create_servod_server_proxy()
238 return self._servod_server_proxy
239
240
Richard Barnette1edbb162016-11-01 11:47:50 -0700241 def verify(self, silent=False):
242 """Update the servo host and verify it's in a good state.
243
244 @param silent If true, suppress logging in `status.log`.
245 """
Richard Barnetteabbdc252018-07-26 16:57:42 -0700246 message = 'Beginning verify for servo host %s port %s serial %s'
247 message %= (self.hostname, self.servo_port, self.servo_serial)
248 self.record('INFO', None, None, message)
Richard Barnette9a26ad62016-06-10 12:03:08 -0700249 try:
Richard Barnette1edbb162016-11-01 11:47:50 -0700250 self._repair_strategy.verify(self, silent)
Garry Wang11b5e872020-03-11 15:14:08 -0700251 self._servo_state = servo_constants.SERVO_STATE_WORKING
252 self.record('INFO', None, None,
253 'ServoHost verify set servo_state as WORKING')
Garry Wang63b8c382020-03-11 22:28:40 -0700254 except Exception as e:
Garry Wang11b5e872020-03-11 15:14:08 -0700255 self._servo_state = servo_constants.SERVO_STATE_BROKEN
256 self.record('INFO', None, None,
257 'ServoHost verify set servo_state as BROKEN')
Garry Wang63b8c382020-03-11 22:28:40 -0700258 if self._is_critical_error(e):
259 raise
Fang Deng5d518f42013-08-02 14:04:32 -0700260
261
Richard Barnette1edbb162016-11-01 11:47:50 -0700262 def repair(self, silent=False):
263 """Attempt to repair servo host.
264
265 @param silent If true, suppress logging in `status.log`.
266 """
Richard Barnetteabbdc252018-07-26 16:57:42 -0700267 message = 'Beginning repair for servo host %s port %s serial %s'
268 message %= (self.hostname, self.servo_port, self.servo_serial)
269 self.record('INFO', None, None, message)
Richard Barnette9a26ad62016-06-10 12:03:08 -0700270 try:
Richard Barnette1edbb162016-11-01 11:47:50 -0700271 self._repair_strategy.repair(self, silent)
Garry Wang11b5e872020-03-11 15:14:08 -0700272 self._servo_state = servo_constants.SERVO_STATE_WORKING
273 self.record('INFO', None, None,
274 'ServoHost repair set servo_state as WORKING')
Garry Wang464ff1e2019-07-18 17:20:34 -0700275 # If target is a labstation then try to withdraw any existing
276 # reboot request created by this servo because it passed repair.
277 if self.is_labstation():
278 self.withdraw_reboot_request()
Garry Wang63b8c382020-03-11 22:28:40 -0700279 except Exception as e:
Garry Wang11b5e872020-03-11 15:14:08 -0700280 self._servo_state = servo_constants.SERVO_STATE_BROKEN
281 self.record('INFO', None, None,
282 'ServoHost repair set servo_state as BROKEN')
Garry Wang63b8c382020-03-11 22:28:40 -0700283 if self._is_critical_error(e):
284 self.disconnect_servo()
285 self.stop_servod()
286 raise
287
288
289 def _is_critical_error(self, error):
290 if (isinstance(error, hosts.AutoservVerifyDependencyError)
291 and not error.is_critical()):
292 logging.warning('Non-critical verify failure(s) detected during'
293 ' verify/repair servo, servo connection will'
294 ' still up but may not fully functional.'
295 ' Some repair actions and servo depended'
296 ' tests may not run.')
297 return False
298 logging.info('Critical verify failure(s) detected during repair/verify'
299 ' servo. Disconnecting servo and stop servod, all repair '
300 'action and tests that depends on servo will not run.')
301 return True
Fang Deng5d518f42013-08-02 14:04:32 -0700302
303
Dan Shi4d478522014-02-14 13:46:32 -0800304 def get_servo(self):
305 """Get the cached servo.Servo object.
Fang Deng5d518f42013-08-02 14:04:32 -0700306
Dan Shi4d478522014-02-14 13:46:32 -0800307 @return: a servo.Servo object.
Dana Goyette353d1d92019-06-27 10:43:59 -0700308 @rtype: autotest_lib.server.cros.servo.servo.Servo
Fang Deng5d518f42013-08-02 14:04:32 -0700309 """
Dan Shi4d478522014-02-14 13:46:32 -0800310 return self._servo
311
312
Garry Wang79e9af62019-06-12 15:19:19 -0700313 def request_reboot(self):
314 """Request servohost to be rebooted when it's safe to by touch a file.
315 """
316 logging.debug('Request to reboot servohost %s has been created by '
Garry Wang464ff1e2019-07-18 17:20:34 -0700317 'servo with port # %s', self.hostname, self.servo_port)
Garry Wang79e9af62019-06-12 15:19:19 -0700318 self.run('touch %s' % self._reboot_file, ignore_status=True)
319
320
Garry Wang464ff1e2019-07-18 17:20:34 -0700321 def withdraw_reboot_request(self):
322 """Withdraw a servohost reboot request if exists by remove the flag
323 file.
324 """
325 logging.debug('Withdrawing request to reboot servohost %s that created'
326 ' by servo with port # %s if exists.',
327 self.hostname, self.servo_port)
328 self.run('rm -f %s' % self._reboot_file, ignore_status=True)
329
330
Garry Wangc1288cf2019-12-17 14:58:00 -0800331 def start_servod(self, quick_startup=False):
332 """Start the servod process on servohost.
333 """
Garry Wang2ac15ee2019-12-30 19:03:02 -0800334 # Skip if running on the localhost.(crbug.com/1038168)
335 if self.is_localhost():
336 logging.debug("Servohost is a localhost, skipping start servod.")
337 return
338
339 cmd = 'start servod'
Garry Wangc1288cf2019-12-17 14:58:00 -0800340 if self.servo_board:
Garry Wang2ac15ee2019-12-30 19:03:02 -0800341 cmd += ' BOARD=%s' % self.servo_board
Garry Wangc1288cf2019-12-17 14:58:00 -0800342 if self.servo_model:
343 cmd += ' MODEL=%s' % self.servo_model
Garry Wangc1288cf2019-12-17 14:58:00 -0800344 else:
Garry Wang2ac15ee2019-12-30 19:03:02 -0800345 logging.warning('Board for DUT is unknown; starting servod'
346 ' assuming a pre-configured board.')
347
348 cmd += ' PORT=%d' % self.servo_port
349 if self.servo_serial:
350 cmd += ' SERIAL=%s' % self.servo_serial
Garry Wangd7367482020-02-27 13:52:40 -0800351
352 # Start servod with dual_v4 if the DUT/servo from designated pools.
353 dut_host_info = self.get_dut_host_info()
354 if dut_host_info:
Garry Wang11b5e872020-03-11 15:14:08 -0700355 if bool(dut_host_info.pools &
356 servo_constants.POOLS_SUPPORT_DUAL_V4):
Garry Wangd7367482020-02-27 13:52:40 -0800357 logging.debug('The DUT is detected in following designated'
358 ' pools %s,starting servod with DUAL_V4 option.',
Garry Wang11b5e872020-03-11 15:14:08 -0700359 servo_constants.POOLS_SUPPORT_DUAL_V4)
Garry Wangd7367482020-02-27 13:52:40 -0800360 cmd += ' DUAL_V4=1'
361
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800362 # Remove the symbolic links from the logs. This helps ensure that
363 # a failed servod instantiation does not cause us to grab old logs
364 # by mistake.
365 self.remove_latest_log_symlinks()
Garry Wangcdd27b22020-01-13 14:59:11 -0800366 self.run(cmd, timeout=60)
Garry Wangc1288cf2019-12-17 14:58:00 -0800367
368 # There's a lag between when `start servod` completes and when
369 # the _ServodConnectionVerifier trigger can actually succeed.
370 # The call to time.sleep() below gives time to make sure that
371 # the trigger won't fail after we return.
372
373 # Normally servod on servo_v3 and labstation take ~10 seconds to ready,
374 # But in the rare case all servo on a labstation are in heavy use they
375 # may take ~30 seconds. So the timeout value will double these value,
376 # and we'll try quick start up when first time initialize servohost,
377 # and use standard start up timeout in repair.
378 if quick_startup:
Garry Wang11b5e872020-03-11 15:14:08 -0700379 timeout = servo_constants.SERVOD_QUICK_STARTUP_TIMEOUT
Garry Wangc1288cf2019-12-17 14:58:00 -0800380 else:
Garry Wang11b5e872020-03-11 15:14:08 -0700381 timeout = servo_constants.SERVOD_STARTUP_TIMEOUT
Garry Wangc1288cf2019-12-17 14:58:00 -0800382 logging.debug('Wait %s seconds for servod process fully up.', timeout)
383 time.sleep(timeout)
384
385
386 def stop_servod(self):
387 """Stop the servod process on servohost.
388 """
Garry Wang2ac15ee2019-12-30 19:03:02 -0800389 # Skip if running on the localhost.(crbug.com/1038168)
390 if self.is_localhost():
391 logging.debug("Servohost is a localhost, skipping stop servod.")
392 return
393
Garry Wangc1288cf2019-12-17 14:58:00 -0800394 logging.debug('Stopping servod on port %s', self.servo_port)
Garry Wangcdd27b22020-01-13 14:59:11 -0800395 self.run('stop servod PORT=%d' % self.servo_port,
396 timeout=60, ignore_status=True)
Garry Wangc1288cf2019-12-17 14:58:00 -0800397 logging.debug('Wait %s seconds for servod process fully teardown.',
Garry Wang11b5e872020-03-11 15:14:08 -0700398 servo_constants.SERVOD_TEARDOWN_TIMEOUT)
399 time.sleep(servo_constants.SERVOD_TEARDOWN_TIMEOUT)
Garry Wangc1288cf2019-12-17 14:58:00 -0800400
401
402 def restart_servod(self, quick_startup=False):
403 """Restart the servod process on servohost.
404 """
405 self.stop_servod()
406 self.start_servod(quick_startup)
407
Garry Wang11b5e872020-03-11 15:14:08 -0700408
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800409 def _extract_compressed_logs(self, logdir, relevant_files):
410 """Decompress servod logs in |logdir|.
411
412 @param logdir: directory containing compressed servod logs.
413 @param relevant_files: list of files in |logdir| to consider.
414
415 @returns: tuple, (tarfiles, files) where
416 tarfiles: list of the compressed filenames that have been
417 extracted and deleted
418 files: list of the uncompressed files that were generated
419 """
420 # For all tar-files, first extract them to the directory, and
421 # then let the common flow handle them.
422 tarfiles = [cf for cf in relevant_files if
423 cf.endswith(self.COMPRESSION_SUFFIX)]
424 files = []
425 for f in tarfiles:
426 norm_name = os.path.basename(f)[:-len(self.COMPRESSION_SUFFIX)]
427 with tarfile.open(f) as tf:
428 # Each tarfile has only one member, as
429 # that's the compressed log.
430 member = tf.members[0]
431 # Manipulate so that it only extracts the basename, and not
432 # the directories etc.
433 member.name = norm_name
434 files.append(os.path.join(logdir, member.name))
435 tf.extract(member, logdir)
436 # File has been extracted: remove the compressed file.
437 os.remove(f)
438 return tarfiles, files
439
440 def _extract_mcu_logs(self, log_subdir):
441 """Extract MCU (EC, Cr50, etc) console output from servod debug logs.
442
443 Using the MCU_EXTRACTOR regex (above) extract and split out MCU console
444 lines from the logs to generate invidiual console logs e.g. after
445 this method, you can find an ec.txt and servo_v4.txt in |log_dir| if
446 those MCUs had any console input/output.
447
448 @param log_subdir: directory with log.DEBUG.txt main servod debug logs.
449 """
450 # Extract the MCU for each one. The MCU logs are only in the .DEBUG
451 # files
452 mcu_lines_file = os.path.join(log_subdir, 'log.DEBUG.txt')
453 if not os.path.exists(mcu_lines_file):
454 logging.info('No DEBUG logs found to extract MCU logs from.')
455 return
456 mcu_files = {}
457 mcu_file_template = '%s.txt'
458 with open(mcu_lines_file, 'r') as f:
459 for line in f:
460 match = self.MCU_EXTRACTOR.match(line)
461 if match:
462 mcu = match.group(self.MCU_GROUP).lower()
463 line = match.group(self.LINE_GROUP)
464 if mcu not in mcu_files:
465 mcu_file = os.path.join(log_subdir,
466 mcu_file_template % mcu)
467 mcu_files[mcu] = open(mcu_file, 'a')
468 fd = mcu_files[mcu]
469 fd.write(line + '\n')
470 for f in mcu_files:
471 mcu_files[f].close()
472
473
474 def remove_latest_log_symlinks(self):
475 """Remove the conveninence symlinks 'latest' servod logs."""
476 symlink_wildcard = '%s/latest*' % self.remote_log_dir
477 cmd = 'rm ' + symlink_wildcard
478 self.run(cmd, stderr_tee=None, ignore_status=True)
479
480 def grab_logs(self, outdir):
481 """Retrieve logs from servo_host to |outdir|/servod_{port}.{ts}/.
482
483 This method first collects all logs on the servo_host side pertaining
484 to this servod instance (port, instatiation). It glues them together
485 into combined log.[level].txt files and extracts all available MCU
486 console I/O from the logs into individual files e.g. servo_v4.txt
487
488 All the output can be found in a directory inside |outdir| that
489 this generates based on |LOG_DIR|, the servod port, and the instance
490 timestamp on the servo_host side.
491
492 @param outdir: directory to create a subdirectory into to place the
493 servod logs into.
494 """
495 # First, extract the timestamp. This cmd gives the real filename of
496 # the latest aka current log file.
497 cmd = ('if [ -f %(dir)s/latest.DEBUG ];'
498 'then realpath %(dir)s/latest.DEBUG;'
499 'elif [ -f %(dir)s/latest ];'
500 'then realpath %(dir)s/latest;'
501 'else exit %(code)d;'
502 'fi' % {'dir': self.remote_log_dir,
503 'code': self.NO_SYMLINKS_CODE})
504 res = self.run(cmd, stderr_tee=None, ignore_status=True)
505 if res.exit_status != 0:
506 if res.exit_status == self.NO_SYMLINKS_CODE:
507 logging.warning('servod log latest symlinks not found. '
508 'This is likely due to an error starting up '
509 'servod. Ignoring..')
510 else:
511 logging.warning('Failed to find servod logs on servo host.')
512 logging.warning(res.stderr.strip())
513 return
514 fname = os.path.basename(res.stdout.strip())
515 # From the fname, ought to extract the timestamp using the TS_EXTRACTOR
Ruben Rodriguez Buchillone9aa2b02020-03-04 12:14:28 -0800516 ts_match = self.TS_EXTRACTOR.match(fname)
517 if not ts_match:
518 logging.warning('Failed to extract timestamp from servod log file '
519 '%r. Skipping. The servo host is using outdated '
520 'servod logging and needs to be updated.', fname)
521 return
522 instance_ts = ts_match.group(self.TS_GROUP)
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800523 # Create the local results log dir.
524 log_dir = os.path.join(outdir, '%s_%s.%s' % (self.LOG_DIR,
525 str(self.servo_port),
526 instance_ts))
527 logging.info('Saving servod logs to %s.', log_dir)
528 os.mkdir(log_dir)
529 # Now, get all files with that timestamp.
530 cmd = 'find %s -maxdepth 1 -name "log.%s*"' % (self.remote_log_dir,
531 instance_ts)
532 res = self.run(cmd, stderr_tee=None, ignore_status=True)
533 files = res.stdout.strip().split()
534 try:
535 self.get_file(files, log_dir, try_rsync=False)
536
537 except error.AutoservRunError as e:
538 result = e.result_obj
539 if result.exit_status != 0:
540 stderr = result.stderr.strip()
541 logging.warning("Couldn't retrieve servod logs. Ignoring: %s",
542 stderr or '\n%s' % result)
543 return
544 local_files = [os.path.join(log_dir, f) for f in os.listdir(log_dir)]
545 # TODO(crrev.com/c/1793030): remove no-level case once CL is pushed
546 for level_name in ('DEBUG', 'INFO', 'WARNING', ''):
547 # Create the joint files for each loglevel. i.e log.DEBUG
548 joint_file = self.JOINT_LOG_PREFIX
549 if level_name:
550 joint_file = '%s.%s' % (self.JOINT_LOG_PREFIX, level_name)
551 # This helps with some online tools to avoid complaints about an
552 # unknown filetype.
553 joint_file = joint_file + '.txt'
554 joint_path = os.path.join(log_dir, joint_file)
555 files = [f for f in local_files if level_name in f]
556 if not files:
557 # TODO(crrev.com/c/1793030): remove no-level case once CL
558 # is pushed
559 continue
560 # Extract compressed logs if any.
561 compressed, extracted = self._extract_compressed_logs(log_dir,
562 files)
563 files = list(set(files) - set(compressed))
564 files.extend(extracted)
565 # Need to sort. As they all share the same timestamp, and
566 # loglevel, the index itself is sufficient. The highest index
567 # is the oldest file, therefore we need a descending sort.
568 def sortkey(f, level=level_name):
569 """Custom sortkey to sort based on rotation number int."""
570 if f.endswith(level_name): return 0
571 return int(f.split('.')[-1])
572
573 files.sort(reverse=True, key=sortkey)
574 # Just rename the first file rather than building from scratch.
575 os.rename(files[0], joint_path)
576 with open(joint_path, 'a') as joint_f:
577 for logfile in files[1:]:
578 # Transfer the file to the joint file line by line.
579 with open(logfile, 'r') as log_f:
580 for line in log_f:
581 joint_f.write(line)
582 # File has been written over. Delete safely.
583 os.remove(logfile)
584 # Need to remove all files form |local_files| so we don't
585 # analyze them again.
586 local_files = list(set(local_files) - set(files) - set(compressed))
587 # Lastly, extract MCU logs from the joint logs.
588 self._extract_mcu_logs(log_dir)
589
Garry Wangc1288cf2019-12-17 14:58:00 -0800590
Garry Wang79e9af62019-06-12 15:19:19 -0700591 def _lock(self):
592 """lock servohost by touching a file.
593 """
594 logging.debug('Locking servohost %s by touching %s file',
595 self.hostname, self._lock_file)
596 self.run('touch %s' % self._lock_file, ignore_status=True)
Garry Wang7c00b0f2019-06-25 17:28:17 -0700597 self._is_locked = True
Garry Wang79e9af62019-06-12 15:19:19 -0700598
599
600 def _unlock(self):
601 """Unlock servohost by removing the lock file.
602 """
603 logging.debug('Unlocking servohost by removing %s file',
604 self._lock_file)
605 self.run('rm %s' % self._lock_file, ignore_status=True)
Garry Wang7c00b0f2019-06-25 17:28:17 -0700606 self._is_locked = False
Garry Wang79e9af62019-06-12 15:19:19 -0700607
608
Congbin Guoa1f9cba2018-07-03 11:36:59 -0700609 def close(self):
Congbin Guofc3b8962019-03-22 17:38:46 -0700610 """Close the associated servo and the host object."""
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800611 if self._closed:
612 logging.debug('ServoHost is already closed.')
613 return
Congbin Guoa1f9cba2018-07-03 11:36:59 -0700614 if self._servo:
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800615 outdir = None if not self.job else self.job.resultdir
Congbin Guo2e5e2a22018-07-27 10:32:48 -0700616 # In some cases when we run as lab-tools, the job object is None.
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800617 self._servo.close(outdir)
618
619 if self.job and not self.is_localhost():
620 # Grab all logs from this servod instance before stopping servod.
621 # TODO(crbug.com/1011516): once enabled, remove the check against
622 # localhost and instead check against log-rotiation enablement.
623 try:
624 self.grab_logs(self.job.resultdir)
625 except error.AutoservRunError as e:
626 logging.info('Failed to grab servo logs due to: %s. '
627 'This error is forgiven.', str(e))
Congbin Guoa1f9cba2018-07-03 11:36:59 -0700628
Garry Wang7c00b0f2019-06-25 17:28:17 -0700629 if self._is_locked:
630 # Remove the lock if the servohost has been locked.
Garry Wang79e9af62019-06-12 15:19:19 -0700631 try:
632 self._unlock()
633 except error.AutoservSSHTimeout:
634 logging.error('Unlock servohost failed due to ssh timeout.'
635 ' It may caused by servohost went down during'
636 ' the task.')
Garry Wangc1288cf2019-12-17 14:58:00 -0800637 # We want always stop servod after task to minimum the impact of bad
638 # servod process interfere other servods.(see crbug.com/1028665)
Garry Wang4c624bc2020-01-27 16:34:43 -0800639 try:
640 self.stop_servod()
641 except error.AutoservRunError as e:
642 logging.info("Failed to stop servod due to:\n%s\n"
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800643 "This error is forgiven.", str(e))
Garry Wangc1288cf2019-12-17 14:58:00 -0800644
Congbin Guoa1f9cba2018-07-03 11:36:59 -0700645 super(ServoHost, self).close()
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800646 # Mark closed.
647 self._closed = True
Congbin Guoa1f9cba2018-07-03 11:36:59 -0700648
649
Otabek Kasimovcc9738e2020-02-14 16:17:15 -0800650 def get_servo_state(self):
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700651 if self._servo_state is None:
Garry Wang11b5e872020-03-11 15:14:08 -0700652 return servo_constants.SERVO_STATE_UNKNOWN
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700653 return self._servo_state
Otabek Kasimovcc9738e2020-02-14 16:17:15 -0800654
655
Richard Barnetteea3e4602016-06-10 12:36:41 -0700656def make_servo_hostname(dut_hostname):
657 """Given a DUT's hostname, return the hostname of its servo.
658
659 @param dut_hostname: hostname of a DUT.
660
661 @return hostname of the DUT's servo.
662
663 """
664 host_parts = dut_hostname.split('.')
665 host_parts[0] = host_parts[0] + '-servo'
666 return '.'.join(host_parts)
667
668
669def servo_host_is_up(servo_hostname):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700670 """Given a servo host name, return if it's up or not.
Richard Barnetteea3e4602016-06-10 12:36:41 -0700671
672 @param servo_hostname: hostname of the servo host.
673
674 @return True if it's up, False otherwise
675 """
676 # Technically, this duplicates the SSH ping done early in the servo
677 # proxy initialization code. However, this ping ends in a couple
678 # seconds when if fails, rather than the 60 seconds it takes to decide
679 # that an SSH ping has timed out. Specifically, that timeout happens
680 # when our servo DNS name resolves, but there is no host at that IP.
681 logging.info('Pinging servo host at %s', servo_hostname)
682 ping_config = ping_runner.PingConfig(
683 servo_hostname, count=3,
684 ignore_result=True, ignore_status=True)
685 return ping_runner.PingRunner().ping(ping_config).received > 0
686
687
Richard Barnettee519dcd2016-08-15 17:37:17 -0700688def _map_afe_board_to_servo_board(afe_board):
689 """Map a board we get from the AFE to a servo appropriate value.
690
691 Many boards are identical to other boards for servo's purposes.
692 This function makes that mapping.
693
694 @param afe_board string board name received from AFE.
695 @return board we expect servo to have.
696
697 """
698 KNOWN_SUFFIXES = ['-freon', '_freon', '_moblab', '-cheets']
699 BOARD_MAP = {'gizmo': 'panther'}
700 mapped_board = afe_board
701 if afe_board in BOARD_MAP:
702 mapped_board = BOARD_MAP[afe_board]
703 else:
704 for suffix in KNOWN_SUFFIXES:
705 if afe_board.endswith(suffix):
706 mapped_board = afe_board[0:-len(suffix)]
707 break
708 if mapped_board != afe_board:
709 logging.info('Mapping AFE board=%s to %s', afe_board, mapped_board)
710 return mapped_board
711
712
Prathmesh Prabhub4810232018-09-07 13:24:08 -0700713def get_servo_args_for_host(dut_host):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700714 """Return servo data associated with a given DUT.
Richard Barnetteea3e4602016-06-10 12:36:41 -0700715
Richard Barnetteea3e4602016-06-10 12:36:41 -0700716 @param dut_host Instance of `Host` on which to find the servo
717 attributes.
Prathmesh Prabhuf605dd32018-08-28 17:09:04 -0700718 @return `servo_args` dict with host and an optional port.
Richard Barnetteea3e4602016-06-10 12:36:41 -0700719 """
Prathmesh Prabhucba44292018-08-28 17:44:45 -0700720 info = dut_host.host_info_store.get()
721 servo_args = {k: v for k, v in info.attributes.iteritems()
Garry Wang11b5e872020-03-11 15:14:08 -0700722 if k in servo_constants.SERVO_ATTR_KEYS}
Richard Barnetteea3e4602016-06-10 12:36:41 -0700723
Garry Wang11b5e872020-03-11 15:14:08 -0700724 if servo_constants.SERVO_PORT_ATTR in servo_args:
Prathmesh Prabhucba44292018-08-28 17:44:45 -0700725 try:
Garry Wang11b5e872020-03-11 15:14:08 -0700726 servo_args[servo_constants.SERVO_PORT_ATTR] = int(
727 servo_args[servo_constants.SERVO_PORT_ATTR])
Prathmesh Prabhucba44292018-08-28 17:44:45 -0700728 except ValueError:
729 logging.error('servo port is not an int: %s',
Garry Wang11b5e872020-03-11 15:14:08 -0700730 servo_args[servo_constants.SERVO_PORT_ATTR])
Prathmesh Prabhucba44292018-08-28 17:44:45 -0700731 # Reset servo_args because we don't want to use an invalid port.
Garry Wang11b5e872020-03-11 15:14:08 -0700732 servo_args.pop(servo_constants.SERVO_HOST_ATTR, None)
Prathmesh Prabhucba44292018-08-28 17:44:45 -0700733
734 if info.board:
Garry Wang11b5e872020-03-11 15:14:08 -0700735 servo_board = _map_afe_board_to_servo_board(info.board)
736 servo_args[servo_constants.SERVO_BOARD_ATTR] = servo_board
Nick Sanders2f3c9852018-10-24 12:10:24 -0700737 if info.model:
Garry Wang11b5e872020-03-11 15:14:08 -0700738 servo_args[servo_constants.SERVO_MODEL_ATTR] = info.model
739 return servo_args if servo_constants.SERVO_HOST_ATTR in servo_args else None
Richard Barnetteea3e4602016-06-10 12:36:41 -0700740
741
Prathmesh Prabhuefb1b482018-08-28 17:15:05 -0700742def _tweak_args_for_ssp_moblab(servo_args):
Garry Wang11b5e872020-03-11 15:14:08 -0700743 if (servo_args[servo_constants.SERVO_HOST_ATTR]
744 in ['localhost', '127.0.0.1']):
745 servo_args[servo_constants.SERVO_HOST_ATTR] = _CONFIG.get_config_value(
Prathmesh Prabhuefb1b482018-08-28 17:15:05 -0700746 'SSP', 'host_container_ip', type=str, default=None)
747
748
Dan Shi023aae32016-05-25 11:13:01 -0700749def create_servo_host(dut, servo_args, try_lab_servo=False,
Gregory Nisbetde13e2a2019-12-09 22:44:00 -0800750 try_servo_repair=False, dut_host_info=None):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700751 """Create a ServoHost object for a given DUT, if appropriate.
Dan Shi4d478522014-02-14 13:46:32 -0800752
Richard Barnette9a26ad62016-06-10 12:03:08 -0700753 This function attempts to create and verify or repair a `ServoHost`
754 object for a servo connected to the given `dut`, subject to various
755 constraints imposed by the parameters:
756 * When the `servo_args` parameter is not `None`, a servo
757 host must be created, and must be checked with `repair()`.
758 * Otherwise, if a servo exists in the lab and `try_lab_servo` is
759 true:
760 * If `try_servo_repair` is true, then create a servo host and
761 check it with `repair()`.
762 * Otherwise, if the servo responds to `ping` then create a
763 servo host and check it with `verify()`.
Fang Denge545abb2014-12-30 18:43:47 -0800764
Richard Barnette9a26ad62016-06-10 12:03:08 -0700765 In cases where `servo_args` was not `None`, repair failure
766 exceptions are passed back to the caller; otherwise, exceptions
Richard Barnette07c2e1d2016-10-26 14:24:28 -0700767 are logged and then discarded. Note that this only happens in cases
768 where we're called from a test (not special task) control file that
769 has an explicit dependency on servo. In that case, we require that
770 repair not write to `status.log`, so as to avoid polluting test
771 results.
772
773 TODO(jrbarnette): The special handling for servo in test control
774 files is a thorn in my flesh; I dearly hope to see it cut out before
775 my retirement.
Richard Barnette9a26ad62016-06-10 12:03:08 -0700776
777 Parameters for a servo host consist of a host name, port number, and
778 DUT board, and are determined from one of these sources, in order of
779 priority:
Richard Barnetteea3e4602016-06-10 12:36:41 -0700780 * Servo attributes from the `dut` parameter take precedence over
781 all other sources of information.
782 * If a DNS entry for the servo based on the DUT hostname exists in
783 the CrOS lab network, that hostname is used with the default
Richard Barnette9a26ad62016-06-10 12:03:08 -0700784 port and the DUT's board.
Richard Barnetteea3e4602016-06-10 12:36:41 -0700785 * If no other options are found, the parameters will be taken
Richard Barnette9a26ad62016-06-10 12:03:08 -0700786 from the `servo_args` dict passed in from the caller.
Richard Barnetteea3e4602016-06-10 12:36:41 -0700787
788 @param dut An instance of `Host` from which to take
789 servo parameters (if available).
790 @param servo_args A dictionary with servo parameters to use if
791 they can't be found from `dut`. If this
792 argument is supplied, unrepaired exceptions
793 from `verify()` will be passed back to the
794 caller.
795 @param try_lab_servo If not true, servo host creation will be
796 skipped unless otherwise required by the
797 caller.
Richard Barnette9a26ad62016-06-10 12:03:08 -0700798 @param try_servo_repair If true, check a servo host with
799 `repair()` instead of `verify()`.
Dan Shi4d478522014-02-14 13:46:32 -0800800
801 @returns: A ServoHost object or None. See comments above.
802
803 """
Richard Barnette07c2e1d2016-10-26 14:24:28 -0700804 servo_dependency = servo_args is not None
Richard Barnette07c2e1d2016-10-26 14:24:28 -0700805 if dut is not None and (try_lab_servo or servo_dependency):
Prathmesh Prabhub4810232018-09-07 13:24:08 -0700806 servo_args_override = get_servo_args_for_host(dut)
Richard Barnetteea3e4602016-06-10 12:36:41 -0700807 if servo_args_override is not None:
Prathmesh Prabhuefb1b482018-08-28 17:15:05 -0700808 if utils.in_moblab_ssp():
809 _tweak_args_for_ssp_moblab(servo_args_override)
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -0700810 logging.debug(
811 'Overriding provided servo_args (%s) with arguments'
812 ' determined from the host (%s)',
813 servo_args,
814 servo_args_override,
815 )
Richard Barnetteea3e4602016-06-10 12:36:41 -0700816 servo_args = servo_args_override
Prathmesh Prabhucba44292018-08-28 17:44:45 -0700817
Richard Barnetteea3e4602016-06-10 12:36:41 -0700818 if servo_args is None:
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -0700819 logging.debug('No servo_args provided, and failed to find overrides.')
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700820 if try_lab_servo or servo_dependency:
Garry Wang11b5e872020-03-11 15:14:08 -0700821 return None, servo_constants.SERVO_STATE_NOT_CONNECTED
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700822 else:
823 # For regular test case which not required the servo
824 return None, None
825
Garry Wang11b5e872020-03-11 15:14:08 -0700826 servo_hostname = servo_args.get(servo_constants.SERVO_HOST_ATTR)
827 servo_port = servo_args.get(servo_constants.SERVO_PORT_ATTR)
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700828 if not _is_servo_host_information_exist(servo_hostname, servo_port):
829 logging.debug(
830 'Servo connection info missed hostname: %s , port: %s',
831 servo_hostname, servo_port)
Garry Wang11b5e872020-03-11 15:14:08 -0700832 return None, servo_constants.SERVO_STATE_NOT_CONNECTED
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700833 if not is_servo_host_information_valid(servo_hostname, servo_port):
834 logging.debug(
835 'Servo connection info is incorrect hostname: %s , port: %s',
836 servo_hostname, servo_port)
Garry Wang11b5e872020-03-11 15:14:08 -0700837 return None, servo_constants.SERVO_STATE_WRONG_CONFIG
Richard Barnette07c2e1d2016-10-26 14:24:28 -0700838 if (not servo_dependency and not try_servo_repair and
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700839 not servo_host_is_up(servo_hostname)):
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -0700840 logging.debug('ServoHost is not up.')
Garry Wang11b5e872020-03-11 15:14:08 -0700841 return None, servo_constants.SERVO_STATE_BROKEN
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -0700842
Garry Wangebc015b2019-06-06 17:45:06 -0700843 newhost = ServoHost(**servo_args)
Garry Wangcdd27b22020-01-13 14:59:11 -0800844 try:
845 newhost.restart_servod(quick_startup=True)
846 except error.AutoservSSHTimeout:
847 logging.warning("Restart servod failed due ssh connection "
848 "to servohost timed out. This error is forgiven"
849 " here, we will retry in servo repair process.")
850 except error.AutoservRunError as e:
851 logging.warning("Restart servod failed due to:\n%s\n"
852 "This error is forgiven here, we will retry"
853 " in servo repair process.", str(e))
Garry Wangebc015b2019-06-06 17:45:06 -0700854
Gregory Nisbetde13e2a2019-12-09 22:44:00 -0800855 # TODO(gregorynisbet): Clean all of this up.
856 logging.debug('create_servo_host: attempt to set info store on '
857 'servo host')
858 try:
859 if dut_host_info is None:
860 logging.debug('create_servo_host: dut_host_info is '
861 'None, skipping')
862 else:
863 newhost.set_dut_host_info(dut_host_info)
864 logging.debug('create_servo_host: successfully set info '
865 'store')
866 except Exception:
867 logging.error("create_servo_host: (%s)", traceback.format_exc())
868
Richard Barnette9a26ad62016-06-10 12:03:08 -0700869 # Note that the logic of repair() includes everything done
870 # by verify(). It's sufficient to call one or the other;
871 # we don't need both.
Richard Barnette07c2e1d2016-10-26 14:24:28 -0700872 if servo_dependency:
873 newhost.repair(silent=True)
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700874 return newhost, newhost.get_servo_state()
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -0700875
876 if try_servo_repair:
877 try:
878 newhost.repair()
879 except Exception:
880 logging.exception('servo repair failed for %s', newhost.hostname)
Richard Barnette9a26ad62016-06-10 12:03:08 -0700881 else:
882 try:
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -0700883 newhost.verify()
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700884 except Exception:
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -0700885 logging.exception('servo verify failed for %s', newhost.hostname)
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700886 return newhost, newhost.get_servo_state()
Otabek Kasimov7267a7a2020-03-04 11:18:45 -0800887
888
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700889def _is_servo_host_information_exist(hostname, port):
Otabek Kasimov7267a7a2020-03-04 11:18:45 -0800890 if hostname is None or len(hostname.strip()) == 0:
891 return False
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700892 if port is None:
Otabek Kasimov7267a7a2020-03-04 11:18:45 -0800893 return False
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700894 if not type(port) is int:
895 try:
896 int(port)
897 except ValueError:
898 return False
899
Otabek Kasimov7267a7a2020-03-04 11:18:45 -0800900 return True
901
902
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700903def is_servo_host_information_valid(hostname, port):
904 if not _is_servo_host_information_exist(hostname, port):
Otabek Kasimov7267a7a2020-03-04 11:18:45 -0800905 return False
906 # checking range and correct of the port
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700907 port_int = int(port)
Otabek Kasimov7267a7a2020-03-04 11:18:45 -0800908 if port_int < 1 or port_int > 65000:
909 return False
910 # we expecting host contain only latters, digits and '-' or '_'
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700911 if not re.match('[a-zA-Z0-9-_\.]*$', hostname) or len(hostname) < 5:
Otabek Kasimov7267a7a2020-03-04 11:18:45 -0800912 return False
913 return True