blob: e17a16c0935978c9d0ba0a1ab04e460ee048d366 [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
Richard Barnette9a26ad62016-06-10 12:03:08 -0700170 def connect_servo(self):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700171 """Establish a connection to the servod server on this host.
Richard Barnette9a26ad62016-06-10 12:03:08 -0700172
173 Initializes `self._servo` and then verifies that all network
174 connections are working. This will create an ssh tunnel if
175 it's required.
176
177 As a side effect of testing the connection, all signals on the
178 target servo are reset to default values, and the USB stick is
179 set to the neutral (off) position.
180 """
Kevin Cheng643ce8a2016-09-15 15:42:12 -0700181 servo_obj = servo.Servo(servo_host=self, servo_serial=self.servo_serial)
Kuang-che Wu05763f52019-08-30 16:48:21 +0800182 self._servo = servo_obj
Richard Barnette9a26ad62016-06-10 12:03:08 -0700183 timeout, _ = retry.timeout(
184 servo_obj.initialize_dut,
185 timeout_sec=self.INITIALIZE_SERVO_TIMEOUT_SECS)
186 if timeout:
187 raise hosts.AutoservVerifyError(
188 'Servo initialize timed out.')
Richard Barnette9a26ad62016-06-10 12:03:08 -0700189
190
191 def disconnect_servo(self):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700192 """Disconnect our servo if it exists.
Richard Barnette9a26ad62016-06-10 12:03:08 -0700193
194 If we've previously successfully connected to our servo,
195 disconnect any established ssh tunnel, and set `self._servo`
196 back to `None`.
197 """
198 if self._servo:
199 # N.B. This call is safe even without a tunnel:
200 # rpc_server_tracker.disconnect() silently ignores
201 # unknown ports.
202 self.rpc_server_tracker.disconnect(self.servo_port)
203 self._servo = None
Fang Deng5d518f42013-08-02 14:04:32 -0700204
Garry Wangc1288cf2019-12-17 14:58:00 -0800205
Wai-Hong Tam3a8a2552019-11-19 14:28:04 +0800206 def _create_servod_server_proxy(self):
207 """Create a proxy that can be used to communicate with servod server.
Fang Deng5d518f42013-08-02 14:04:32 -0700208
209 @returns: An xmlrpclib.ServerProxy that is connected to the servod
210 server on the host.
Fang Deng5d518f42013-08-02 14:04:32 -0700211 """
Garry Wang11b5e872020-03-11 15:14:08 -0700212 if (servo_constants.ENABLE_SSH_TUNNEL_FOR_SERVO
213 and not self.is_localhost()):
Richard Barnette9a26ad62016-06-10 12:03:08 -0700214 return self.rpc_server_tracker.xmlrpc_connect(
215 None, self.servo_port,
216 ready_test_name=self.SERVO_READY_METHOD,
Allen Li2b1a8992018-11-27 14:17:18 -0800217 timeout_seconds=60,
Allen Li556f4532018-12-03 18:11:23 -0800218 request_timeout_seconds=3600)
Richard Barnette9a26ad62016-06-10 12:03:08 -0700219 else:
220 remote = 'http://%s:%s' % (self.hostname, self.servo_port)
221 return xmlrpclib.ServerProxy(remote)
Fang Deng5d518f42013-08-02 14:04:32 -0700222
223
Wai-Hong Tam3a8a2552019-11-19 14:28:04 +0800224 def get_servod_server_proxy(self):
225 """Return a cached proxy if exists; otherwise, create a new one.
226
227 @returns: An xmlrpclib.ServerProxy that is connected to the servod
228 server on the host.
229 """
230 # Single-threaded execution, no race
231 if self._servod_server_proxy is None:
232 self._servod_server_proxy = self._create_servod_server_proxy()
233 return self._servod_server_proxy
234
235
Richard Barnette1edbb162016-11-01 11:47:50 -0700236 def verify(self, silent=False):
237 """Update the servo host and verify it's in a good state.
238
239 @param silent If true, suppress logging in `status.log`.
240 """
Richard Barnetteabbdc252018-07-26 16:57:42 -0700241 message = 'Beginning verify for servo host %s port %s serial %s'
242 message %= (self.hostname, self.servo_port, self.servo_serial)
243 self.record('INFO', None, None, message)
Richard Barnette9a26ad62016-06-10 12:03:08 -0700244 try:
Richard Barnette1edbb162016-11-01 11:47:50 -0700245 self._repair_strategy.verify(self, silent)
Garry Wang11b5e872020-03-11 15:14:08 -0700246 self._servo_state = servo_constants.SERVO_STATE_WORKING
247 self.record('INFO', None, None,
248 'ServoHost verify set servo_state as WORKING')
Garry Wang63b8c382020-03-11 22:28:40 -0700249 except Exception as e:
Garry Wang11b5e872020-03-11 15:14:08 -0700250 self._servo_state = servo_constants.SERVO_STATE_BROKEN
251 self.record('INFO', None, None,
252 'ServoHost verify set servo_state as BROKEN')
Garry Wang63b8c382020-03-11 22:28:40 -0700253 if self._is_critical_error(e):
254 raise
Fang Deng5d518f42013-08-02 14:04:32 -0700255
256
Richard Barnette1edbb162016-11-01 11:47:50 -0700257 def repair(self, silent=False):
258 """Attempt to repair servo host.
259
260 @param silent If true, suppress logging in `status.log`.
261 """
Richard Barnetteabbdc252018-07-26 16:57:42 -0700262 message = 'Beginning repair for servo host %s port %s serial %s'
263 message %= (self.hostname, self.servo_port, self.servo_serial)
264 self.record('INFO', None, None, message)
Richard Barnette9a26ad62016-06-10 12:03:08 -0700265 try:
Richard Barnette1edbb162016-11-01 11:47:50 -0700266 self._repair_strategy.repair(self, silent)
Garry Wang11b5e872020-03-11 15:14:08 -0700267 self._servo_state = servo_constants.SERVO_STATE_WORKING
268 self.record('INFO', None, None,
269 'ServoHost repair set servo_state as WORKING')
Garry Wang464ff1e2019-07-18 17:20:34 -0700270 # If target is a labstation then try to withdraw any existing
271 # reboot request created by this servo because it passed repair.
272 if self.is_labstation():
273 self.withdraw_reboot_request()
Garry Wang63b8c382020-03-11 22:28:40 -0700274 except Exception as e:
Garry Wang11b5e872020-03-11 15:14:08 -0700275 self._servo_state = servo_constants.SERVO_STATE_BROKEN
276 self.record('INFO', None, None,
277 'ServoHost repair set servo_state as BROKEN')
Garry Wang63b8c382020-03-11 22:28:40 -0700278 if self._is_critical_error(e):
279 self.disconnect_servo()
280 self.stop_servod()
281 raise
282
283
284 def _is_critical_error(self, error):
285 if (isinstance(error, hosts.AutoservVerifyDependencyError)
286 and not error.is_critical()):
287 logging.warning('Non-critical verify failure(s) detected during'
288 ' verify/repair servo, servo connection will'
289 ' still up but may not fully functional.'
290 ' Some repair actions and servo depended'
291 ' tests may not run.')
292 return False
293 logging.info('Critical verify failure(s) detected during repair/verify'
294 ' servo. Disconnecting servo and stop servod, all repair '
295 'action and tests that depends on servo will not run.')
296 return True
Fang Deng5d518f42013-08-02 14:04:32 -0700297
298
Dan Shi4d478522014-02-14 13:46:32 -0800299 def get_servo(self):
300 """Get the cached servo.Servo object.
Fang Deng5d518f42013-08-02 14:04:32 -0700301
Dan Shi4d478522014-02-14 13:46:32 -0800302 @return: a servo.Servo object.
Dana Goyette353d1d92019-06-27 10:43:59 -0700303 @rtype: autotest_lib.server.cros.servo.servo.Servo
Fang Deng5d518f42013-08-02 14:04:32 -0700304 """
Dan Shi4d478522014-02-14 13:46:32 -0800305 return self._servo
306
307
Garry Wang79e9af62019-06-12 15:19:19 -0700308 def request_reboot(self):
309 """Request servohost to be rebooted when it's safe to by touch a file.
310 """
311 logging.debug('Request to reboot servohost %s has been created by '
Garry Wang464ff1e2019-07-18 17:20:34 -0700312 'servo with port # %s', self.hostname, self.servo_port)
Garry Wang79e9af62019-06-12 15:19:19 -0700313 self.run('touch %s' % self._reboot_file, ignore_status=True)
314
315
Garry Wang464ff1e2019-07-18 17:20:34 -0700316 def withdraw_reboot_request(self):
317 """Withdraw a servohost reboot request if exists by remove the flag
318 file.
319 """
320 logging.debug('Withdrawing request to reboot servohost %s that created'
321 ' by servo with port # %s if exists.',
322 self.hostname, self.servo_port)
323 self.run('rm -f %s' % self._reboot_file, ignore_status=True)
324
325
Garry Wangc1288cf2019-12-17 14:58:00 -0800326 def start_servod(self, quick_startup=False):
327 """Start the servod process on servohost.
328 """
Garry Wang2ac15ee2019-12-30 19:03:02 -0800329 # Skip if running on the localhost.(crbug.com/1038168)
330 if self.is_localhost():
331 logging.debug("Servohost is a localhost, skipping start servod.")
332 return
333
334 cmd = 'start servod'
Garry Wangc1288cf2019-12-17 14:58:00 -0800335 if self.servo_board:
Garry Wang2ac15ee2019-12-30 19:03:02 -0800336 cmd += ' BOARD=%s' % self.servo_board
Garry Wangc1288cf2019-12-17 14:58:00 -0800337 if self.servo_model:
338 cmd += ' MODEL=%s' % self.servo_model
Garry Wangc1288cf2019-12-17 14:58:00 -0800339 else:
Garry Wang2ac15ee2019-12-30 19:03:02 -0800340 logging.warning('Board for DUT is unknown; starting servod'
341 ' assuming a pre-configured board.')
342
343 cmd += ' PORT=%d' % self.servo_port
344 if self.servo_serial:
345 cmd += ' SERIAL=%s' % self.servo_serial
Garry Wangd7367482020-02-27 13:52:40 -0800346
347 # Start servod with dual_v4 if the DUT/servo from designated pools.
348 dut_host_info = self.get_dut_host_info()
349 if dut_host_info:
Garry Wang11b5e872020-03-11 15:14:08 -0700350 if bool(dut_host_info.pools &
351 servo_constants.POOLS_SUPPORT_DUAL_V4):
Garry Wangd7367482020-02-27 13:52:40 -0800352 logging.debug('The DUT is detected in following designated'
353 ' pools %s,starting servod with DUAL_V4 option.',
Garry Wang11b5e872020-03-11 15:14:08 -0700354 servo_constants.POOLS_SUPPORT_DUAL_V4)
Garry Wangd7367482020-02-27 13:52:40 -0800355 cmd += ' DUAL_V4=1'
356
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800357 # Remove the symbolic links from the logs. This helps ensure that
358 # a failed servod instantiation does not cause us to grab old logs
359 # by mistake.
360 self.remove_latest_log_symlinks()
Garry Wangcdd27b22020-01-13 14:59:11 -0800361 self.run(cmd, timeout=60)
Garry Wangc1288cf2019-12-17 14:58:00 -0800362
363 # There's a lag between when `start servod` completes and when
364 # the _ServodConnectionVerifier trigger can actually succeed.
365 # The call to time.sleep() below gives time to make sure that
366 # the trigger won't fail after we return.
367
368 # Normally servod on servo_v3 and labstation take ~10 seconds to ready,
369 # But in the rare case all servo on a labstation are in heavy use they
370 # may take ~30 seconds. So the timeout value will double these value,
371 # and we'll try quick start up when first time initialize servohost,
372 # and use standard start up timeout in repair.
373 if quick_startup:
Garry Wang11b5e872020-03-11 15:14:08 -0700374 timeout = servo_constants.SERVOD_QUICK_STARTUP_TIMEOUT
Garry Wangc1288cf2019-12-17 14:58:00 -0800375 else:
Garry Wang11b5e872020-03-11 15:14:08 -0700376 timeout = servo_constants.SERVOD_STARTUP_TIMEOUT
Garry Wangc1288cf2019-12-17 14:58:00 -0800377 logging.debug('Wait %s seconds for servod process fully up.', timeout)
378 time.sleep(timeout)
379
380
381 def stop_servod(self):
382 """Stop the servod process on servohost.
383 """
Garry Wang2ac15ee2019-12-30 19:03:02 -0800384 # Skip if running on the localhost.(crbug.com/1038168)
385 if self.is_localhost():
386 logging.debug("Servohost is a localhost, skipping stop servod.")
387 return
388
Garry Wangc1288cf2019-12-17 14:58:00 -0800389 logging.debug('Stopping servod on port %s', self.servo_port)
Garry Wangcdd27b22020-01-13 14:59:11 -0800390 self.run('stop servod PORT=%d' % self.servo_port,
391 timeout=60, ignore_status=True)
Garry Wangc1288cf2019-12-17 14:58:00 -0800392 logging.debug('Wait %s seconds for servod process fully teardown.',
Garry Wang11b5e872020-03-11 15:14:08 -0700393 servo_constants.SERVOD_TEARDOWN_TIMEOUT)
394 time.sleep(servo_constants.SERVOD_TEARDOWN_TIMEOUT)
Garry Wangc1288cf2019-12-17 14:58:00 -0800395
396
397 def restart_servod(self, quick_startup=False):
398 """Restart the servod process on servohost.
399 """
400 self.stop_servod()
401 self.start_servod(quick_startup)
402
Garry Wang11b5e872020-03-11 15:14:08 -0700403
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800404 def _extract_compressed_logs(self, logdir, relevant_files):
405 """Decompress servod logs in |logdir|.
406
407 @param logdir: directory containing compressed servod logs.
408 @param relevant_files: list of files in |logdir| to consider.
409
410 @returns: tuple, (tarfiles, files) where
411 tarfiles: list of the compressed filenames that have been
412 extracted and deleted
413 files: list of the uncompressed files that were generated
414 """
415 # For all tar-files, first extract them to the directory, and
416 # then let the common flow handle them.
417 tarfiles = [cf for cf in relevant_files if
418 cf.endswith(self.COMPRESSION_SUFFIX)]
419 files = []
420 for f in tarfiles:
421 norm_name = os.path.basename(f)[:-len(self.COMPRESSION_SUFFIX)]
422 with tarfile.open(f) as tf:
423 # Each tarfile has only one member, as
424 # that's the compressed log.
425 member = tf.members[0]
426 # Manipulate so that it only extracts the basename, and not
427 # the directories etc.
428 member.name = norm_name
429 files.append(os.path.join(logdir, member.name))
430 tf.extract(member, logdir)
431 # File has been extracted: remove the compressed file.
432 os.remove(f)
433 return tarfiles, files
434
435 def _extract_mcu_logs(self, log_subdir):
436 """Extract MCU (EC, Cr50, etc) console output from servod debug logs.
437
438 Using the MCU_EXTRACTOR regex (above) extract and split out MCU console
439 lines from the logs to generate invidiual console logs e.g. after
440 this method, you can find an ec.txt and servo_v4.txt in |log_dir| if
441 those MCUs had any console input/output.
442
443 @param log_subdir: directory with log.DEBUG.txt main servod debug logs.
444 """
445 # Extract the MCU for each one. The MCU logs are only in the .DEBUG
446 # files
447 mcu_lines_file = os.path.join(log_subdir, 'log.DEBUG.txt')
448 if not os.path.exists(mcu_lines_file):
449 logging.info('No DEBUG logs found to extract MCU logs from.')
450 return
451 mcu_files = {}
452 mcu_file_template = '%s.txt'
453 with open(mcu_lines_file, 'r') as f:
454 for line in f:
455 match = self.MCU_EXTRACTOR.match(line)
456 if match:
457 mcu = match.group(self.MCU_GROUP).lower()
458 line = match.group(self.LINE_GROUP)
459 if mcu not in mcu_files:
460 mcu_file = os.path.join(log_subdir,
461 mcu_file_template % mcu)
462 mcu_files[mcu] = open(mcu_file, 'a')
463 fd = mcu_files[mcu]
464 fd.write(line + '\n')
465 for f in mcu_files:
466 mcu_files[f].close()
467
468
469 def remove_latest_log_symlinks(self):
470 """Remove the conveninence symlinks 'latest' servod logs."""
471 symlink_wildcard = '%s/latest*' % self.remote_log_dir
472 cmd = 'rm ' + symlink_wildcard
473 self.run(cmd, stderr_tee=None, ignore_status=True)
474
475 def grab_logs(self, outdir):
476 """Retrieve logs from servo_host to |outdir|/servod_{port}.{ts}/.
477
478 This method first collects all logs on the servo_host side pertaining
479 to this servod instance (port, instatiation). It glues them together
480 into combined log.[level].txt files and extracts all available MCU
481 console I/O from the logs into individual files e.g. servo_v4.txt
482
483 All the output can be found in a directory inside |outdir| that
484 this generates based on |LOG_DIR|, the servod port, and the instance
485 timestamp on the servo_host side.
486
487 @param outdir: directory to create a subdirectory into to place the
488 servod logs into.
489 """
490 # First, extract the timestamp. This cmd gives the real filename of
491 # the latest aka current log file.
492 cmd = ('if [ -f %(dir)s/latest.DEBUG ];'
493 'then realpath %(dir)s/latest.DEBUG;'
494 'elif [ -f %(dir)s/latest ];'
495 'then realpath %(dir)s/latest;'
496 'else exit %(code)d;'
497 'fi' % {'dir': self.remote_log_dir,
498 'code': self.NO_SYMLINKS_CODE})
499 res = self.run(cmd, stderr_tee=None, ignore_status=True)
500 if res.exit_status != 0:
501 if res.exit_status == self.NO_SYMLINKS_CODE:
502 logging.warning('servod log latest symlinks not found. '
503 'This is likely due to an error starting up '
504 'servod. Ignoring..')
505 else:
506 logging.warning('Failed to find servod logs on servo host.')
507 logging.warning(res.stderr.strip())
508 return
509 fname = os.path.basename(res.stdout.strip())
510 # From the fname, ought to extract the timestamp using the TS_EXTRACTOR
Ruben Rodriguez Buchillone9aa2b02020-03-04 12:14:28 -0800511 ts_match = self.TS_EXTRACTOR.match(fname)
512 if not ts_match:
513 logging.warning('Failed to extract timestamp from servod log file '
514 '%r. Skipping. The servo host is using outdated '
515 'servod logging and needs to be updated.', fname)
516 return
517 instance_ts = ts_match.group(self.TS_GROUP)
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800518 # Create the local results log dir.
519 log_dir = os.path.join(outdir, '%s_%s.%s' % (self.LOG_DIR,
520 str(self.servo_port),
521 instance_ts))
522 logging.info('Saving servod logs to %s.', log_dir)
523 os.mkdir(log_dir)
524 # Now, get all files with that timestamp.
525 cmd = 'find %s -maxdepth 1 -name "log.%s*"' % (self.remote_log_dir,
526 instance_ts)
527 res = self.run(cmd, stderr_tee=None, ignore_status=True)
528 files = res.stdout.strip().split()
529 try:
530 self.get_file(files, log_dir, try_rsync=False)
531
532 except error.AutoservRunError as e:
533 result = e.result_obj
534 if result.exit_status != 0:
535 stderr = result.stderr.strip()
536 logging.warning("Couldn't retrieve servod logs. Ignoring: %s",
537 stderr or '\n%s' % result)
538 return
539 local_files = [os.path.join(log_dir, f) for f in os.listdir(log_dir)]
540 # TODO(crrev.com/c/1793030): remove no-level case once CL is pushed
541 for level_name in ('DEBUG', 'INFO', 'WARNING', ''):
542 # Create the joint files for each loglevel. i.e log.DEBUG
543 joint_file = self.JOINT_LOG_PREFIX
544 if level_name:
545 joint_file = '%s.%s' % (self.JOINT_LOG_PREFIX, level_name)
546 # This helps with some online tools to avoid complaints about an
547 # unknown filetype.
548 joint_file = joint_file + '.txt'
549 joint_path = os.path.join(log_dir, joint_file)
550 files = [f for f in local_files if level_name in f]
551 if not files:
552 # TODO(crrev.com/c/1793030): remove no-level case once CL
553 # is pushed
554 continue
555 # Extract compressed logs if any.
556 compressed, extracted = self._extract_compressed_logs(log_dir,
557 files)
558 files = list(set(files) - set(compressed))
559 files.extend(extracted)
560 # Need to sort. As they all share the same timestamp, and
561 # loglevel, the index itself is sufficient. The highest index
562 # is the oldest file, therefore we need a descending sort.
563 def sortkey(f, level=level_name):
564 """Custom sortkey to sort based on rotation number int."""
565 if f.endswith(level_name): return 0
566 return int(f.split('.')[-1])
567
568 files.sort(reverse=True, key=sortkey)
569 # Just rename the first file rather than building from scratch.
570 os.rename(files[0], joint_path)
571 with open(joint_path, 'a') as joint_f:
572 for logfile in files[1:]:
573 # Transfer the file to the joint file line by line.
574 with open(logfile, 'r') as log_f:
575 for line in log_f:
576 joint_f.write(line)
577 # File has been written over. Delete safely.
578 os.remove(logfile)
579 # Need to remove all files form |local_files| so we don't
580 # analyze them again.
581 local_files = list(set(local_files) - set(files) - set(compressed))
582 # Lastly, extract MCU logs from the joint logs.
583 self._extract_mcu_logs(log_dir)
584
Garry Wangc1288cf2019-12-17 14:58:00 -0800585
Garry Wang79e9af62019-06-12 15:19:19 -0700586 def _lock(self):
587 """lock servohost by touching a file.
588 """
589 logging.debug('Locking servohost %s by touching %s file',
590 self.hostname, self._lock_file)
591 self.run('touch %s' % self._lock_file, ignore_status=True)
Garry Wang7c00b0f2019-06-25 17:28:17 -0700592 self._is_locked = True
Garry Wang79e9af62019-06-12 15:19:19 -0700593
594
595 def _unlock(self):
596 """Unlock servohost by removing the lock file.
597 """
598 logging.debug('Unlocking servohost by removing %s file',
599 self._lock_file)
600 self.run('rm %s' % self._lock_file, ignore_status=True)
Garry Wang7c00b0f2019-06-25 17:28:17 -0700601 self._is_locked = False
Garry Wang79e9af62019-06-12 15:19:19 -0700602
603
Congbin Guoa1f9cba2018-07-03 11:36:59 -0700604 def close(self):
Congbin Guofc3b8962019-03-22 17:38:46 -0700605 """Close the associated servo and the host object."""
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800606 if self._closed:
607 logging.debug('ServoHost is already closed.')
608 return
Congbin Guoa1f9cba2018-07-03 11:36:59 -0700609 if self._servo:
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800610 outdir = None if not self.job else self.job.resultdir
Congbin Guo2e5e2a22018-07-27 10:32:48 -0700611 # In some cases when we run as lab-tools, the job object is None.
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800612 self._servo.close(outdir)
613
614 if self.job and not self.is_localhost():
615 # Grab all logs from this servod instance before stopping servod.
616 # TODO(crbug.com/1011516): once enabled, remove the check against
617 # localhost and instead check against log-rotiation enablement.
618 try:
619 self.grab_logs(self.job.resultdir)
620 except error.AutoservRunError as e:
621 logging.info('Failed to grab servo logs due to: %s. '
622 'This error is forgiven.', str(e))
Congbin Guoa1f9cba2018-07-03 11:36:59 -0700623
Garry Wang7c00b0f2019-06-25 17:28:17 -0700624 if self._is_locked:
625 # Remove the lock if the servohost has been locked.
Garry Wang79e9af62019-06-12 15:19:19 -0700626 try:
627 self._unlock()
628 except error.AutoservSSHTimeout:
629 logging.error('Unlock servohost failed due to ssh timeout.'
630 ' It may caused by servohost went down during'
631 ' the task.')
Garry Wangc1288cf2019-12-17 14:58:00 -0800632 # We want always stop servod after task to minimum the impact of bad
633 # servod process interfere other servods.(see crbug.com/1028665)
Garry Wang4c624bc2020-01-27 16:34:43 -0800634 try:
635 self.stop_servod()
636 except error.AutoservRunError as e:
637 logging.info("Failed to stop servod due to:\n%s\n"
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800638 "This error is forgiven.", str(e))
Garry Wangc1288cf2019-12-17 14:58:00 -0800639
Congbin Guoa1f9cba2018-07-03 11:36:59 -0700640 super(ServoHost, self).close()
Ruben Rodriguez Buchillon93084d02020-01-21 15:17:36 -0800641 # Mark closed.
642 self._closed = True
Congbin Guoa1f9cba2018-07-03 11:36:59 -0700643
644
Otabek Kasimovcc9738e2020-02-14 16:17:15 -0800645 def get_servo_state(self):
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700646 if self._servo_state is None:
Garry Wang11b5e872020-03-11 15:14:08 -0700647 return servo_constants.SERVO_STATE_UNKNOWN
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700648 return self._servo_state
Otabek Kasimovcc9738e2020-02-14 16:17:15 -0800649
650
Richard Barnetteea3e4602016-06-10 12:36:41 -0700651def make_servo_hostname(dut_hostname):
652 """Given a DUT's hostname, return the hostname of its servo.
653
654 @param dut_hostname: hostname of a DUT.
655
656 @return hostname of the DUT's servo.
657
658 """
659 host_parts = dut_hostname.split('.')
660 host_parts[0] = host_parts[0] + '-servo'
661 return '.'.join(host_parts)
662
663
664def servo_host_is_up(servo_hostname):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700665 """Given a servo host name, return if it's up or not.
Richard Barnetteea3e4602016-06-10 12:36:41 -0700666
667 @param servo_hostname: hostname of the servo host.
668
669 @return True if it's up, False otherwise
670 """
671 # Technically, this duplicates the SSH ping done early in the servo
672 # proxy initialization code. However, this ping ends in a couple
673 # seconds when if fails, rather than the 60 seconds it takes to decide
674 # that an SSH ping has timed out. Specifically, that timeout happens
675 # when our servo DNS name resolves, but there is no host at that IP.
676 logging.info('Pinging servo host at %s', servo_hostname)
677 ping_config = ping_runner.PingConfig(
678 servo_hostname, count=3,
679 ignore_result=True, ignore_status=True)
680 return ping_runner.PingRunner().ping(ping_config).received > 0
681
682
Richard Barnettee519dcd2016-08-15 17:37:17 -0700683def _map_afe_board_to_servo_board(afe_board):
684 """Map a board we get from the AFE to a servo appropriate value.
685
686 Many boards are identical to other boards for servo's purposes.
687 This function makes that mapping.
688
689 @param afe_board string board name received from AFE.
690 @return board we expect servo to have.
691
692 """
693 KNOWN_SUFFIXES = ['-freon', '_freon', '_moblab', '-cheets']
694 BOARD_MAP = {'gizmo': 'panther'}
695 mapped_board = afe_board
696 if afe_board in BOARD_MAP:
697 mapped_board = BOARD_MAP[afe_board]
698 else:
699 for suffix in KNOWN_SUFFIXES:
700 if afe_board.endswith(suffix):
701 mapped_board = afe_board[0:-len(suffix)]
702 break
703 if mapped_board != afe_board:
704 logging.info('Mapping AFE board=%s to %s', afe_board, mapped_board)
705 return mapped_board
706
707
Prathmesh Prabhub4810232018-09-07 13:24:08 -0700708def get_servo_args_for_host(dut_host):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700709 """Return servo data associated with a given DUT.
Richard Barnetteea3e4602016-06-10 12:36:41 -0700710
Richard Barnetteea3e4602016-06-10 12:36:41 -0700711 @param dut_host Instance of `Host` on which to find the servo
712 attributes.
Prathmesh Prabhuf605dd32018-08-28 17:09:04 -0700713 @return `servo_args` dict with host and an optional port.
Richard Barnetteea3e4602016-06-10 12:36:41 -0700714 """
Prathmesh Prabhucba44292018-08-28 17:44:45 -0700715 info = dut_host.host_info_store.get()
716 servo_args = {k: v for k, v in info.attributes.iteritems()
Garry Wang11b5e872020-03-11 15:14:08 -0700717 if k in servo_constants.SERVO_ATTR_KEYS}
Richard Barnetteea3e4602016-06-10 12:36:41 -0700718
Garry Wang11b5e872020-03-11 15:14:08 -0700719 if servo_constants.SERVO_PORT_ATTR in servo_args:
Prathmesh Prabhucba44292018-08-28 17:44:45 -0700720 try:
Garry Wang11b5e872020-03-11 15:14:08 -0700721 servo_args[servo_constants.SERVO_PORT_ATTR] = int(
722 servo_args[servo_constants.SERVO_PORT_ATTR])
Prathmesh Prabhucba44292018-08-28 17:44:45 -0700723 except ValueError:
724 logging.error('servo port is not an int: %s',
Garry Wang11b5e872020-03-11 15:14:08 -0700725 servo_args[servo_constants.SERVO_PORT_ATTR])
Prathmesh Prabhucba44292018-08-28 17:44:45 -0700726 # Reset servo_args because we don't want to use an invalid port.
Garry Wang11b5e872020-03-11 15:14:08 -0700727 servo_args.pop(servo_constants.SERVO_HOST_ATTR, None)
Prathmesh Prabhucba44292018-08-28 17:44:45 -0700728
729 if info.board:
Garry Wang11b5e872020-03-11 15:14:08 -0700730 servo_board = _map_afe_board_to_servo_board(info.board)
731 servo_args[servo_constants.SERVO_BOARD_ATTR] = servo_board
Nick Sanders2f3c9852018-10-24 12:10:24 -0700732 if info.model:
Garry Wang11b5e872020-03-11 15:14:08 -0700733 servo_args[servo_constants.SERVO_MODEL_ATTR] = info.model
734 return servo_args if servo_constants.SERVO_HOST_ATTR in servo_args else None
Richard Barnetteea3e4602016-06-10 12:36:41 -0700735
736
Prathmesh Prabhuefb1b482018-08-28 17:15:05 -0700737def _tweak_args_for_ssp_moblab(servo_args):
Garry Wang11b5e872020-03-11 15:14:08 -0700738 if (servo_args[servo_constants.SERVO_HOST_ATTR]
739 in ['localhost', '127.0.0.1']):
740 servo_args[servo_constants.SERVO_HOST_ATTR] = _CONFIG.get_config_value(
Prathmesh Prabhuefb1b482018-08-28 17:15:05 -0700741 'SSP', 'host_container_ip', type=str, default=None)
742
743
Dan Shi023aae32016-05-25 11:13:01 -0700744def create_servo_host(dut, servo_args, try_lab_servo=False,
Gregory Nisbetde13e2a2019-12-09 22:44:00 -0800745 try_servo_repair=False, dut_host_info=None):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700746 """Create a ServoHost object for a given DUT, if appropriate.
Dan Shi4d478522014-02-14 13:46:32 -0800747
Richard Barnette9a26ad62016-06-10 12:03:08 -0700748 This function attempts to create and verify or repair a `ServoHost`
749 object for a servo connected to the given `dut`, subject to various
750 constraints imposed by the parameters:
751 * When the `servo_args` parameter is not `None`, a servo
752 host must be created, and must be checked with `repair()`.
753 * Otherwise, if a servo exists in the lab and `try_lab_servo` is
754 true:
755 * If `try_servo_repair` is true, then create a servo host and
756 check it with `repair()`.
757 * Otherwise, if the servo responds to `ping` then create a
758 servo host and check it with `verify()`.
Fang Denge545abb2014-12-30 18:43:47 -0800759
Richard Barnette9a26ad62016-06-10 12:03:08 -0700760 In cases where `servo_args` was not `None`, repair failure
761 exceptions are passed back to the caller; otherwise, exceptions
Richard Barnette07c2e1d2016-10-26 14:24:28 -0700762 are logged and then discarded. Note that this only happens in cases
763 where we're called from a test (not special task) control file that
764 has an explicit dependency on servo. In that case, we require that
765 repair not write to `status.log`, so as to avoid polluting test
766 results.
767
768 TODO(jrbarnette): The special handling for servo in test control
769 files is a thorn in my flesh; I dearly hope to see it cut out before
770 my retirement.
Richard Barnette9a26ad62016-06-10 12:03:08 -0700771
772 Parameters for a servo host consist of a host name, port number, and
773 DUT board, and are determined from one of these sources, in order of
774 priority:
Richard Barnetteea3e4602016-06-10 12:36:41 -0700775 * Servo attributes from the `dut` parameter take precedence over
776 all other sources of information.
777 * If a DNS entry for the servo based on the DUT hostname exists in
778 the CrOS lab network, that hostname is used with the default
Richard Barnette9a26ad62016-06-10 12:03:08 -0700779 port and the DUT's board.
Richard Barnetteea3e4602016-06-10 12:36:41 -0700780 * If no other options are found, the parameters will be taken
Richard Barnette9a26ad62016-06-10 12:03:08 -0700781 from the `servo_args` dict passed in from the caller.
Richard Barnetteea3e4602016-06-10 12:36:41 -0700782
783 @param dut An instance of `Host` from which to take
784 servo parameters (if available).
785 @param servo_args A dictionary with servo parameters to use if
786 they can't be found from `dut`. If this
787 argument is supplied, unrepaired exceptions
788 from `verify()` will be passed back to the
789 caller.
790 @param try_lab_servo If not true, servo host creation will be
791 skipped unless otherwise required by the
792 caller.
Richard Barnette9a26ad62016-06-10 12:03:08 -0700793 @param try_servo_repair If true, check a servo host with
794 `repair()` instead of `verify()`.
Dan Shi4d478522014-02-14 13:46:32 -0800795
796 @returns: A ServoHost object or None. See comments above.
797
798 """
Richard Barnette07c2e1d2016-10-26 14:24:28 -0700799 servo_dependency = servo_args is not None
Richard Barnette07c2e1d2016-10-26 14:24:28 -0700800 if dut is not None and (try_lab_servo or servo_dependency):
Prathmesh Prabhub4810232018-09-07 13:24:08 -0700801 servo_args_override = get_servo_args_for_host(dut)
Richard Barnetteea3e4602016-06-10 12:36:41 -0700802 if servo_args_override is not None:
Prathmesh Prabhuefb1b482018-08-28 17:15:05 -0700803 if utils.in_moblab_ssp():
804 _tweak_args_for_ssp_moblab(servo_args_override)
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -0700805 logging.debug(
806 'Overriding provided servo_args (%s) with arguments'
807 ' determined from the host (%s)',
808 servo_args,
809 servo_args_override,
810 )
Richard Barnetteea3e4602016-06-10 12:36:41 -0700811 servo_args = servo_args_override
Prathmesh Prabhucba44292018-08-28 17:44:45 -0700812
Richard Barnetteea3e4602016-06-10 12:36:41 -0700813 if servo_args is None:
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -0700814 logging.debug('No servo_args provided, and failed to find overrides.')
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700815 if try_lab_servo or servo_dependency:
Garry Wang11b5e872020-03-11 15:14:08 -0700816 return None, servo_constants.SERVO_STATE_NOT_CONNECTED
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700817 else:
818 # For regular test case which not required the servo
819 return None, None
820
Garry Wang11b5e872020-03-11 15:14:08 -0700821 servo_hostname = servo_args.get(servo_constants.SERVO_HOST_ATTR)
822 servo_port = servo_args.get(servo_constants.SERVO_PORT_ATTR)
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700823 if not _is_servo_host_information_exist(servo_hostname, servo_port):
824 logging.debug(
825 'Servo connection info missed hostname: %s , port: %s',
826 servo_hostname, servo_port)
Garry Wang11b5e872020-03-11 15:14:08 -0700827 return None, servo_constants.SERVO_STATE_NOT_CONNECTED
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700828 if not is_servo_host_information_valid(servo_hostname, servo_port):
829 logging.debug(
830 'Servo connection info is incorrect hostname: %s , port: %s',
831 servo_hostname, servo_port)
Garry Wang11b5e872020-03-11 15:14:08 -0700832 return None, servo_constants.SERVO_STATE_WRONG_CONFIG
Richard Barnette07c2e1d2016-10-26 14:24:28 -0700833 if (not servo_dependency and not try_servo_repair and
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700834 not servo_host_is_up(servo_hostname)):
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -0700835 logging.debug('ServoHost is not up.')
Garry Wang11b5e872020-03-11 15:14:08 -0700836 return None, servo_constants.SERVO_STATE_BROKEN
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -0700837
Garry Wangebc015b2019-06-06 17:45:06 -0700838 newhost = ServoHost(**servo_args)
Garry Wangcdd27b22020-01-13 14:59:11 -0800839 try:
840 newhost.restart_servod(quick_startup=True)
841 except error.AutoservSSHTimeout:
842 logging.warning("Restart servod failed due ssh connection "
843 "to servohost timed out. This error is forgiven"
844 " here, we will retry in servo repair process.")
845 except error.AutoservRunError as e:
846 logging.warning("Restart servod failed due to:\n%s\n"
847 "This error is forgiven here, we will retry"
848 " in servo repair process.", str(e))
Garry Wangebc015b2019-06-06 17:45:06 -0700849
Gregory Nisbetde13e2a2019-12-09 22:44:00 -0800850 # TODO(gregorynisbet): Clean all of this up.
851 logging.debug('create_servo_host: attempt to set info store on '
852 'servo host')
853 try:
854 if dut_host_info is None:
855 logging.debug('create_servo_host: dut_host_info is '
856 'None, skipping')
857 else:
858 newhost.set_dut_host_info(dut_host_info)
859 logging.debug('create_servo_host: successfully set info '
860 'store')
861 except Exception:
862 logging.error("create_servo_host: (%s)", traceback.format_exc())
863
Richard Barnette9a26ad62016-06-10 12:03:08 -0700864 # Note that the logic of repair() includes everything done
865 # by verify(). It's sufficient to call one or the other;
866 # we don't need both.
Richard Barnette07c2e1d2016-10-26 14:24:28 -0700867 if servo_dependency:
868 newhost.repair(silent=True)
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700869 return newhost, newhost.get_servo_state()
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -0700870
871 if try_servo_repair:
872 try:
873 newhost.repair()
874 except Exception:
875 logging.exception('servo repair failed for %s', newhost.hostname)
Richard Barnette9a26ad62016-06-10 12:03:08 -0700876 else:
877 try:
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -0700878 newhost.verify()
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700879 except Exception:
Prathmesh Prabhu88bf6052018-08-28 16:21:26 -0700880 logging.exception('servo verify failed for %s', newhost.hostname)
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700881 return newhost, newhost.get_servo_state()
Otabek Kasimov7267a7a2020-03-04 11:18:45 -0800882
883
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700884def _is_servo_host_information_exist(hostname, port):
Otabek Kasimov7267a7a2020-03-04 11:18:45 -0800885 if hostname is None or len(hostname.strip()) == 0:
886 return False
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700887 if port is None:
Otabek Kasimov7267a7a2020-03-04 11:18:45 -0800888 return False
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700889 if not type(port) is int:
890 try:
891 int(port)
892 except ValueError:
893 return False
894
Otabek Kasimov7267a7a2020-03-04 11:18:45 -0800895 return True
896
897
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700898def is_servo_host_information_valid(hostname, port):
899 if not _is_servo_host_information_exist(hostname, port):
Otabek Kasimov7267a7a2020-03-04 11:18:45 -0800900 return False
901 # checking range and correct of the port
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700902 port_int = int(port)
Otabek Kasimov7267a7a2020-03-04 11:18:45 -0800903 if port_int < 1 or port_int > 65000:
904 return False
905 # we expecting host contain only latters, digits and '-' or '_'
Otabek Kasimova7ba91a2020-03-09 08:31:01 -0700906 if not re.match('[a-zA-Z0-9-_\.]*$', hostname) or len(hostname) < 5:
Otabek Kasimov7267a7a2020-03-04 11:18:45 -0800907 return False
908 return True