blob: 0f2b7d97d832962e201aaf7a93a6b1277af610fa [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
12import httplib
13import logging
14import socket
Kevin Cheng79589982016-10-25 13:26:04 -070015import traceback
Fang Deng5d518f42013-08-02 14:04:32 -070016import xmlrpclib
17
18from autotest_lib.client.bin import utils
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -070019from autotest_lib.client.common_lib import control_data
Fang Deng5d518f42013-08-02 14:04:32 -070020from autotest_lib.client.common_lib import error
beeps5e8c45a2013-12-17 22:05:11 -080021from autotest_lib.client.common_lib import global_config
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -070022from autotest_lib.client.common_lib import host_states
Richard Barnette9a26ad62016-06-10 12:03:08 -070023from autotest_lib.client.common_lib import hosts
Dan Shi0942b1d2015-03-31 11:07:00 -070024from autotest_lib.client.common_lib import lsbrelease_utils
beeps5e8c45a2013-12-17 22:05:11 -080025from autotest_lib.client.common_lib.cros import autoupdater
26from autotest_lib.client.common_lib.cros import dev_server
Fang Deng5d518f42013-08-02 14:04:32 -070027from autotest_lib.client.common_lib.cros import retry
Kevin Cheng79589982016-10-25 13:26:04 -070028from autotest_lib.client.common_lib.cros.graphite import autotest_es
Christopher Wileycef1f902014-06-19 11:11:23 -070029from autotest_lib.client.common_lib.cros.network import ping_runner
Hsinyu Chaoe0b08e62015-08-11 10:50:37 +000030from autotest_lib.client.cros import constants as client_constants
Richard Barnettee519dcd2016-08-15 17:37:17 -070031from autotest_lib.server import afe_utils
beeps5e8c45a2013-12-17 22:05:11 -080032from autotest_lib.server import site_utils as server_site_utils
Cheng-Yi Chiang22612862015-08-20 20:39:57 +080033from autotest_lib.server.cros import dnsname_mangler
Simran Basi0739d682015-02-25 16:22:56 -080034from autotest_lib.server.cros.dynamic_suite import frontend_wrappers
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -070035from autotest_lib.server.cros.dynamic_suite import control_file_getter
Richard Barnette9a26ad62016-06-10 12:03:08 -070036from autotest_lib.server.cros.servo import servo
37from autotest_lib.server.hosts import servo_repair
Fang Deng5d518f42013-08-02 14:04:32 -070038from autotest_lib.server.hosts import ssh_host
Fang Dengd4fe7392013-09-20 12:18:21 -070039from autotest_lib.site_utils.rpm_control_system import rpm_client
Fang Deng5d518f42013-08-02 14:04:32 -070040
Aviv Keshet11836322016-11-22 11:32:01 -080041from chromite.lib import metrics
Fang Deng5d518f42013-08-02 14:04:32 -070042
Simran Basi0739d682015-02-25 16:22:56 -080043# Names of the host attributes in the database that represent the values for
44# the servo_host and servo_port for a servo connected to the DUT.
45SERVO_HOST_ATTR = 'servo_host'
46SERVO_PORT_ATTR = 'servo_port'
Richard Barnettee519dcd2016-08-15 17:37:17 -070047SERVO_BOARD_ATTR = 'servo_board'
Kevin Cheng643ce8a2016-09-15 15:42:12 -070048SERVO_SERIAL_ATTR = 'servo_serial'
Simran Basi0739d682015-02-25 16:22:56 -080049
Dan Shi3b2adf62015-09-02 17:46:54 -070050_CONFIG = global_config.global_config
xixuan6cf6d2f2016-01-29 15:29:00 -080051ENABLE_SSH_TUNNEL_FOR_SERVO = _CONFIG.get_config_value(
52 'CROS', 'enable_ssh_tunnel_for_servo', type=bool, default=False)
Simran Basi0739d682015-02-25 16:22:56 -080053
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -070054AUTOTEST_BASE = _CONFIG.get_config_value(
55 'SCHEDULER', 'drone_installation_directory',
56 default='/usr/local/autotest')
57
58_SERVO_HOST_REBOOT_TEST_NAME = 'servohost_Reboot'
Kevin Cheng55265902016-10-19 12:46:50 -070059_SERVO_HOST_FORCE_REBOOT_TEST_NAME = 'servohost_Reboot.force_reboot'
Fang Deng5d518f42013-08-02 14:04:32 -070060
Fang Deng5d518f42013-08-02 14:04:32 -070061class ServoHost(ssh_host.SSHHost):
62 """Host class for a host that controls a servo, e.g. beaglebone."""
63
Richard Barnette9a26ad62016-06-10 12:03:08 -070064 DEFAULT_PORT = 9999
65
Dan Shie5b3c512014-08-21 12:12:09 -070066 # Timeout for initializing servo signals.
67 INITIALIZE_SERVO_TIMEOUT_SECS = 30
Richard Barnette9a26ad62016-06-10 12:03:08 -070068
xixuan6cf6d2f2016-01-29 15:29:00 -080069 # Ready test function
70 SERVO_READY_METHOD = 'get_version'
Fang Deng5d518f42013-08-02 14:04:32 -070071
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -070072 REBOOT_CMD = 'sleep 1; reboot & sleep 10; reboot -f'
73
Fang Deng5d518f42013-08-02 14:04:32 -070074
Richard Barnette17bfc6c2016-08-04 18:41:43 -070075 def _initialize(self, servo_host='localhost',
Richard Barnettee519dcd2016-08-15 17:37:17 -070076 servo_port=DEFAULT_PORT, servo_board=None,
Kevin Cheng643ce8a2016-09-15 15:42:12 -070077 servo_serial=None, is_in_lab=None, *args, **dargs):
Fang Deng5d518f42013-08-02 14:04:32 -070078 """Initialize a ServoHost instance.
79
80 A ServoHost instance represents a host that controls a servo.
81
82 @param servo_host: Name of the host where the servod process
83 is running.
84 @param servo_port: Port the servod process is listening on.
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -070085 @param servo_board: Board that the servo is connected to.
Dan Shi4d478522014-02-14 13:46:32 -080086 @param is_in_lab: True if the servo host is in Cros Lab. Default is set
87 to None, for which utils.host_is_in_lab_zone will be
88 called to check if the servo host is in Cros lab.
Fang Deng5d518f42013-08-02 14:04:32 -070089
90 """
91 super(ServoHost, self)._initialize(hostname=servo_host,
92 *args, **dargs)
Richard Barnettee519dcd2016-08-15 17:37:17 -070093 self.servo_port = servo_port
94 self.servo_board = servo_board
Kevin Cheng643ce8a2016-09-15 15:42:12 -070095 self.servo_serial = servo_serial
Richard Barnettee519dcd2016-08-15 17:37:17 -070096 self._servo = None
Richard Barnette9a26ad62016-06-10 12:03:08 -070097 self._repair_strategy = (
98 servo_repair.create_servo_repair_strategy())
Richard Barnettee519dcd2016-08-15 17:37:17 -070099 self._is_localhost = (self.hostname == 'localhost')
100 if self._is_localhost:
101 self._is_in_lab = False
102 elif is_in_lab is None:
Dan Shi4d478522014-02-14 13:46:32 -0800103 self._is_in_lab = utils.host_is_in_lab_zone(self.hostname)
104 else:
105 self._is_in_lab = is_in_lab
xixuan6cf6d2f2016-01-29 15:29:00 -0800106
Richard Barnettee519dcd2016-08-15 17:37:17 -0700107 # Commands on the servo host must be run by the superuser.
108 # Our account on a remote host is root, but if our target is
109 # localhost then we might be running unprivileged. If so,
110 # `sudo` will have to be added to the commands.
Fang Deng5d518f42013-08-02 14:04:32 -0700111 if self._is_localhost:
112 self._sudo_required = utils.system_output('id -u') != '0'
113 else:
114 self._sudo_required = False
Richard Barnettee519dcd2016-08-15 17:37:17 -0700115
Richard Barnette9a26ad62016-06-10 12:03:08 -0700116
117 def connect_servo(self):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700118 """Establish a connection to the servod server on this host.
Richard Barnette9a26ad62016-06-10 12:03:08 -0700119
120 Initializes `self._servo` and then verifies that all network
121 connections are working. This will create an ssh tunnel if
122 it's required.
123
124 As a side effect of testing the connection, all signals on the
125 target servo are reset to default values, and the USB stick is
126 set to the neutral (off) position.
127 """
Kevin Cheng643ce8a2016-09-15 15:42:12 -0700128 servo_obj = servo.Servo(servo_host=self, servo_serial=self.servo_serial)
Richard Barnette9a26ad62016-06-10 12:03:08 -0700129 timeout, _ = retry.timeout(
130 servo_obj.initialize_dut,
131 timeout_sec=self.INITIALIZE_SERVO_TIMEOUT_SECS)
132 if timeout:
133 raise hosts.AutoservVerifyError(
134 'Servo initialize timed out.')
135 self._servo = servo_obj
136
137
138 def disconnect_servo(self):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700139 """Disconnect our servo if it exists.
Richard Barnette9a26ad62016-06-10 12:03:08 -0700140
141 If we've previously successfully connected to our servo,
142 disconnect any established ssh tunnel, and set `self._servo`
143 back to `None`.
144 """
145 if self._servo:
146 # N.B. This call is safe even without a tunnel:
147 # rpc_server_tracker.disconnect() silently ignores
148 # unknown ports.
149 self.rpc_server_tracker.disconnect(self.servo_port)
150 self._servo = None
Fang Deng5d518f42013-08-02 14:04:32 -0700151
152
153 def is_in_lab(self):
154 """Check whether the servo host is a lab device.
155
156 @returns: True if the servo host is in Cros Lab, otherwise False.
157
158 """
159 return self._is_in_lab
160
161
162 def is_localhost(self):
163 """Checks whether the servo host points to localhost.
164
165 @returns: True if it points to localhost, otherwise False.
166
167 """
168 return self._is_localhost
169
170
171 def get_servod_server_proxy(self):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700172 """Return a proxy that can be used to communicate with servod server.
Fang Deng5d518f42013-08-02 14:04:32 -0700173
174 @returns: An xmlrpclib.ServerProxy that is connected to the servod
175 server on the host.
Fang Deng5d518f42013-08-02 14:04:32 -0700176 """
Richard Barnette9a26ad62016-06-10 12:03:08 -0700177 if ENABLE_SSH_TUNNEL_FOR_SERVO and not self.is_localhost():
178 return self.rpc_server_tracker.xmlrpc_connect(
179 None, self.servo_port,
180 ready_test_name=self.SERVO_READY_METHOD,
181 timeout_seconds=60)
182 else:
183 remote = 'http://%s:%s' % (self.hostname, self.servo_port)
184 return xmlrpclib.ServerProxy(remote)
Fang Deng5d518f42013-08-02 14:04:32 -0700185
186
Richard Barnette9a26ad62016-06-10 12:03:08 -0700187 def is_cros_host(self):
beeps5e8c45a2013-12-17 22:05:11 -0800188 """Check if a servo host is running chromeos.
189
190 @return: True if the servo host is running chromeos.
191 False if it isn't, or we don't have enough information.
192 """
193 try:
194 result = self.run('grep -q CHROMEOS /etc/lsb-release',
195 ignore_status=True, timeout=10)
196 except (error.AutoservRunError, error.AutoservSSHTimeout):
197 return False
198 return result.exit_status == 0
199
200
Fang Deng5d518f42013-08-02 14:04:32 -0700201 def make_ssh_command(self, user='root', port=22, opts='', hosts_file=None,
202 connect_timeout=None, alive_interval=None):
203 """Override default make_ssh_command to use tuned options.
204
205 Tuning changes:
206 - ConnectTimeout=30; maximum of 30 seconds allowed for an SSH
207 connection failure. Consistency with remote_access.py.
208
209 - ServerAliveInterval=180; which causes SSH to ping connection every
210 180 seconds. In conjunction with ServerAliveCountMax ensures
211 that if the connection dies, Autotest will bail out quickly.
212
213 - ServerAliveCountMax=3; consistency with remote_access.py.
214
215 - ConnectAttempts=4; reduce flakiness in connection errors;
216 consistency with remote_access.py.
217
218 - UserKnownHostsFile=/dev/null; we don't care about the keys.
219
220 - SSH protocol forced to 2; needed for ServerAliveInterval.
221
222 @param user User name to use for the ssh connection.
223 @param port Port on the target host to use for ssh connection.
224 @param opts Additional options to the ssh command.
225 @param hosts_file Ignored.
226 @param connect_timeout Ignored.
227 @param alive_interval Ignored.
228
229 @returns: An ssh command with the requested settings.
230
231 """
232 base_command = ('/usr/bin/ssh -a -x %s -o StrictHostKeyChecking=no'
233 ' -o UserKnownHostsFile=/dev/null -o BatchMode=yes'
234 ' -o ConnectTimeout=30 -o ServerAliveInterval=180'
235 ' -o ServerAliveCountMax=3 -o ConnectionAttempts=4'
236 ' -o Protocol=2 -l %s -p %d')
237 return base_command % (opts, user, port)
238
239
240 def _make_scp_cmd(self, sources, dest):
241 """Format scp command.
242
243 Given a list of source paths and a destination path, produces the
244 appropriate scp command for encoding it. Remote paths must be
245 pre-encoded. Overrides _make_scp_cmd in AbstractSSHHost
246 to allow additional ssh options.
247
248 @param sources: A list of source paths to copy from.
249 @param dest: Destination path to copy to.
250
251 @returns: An scp command that copies |sources| on local machine to
252 |dest| on the remote servo host.
253
254 """
255 command = ('scp -rq %s -o BatchMode=yes -o StrictHostKeyChecking=no '
256 '-o UserKnownHostsFile=/dev/null -P %d %s "%s"')
257 return command % (self.master_ssh_option,
258 self.port, ' '.join(sources), dest)
259
260
261 def run(self, command, timeout=3600, ignore_status=False,
262 stdout_tee=utils.TEE_TO_LOGS, stderr_tee=utils.TEE_TO_LOGS,
263 connect_timeout=30, options='', stdin=None, verbose=True, args=()):
264 """Run a command on the servo host.
265
266 Extends method `run` in SSHHost. If the servo host is a remote device,
267 it will call `run` in SSHost without changing anything.
268 If the servo host is 'localhost', it will call utils.system_output.
269
270 @param command: The command line string.
271 @param timeout: Time limit in seconds before attempting to
272 kill the running process. The run() function
273 will take a few seconds longer than 'timeout'
274 to complete if it has to kill the process.
275 @param ignore_status: Do not raise an exception, no matter
276 what the exit code of the command is.
277 @param stdout_tee/stderr_tee: Where to tee the stdout/stderr.
278 @param connect_timeout: SSH connection timeout (in seconds)
279 Ignored if host is 'localhost'.
280 @param options: String with additional ssh command options
281 Ignored if host is 'localhost'.
282 @param stdin: Stdin to pass (a string) to the executed command.
283 @param verbose: Log the commands.
284 @param args: Sequence of strings to pass as arguments to command by
285 quoting them in " and escaping their contents if necessary.
286
287 @returns: A utils.CmdResult object.
288
289 @raises AutoservRunError if the command failed.
290 @raises AutoservSSHTimeout SSH connection has timed out. Only applies
291 when servo host is not 'localhost'.
292
293 """
294 run_args = {'command': command, 'timeout': timeout,
295 'ignore_status': ignore_status, 'stdout_tee': stdout_tee,
296 'stderr_tee': stderr_tee, 'stdin': stdin,
297 'verbose': verbose, 'args': args}
298 if self.is_localhost():
299 if self._sudo_required:
Michael Tangf9b3ada2016-11-18 16:01:05 -0800300 run_args['command'] = 'sudo -n sh -c "%s"' % utils.sh_escape(
301 command)
Fang Deng5d518f42013-08-02 14:04:32 -0700302 try:
303 return utils.run(**run_args)
304 except error.CmdError as e:
305 logging.error(e)
306 raise error.AutoservRunError('command execution error',
307 e.result_obj)
308 else:
309 run_args['connect_timeout'] = connect_timeout
310 run_args['options'] = options
311 return super(ServoHost, self).run(**run_args)
312
313
Richard Barnette9a26ad62016-06-10 12:03:08 -0700314 def _get_release_version(self):
Dan Shi0942b1d2015-03-31 11:07:00 -0700315 """Get the value of attribute CHROMEOS_RELEASE_VERSION from lsb-release.
316
317 @returns The version string in lsb-release, under attribute
318 CHROMEOS_RELEASE_VERSION.
319 """
320 lsb_release_content = self.run(
321 'cat "%s"' % client_constants.LSB_RELEASE).stdout.strip()
322 return lsbrelease_utils.get_chromeos_release_version(
323 lsb_release_content=lsb_release_content)
324
325
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700326 def get_attached_duts(self, afe):
327 """Gather a list of duts that use this servo host.
328
329 @param afe: afe instance.
330
331 @returns list of duts.
Richard Barnette3a7697f2016-04-20 11:33:27 -0700332 """
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700333 return afe.get_hosts_by_attribute(
334 attribute=SERVO_HOST_ATTR, value=self.hostname)
335
336
337 def get_board(self):
338 """Determine the board for this servo host.
339
340 @returns a string representing this servo host's board.
341 """
342 return lsbrelease_utils.get_current_board(
343 lsb_release_content=self.run('cat /etc/lsb-release').stdout)
344
345
346 def _choose_dut_for_synchronized_reboot(self, dut_list, afe):
347 """Choose which dut to schedule servo host reboot job.
348
349 We'll want a semi-deterministic way of selecting which host should be
350 scheduled for the servo host reboot job. For now we'll sort the
351 list with the expectation the dut list will stay consistent.
352 From there we'll grab the first dut that is available so we
353 don't schedule a job on a dut that will never run.
354
355 @param dut_list: List of the dut hostnames to choose from.
356 @param afe: Instance of the AFE.
357
358 @return hostname of dut to schedule job on.
359 """
360 afe_hosts = afe.get_hosts(dut_list)
361 afe_hosts.sort()
362 for afe_host in afe_hosts:
363 if afe_host.status not in host_states.UNAVAILABLE_STATES:
364 return afe_host.hostname
365 # If they're all unavailable, just return the first sorted dut.
366 dut_list.sort()
367 return dut_list[0]
368
369
370 def _sync_job_scheduled_for_duts(self, dut_list, afe):
371 """Checks if a synchronized reboot has been scheduled for these duts.
372
373 Grab all the host queue entries that aren't completed for the duts and
374 see if any of them have the expected job name.
375
376 @param dut_list: List of duts to check on.
377 @param afe: Instance of the AFE.
378
379 @returns True if the job is scheduled, False otherwise.
380 """
381 afe_hosts = afe.get_hosts(dut_list)
382 for afe_host in afe_hosts:
383 hqes = afe.get_host_queue_entries(host=afe_host.id, complete=0)
384 for hqe in hqes:
385 job = afe.get_jobs(id=hqe.job.id)
Kevin Cheng55265902016-10-19 12:46:50 -0700386 if job and job[0].name in (_SERVO_HOST_REBOOT_TEST_NAME,
387 _SERVO_HOST_FORCE_REBOOT_TEST_NAME):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700388 return True
389 return False
390
391
Kevin Cheng55265902016-10-19 12:46:50 -0700392 def schedule_synchronized_reboot(self, dut_list, afe, force_reboot=False):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700393 """Schedule a job to reboot the servo host.
394
395 When we schedule a job, it will create a ServoHost object which will
396 go through this entire flow of checking if a reboot is needed and
397 trying to schedule it. There is probably a better approach to setting
398 up a synchronized reboot but I'm coming up short on better ideas so I
399 apologize for this circus show.
400
Kevin Cheng55265902016-10-19 12:46:50 -0700401 @param dut_list: List of duts that need to be locked.
402 @param afe: Instance of afe.
403 @param force_reboot: Boolean to indicate if a forced reboot should be
404 scheduled or not.
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700405 """
406 # If we've already scheduled job on a dut, we're done here.
407 if self._sync_job_scheduled_for_duts(dut_list, afe):
408 return
409
410 # Looks like we haven't scheduled a job yet.
Kevin Cheng55265902016-10-19 12:46:50 -0700411 test = (_SERVO_HOST_REBOOT_TEST_NAME if not force_reboot
412 else _SERVO_HOST_FORCE_REBOOT_TEST_NAME)
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700413 dut = self._choose_dut_for_synchronized_reboot(dut_list, afe)
414 getter = control_file_getter.FileSystemGetter([AUTOTEST_BASE])
Kevin Cheng55265902016-10-19 12:46:50 -0700415 control_file = getter.get_control_file_contents_by_name(test)
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700416 control_type = control_data.CONTROL_TYPE_NAMES.SERVER
Kevin Cheng79589982016-10-25 13:26:04 -0700417 try:
418 afe.create_job(control_file=control_file, name=test,
419 control_type=control_type, hosts=[dut])
420 except Exception as e:
421 # Sometimes creating the job will raise an exception. We'll log it
422 # but we don't want to fail because of it.
423 logging.exception('Scheduling reboot job failed: %s', e)
424 metadata = {'dut': dut,
425 'servo_host': self.hostname,
426 'error': str(e),
427 'details': traceback.format_exc()}
428 # We want to track how often we fail here so we can justify
429 # investing some effort into hardening up afe.create_job().
430 autotest_es.post(use_http=True,
431 type_str='servohost_Reboot_schedule_fail',
432 metadata=metadata)
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700433
434
435 def reboot(self, *args, **dargs):
436 """Reboot using special servo host reboot command."""
437 super(ServoHost, self).reboot(reboot_cmd=self.REBOOT_CMD,
438 *args, **dargs)
439
440
441 def _check_for_reboot(self, updater):
442 """Reboot this servo host if an upgrade is waiting.
Richard Barnette3a7697f2016-04-20 11:33:27 -0700443
444 If the host has successfully downloaded and finalized a new
445 build, reboot.
446
447 @param updater: a ChromiumOSUpdater instance for checking
448 whether reboot is needed.
449 @return Return a (status, build) tuple reflecting the
450 update_engine status and current build of the host
451 at the end of the call.
452 """
Richard Barnette9a26ad62016-06-10 12:03:08 -0700453 current_build_number = self._get_release_version()
Richard Barnette3a7697f2016-04-20 11:33:27 -0700454 status = updater.check_update_status()
455 if status == autoupdater.UPDATER_NEED_REBOOT:
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700456 # Check if we need to schedule an organized reboot.
Kevin Cheng79589982016-10-25 13:26:04 -0700457 afe = frontend_wrappers.RetryingAFE(
458 timeout_min=5, delay_sec=10,
459 server=server_site_utils.get_global_afe_hostname())
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700460 dut_list = self.get_attached_duts(afe)
461 logging.info('servo host has the following duts: %s', dut_list)
462 if len(dut_list) > 1:
463 logging.info('servo host has multiple duts, scheduling '
464 'synchronized reboot')
465 self.schedule_synchronized_reboot(dut_list, afe)
466 return status, current_build_number
467
468 logging.info('Rebooting servo host %s from build %s',
Richard Barnette3a7697f2016-04-20 11:33:27 -0700469 self.hostname, current_build_number)
470 # Tell the reboot() call not to wait for completion.
471 # Otherwise, the call will log reboot failure if servo does
472 # not come back. The logged reboot failure will lead to
473 # test job failure. If the test does not require servo, we
474 # don't want servo failure to fail the test with error:
475 # `Host did not return from reboot` in status.log.
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700476 self.reboot(fastsync=True, wait=False)
Richard Barnette3a7697f2016-04-20 11:33:27 -0700477
478 # We told the reboot() call not to wait, but we need to wait
479 # for the reboot before we continue. Alas. The code from
480 # here below is basically a copy of Host.wait_for_restart(),
481 # with the logging bits ripped out, so that they can't cause
482 # the failure logging problem described above.
483 #
484 # The black stain that this has left on my soul can never be
485 # erased.
486 old_boot_id = self.get_boot_id()
487 if not self.wait_down(timeout=self.WAIT_DOWN_REBOOT_TIMEOUT,
488 warning_timer=self.WAIT_DOWN_REBOOT_WARNING,
489 old_boot_id=old_boot_id):
490 raise error.AutoservHostError(
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700491 'servo host %s failed to shut down.' %
492 self.hostname)
Richard Barnette3a7697f2016-04-20 11:33:27 -0700493 if self.wait_up(timeout=120):
Richard Barnette9a26ad62016-06-10 12:03:08 -0700494 current_build_number = self._get_release_version()
Richard Barnette3a7697f2016-04-20 11:33:27 -0700495 status = updater.check_update_status()
496 logging.info('servo host %s back from reboot, with build %s',
497 self.hostname, current_build_number)
498 else:
499 raise error.AutoservHostError(
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700500 'servo host %s failed to come back from reboot.' %
501 self.hostname)
Richard Barnette3a7697f2016-04-20 11:33:27 -0700502 return status, current_build_number
503
504
Richard Barnette3a7697f2016-04-20 11:33:27 -0700505 def update_image(self, wait_for_update=False):
beeps5e8c45a2013-12-17 22:05:11 -0800506 """Update the image on the servo host, if needed.
507
J. Richard Barnette84895392015-04-30 12:31:01 -0700508 This method recognizes the following cases:
509 * If the Host is not running Chrome OS, do nothing.
510 * If a previously triggered update is now complete, reboot
511 to the new version.
512 * If the host is processing a previously triggered update,
513 do nothing.
514 * If the host is running a version of Chrome OS different
515 from the default for servo Hosts, trigger an update, but
516 don't wait for it to complete.
beeps5e8c45a2013-12-17 22:05:11 -0800517
Richard Barnette3a7697f2016-04-20 11:33:27 -0700518 @param wait_for_update If an update needs to be applied and
519 this is true, then don't return until the update is
520 downloaded and finalized, and the host rebooted.
beeps5e8c45a2013-12-17 22:05:11 -0800521 @raises dev_server.DevServerException: If all the devservers are down.
522 @raises site_utils.ParseBuildNameException: If the devserver returns
523 an invalid build name.
524 @raises autoupdater.ChromiumOSError: If something goes wrong in the
525 checking update engine client status or applying an update.
526 @raises AutoservRunError: If the update_engine_client isn't present on
527 the host, and the host is a cros_host.
J. Richard Barnette84895392015-04-30 12:31:01 -0700528
beeps5e8c45a2013-12-17 22:05:11 -0800529 """
Dan Shib795b5a2015-09-24 13:26:35 -0700530 # servod could be running in a Ubuntu workstation.
Richard Barnette9a26ad62016-06-10 12:03:08 -0700531 if not self.is_cros_host():
beeps5e8c45a2013-12-17 22:05:11 -0800532 logging.info('Not attempting an update, either %s is not running '
533 'chromeos or we cannot find enough information about '
534 'the host.', self.hostname)
535 return
536
Dan Shib795b5a2015-09-24 13:26:35 -0700537 if lsbrelease_utils.is_moblab():
538 logging.info('Not attempting an update, %s is running moblab.',
539 self.hostname)
540 return
541
Richard Barnette383ef9c2016-12-13 11:56:49 -0800542 target_build = afe_utils.get_stable_cros_image_name(self.get_board())
J. Richard Barnette84895392015-04-30 12:31:01 -0700543 target_build_number = server_site_utils.ParseBuildName(
544 target_build)[3]
beeps5e8c45a2013-12-17 22:05:11 -0800545 ds = dev_server.ImageServer.resolve(self.hostname)
J. Richard Barnette84895392015-04-30 12:31:01 -0700546 url = ds.get_update_url(target_build)
beeps5e8c45a2013-12-17 22:05:11 -0800547
548 updater = autoupdater.ChromiumOSUpdater(update_url=url, host=self)
Richard Barnette3a7697f2016-04-20 11:33:27 -0700549 status, current_build_number = self._check_for_reboot(updater)
550 update_pending = True
beeps5e8c45a2013-12-17 22:05:11 -0800551 if status in autoupdater.UPDATER_PROCESSING_UPDATE:
552 logging.info('servo host %s already processing an update, update '
553 'engine client status=%s', self.hostname, status)
J. Richard Barnette84895392015-04-30 12:31:01 -0700554 elif current_build_number != target_build_number:
beeps5e8c45a2013-12-17 22:05:11 -0800555 logging.info('Using devserver url: %s to trigger update on '
556 'servo host %s, from %s to %s', url, self.hostname,
J. Richard Barnette84895392015-04-30 12:31:01 -0700557 current_build_number, target_build_number)
beeps5e8c45a2013-12-17 22:05:11 -0800558 try:
J. Richard Barnette84895392015-04-30 12:31:01 -0700559 ds.stage_artifacts(target_build,
560 artifacts=['full_payload'])
561 except Exception as e:
562 logging.error('Staging artifacts failed: %s', str(e))
563 logging.error('Abandoning update for this cycle.')
beeps5e8c45a2013-12-17 22:05:11 -0800564 else:
J. Richard Barnette84895392015-04-30 12:31:01 -0700565 try:
Richard Barnette7e53aa02016-05-20 10:49:40 -0700566 # TODO(jrbarnette): This 'touch' is a gross hack
567 # to get us past crbug.com/613603. Once that
568 # bug is resolved, we should remove this code.
569 self.run('touch /home/chronos/.oobe_completed')
J. Richard Barnette84895392015-04-30 12:31:01 -0700570 updater.trigger_update()
571 except autoupdater.RootFSUpdateError as e:
572 trigger_download_status = 'failed with %s' % str(e)
Aviv Keshet11836322016-11-22 11:32:01 -0800573 metrics.Counter('chromeos/autotest/servo/'
574 'rootfs_update_failed').increment()
J. Richard Barnette84895392015-04-30 12:31:01 -0700575 else:
576 trigger_download_status = 'passed'
577 logging.info('Triggered download and update %s for %s, '
578 'update engine currently in status %s',
579 trigger_download_status, self.hostname,
580 updater.check_update_status())
beeps5e8c45a2013-12-17 22:05:11 -0800581 else:
582 logging.info('servo host %s does not require an update.',
583 self.hostname)
Richard Barnette3a7697f2016-04-20 11:33:27 -0700584 update_pending = False
585
586 if update_pending and wait_for_update:
587 logging.info('Waiting for servo update to complete.')
588 self.run('update_engine_client --follow', ignore_status=True)
beeps5e8c45a2013-12-17 22:05:11 -0800589
590
Richard Barnette1edbb162016-11-01 11:47:50 -0700591 def verify(self, silent=False):
592 """Update the servo host and verify it's in a good state.
593
594 @param silent If true, suppress logging in `status.log`.
595 """
Richard Barnette79d78c42016-05-25 09:31:21 -0700596 # TODO(jrbarnette) Old versions of beaglebone_servo include
Richard Barnette9a26ad62016-06-10 12:03:08 -0700597 # the powerd package. If you touch the .oobe_completed file
598 # (as we do to work around an update_engine problem), then
599 # powerd will eventually shut down the beaglebone for lack
600 # of (apparent) activity. Current versions of
Richard Barnette79d78c42016-05-25 09:31:21 -0700601 # beaglebone_servo don't have powerd, but until we can purge
602 # the lab of the old images, we need to make sure powerd
603 # isn't running.
604 self.run('stop powerd', ignore_status=True)
Richard Barnette9a26ad62016-06-10 12:03:08 -0700605 try:
Richard Barnette1edbb162016-11-01 11:47:50 -0700606 self._repair_strategy.verify(self, silent)
Richard Barnette9a26ad62016-06-10 12:03:08 -0700607 except:
608 self.disconnect_servo()
609 raise
Fang Deng5d518f42013-08-02 14:04:32 -0700610
611
Richard Barnette1edbb162016-11-01 11:47:50 -0700612 def repair(self, silent=False):
613 """Attempt to repair servo host.
614
615 @param silent If true, suppress logging in `status.log`.
616 """
Richard Barnette9a26ad62016-06-10 12:03:08 -0700617 try:
Richard Barnette1edbb162016-11-01 11:47:50 -0700618 self._repair_strategy.repair(self, silent)
Richard Barnette9a26ad62016-06-10 12:03:08 -0700619 except:
620 self.disconnect_servo()
621 raise
Fang Deng5d518f42013-08-02 14:04:32 -0700622
623
Fang Dengd4fe7392013-09-20 12:18:21 -0700624 def has_power(self):
625 """Return whether or not the servo host is powered by PoE."""
626 # TODO(fdeng): See crbug.com/302791
627 # For now, assume all servo hosts in the lab have power.
628 return self.is_in_lab()
629
630
631 def power_cycle(self):
632 """Cycle power to this host via PoE if it is a lab device.
633
Richard Barnette9a26ad62016-06-10 12:03:08 -0700634 @raises AutoservRepairError if it fails to power cycle the
Fang Dengd4fe7392013-09-20 12:18:21 -0700635 servo host.
636
637 """
638 if self.has_power():
639 try:
640 rpm_client.set_power(self.hostname, 'CYCLE')
641 except (socket.error, xmlrpclib.Error,
642 httplib.BadStatusLine,
643 rpm_client.RemotePowerException) as e:
Richard Barnette9a26ad62016-06-10 12:03:08 -0700644 raise hosts.AutoservRepairError(
Fang Dengd4fe7392013-09-20 12:18:21 -0700645 'Power cycling %s failed: %s' % (self.hostname, e))
646 else:
647 logging.info('Skipping power cycling, not a lab device.')
648
649
Dan Shi4d478522014-02-14 13:46:32 -0800650 def get_servo(self):
651 """Get the cached servo.Servo object.
Fang Deng5d518f42013-08-02 14:04:32 -0700652
Dan Shi4d478522014-02-14 13:46:32 -0800653 @return: a servo.Servo object.
Fang Deng5d518f42013-08-02 14:04:32 -0700654 """
Dan Shi4d478522014-02-14 13:46:32 -0800655 return self._servo
656
657
Richard Barnetteea3e4602016-06-10 12:36:41 -0700658def make_servo_hostname(dut_hostname):
659 """Given a DUT's hostname, return the hostname of its servo.
660
661 @param dut_hostname: hostname of a DUT.
662
663 @return hostname of the DUT's servo.
664
665 """
666 host_parts = dut_hostname.split('.')
667 host_parts[0] = host_parts[0] + '-servo'
668 return '.'.join(host_parts)
669
670
671def servo_host_is_up(servo_hostname):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700672 """Given a servo host name, return if it's up or not.
Richard Barnetteea3e4602016-06-10 12:36:41 -0700673
674 @param servo_hostname: hostname of the servo host.
675
676 @return True if it's up, False otherwise
677 """
678 # Technically, this duplicates the SSH ping done early in the servo
679 # proxy initialization code. However, this ping ends in a couple
680 # seconds when if fails, rather than the 60 seconds it takes to decide
681 # that an SSH ping has timed out. Specifically, that timeout happens
682 # when our servo DNS name resolves, but there is no host at that IP.
683 logging.info('Pinging servo host at %s', servo_hostname)
684 ping_config = ping_runner.PingConfig(
685 servo_hostname, count=3,
686 ignore_result=True, ignore_status=True)
687 return ping_runner.PingRunner().ping(ping_config).received > 0
688
689
Richard Barnettee519dcd2016-08-15 17:37:17 -0700690def _map_afe_board_to_servo_board(afe_board):
691 """Map a board we get from the AFE to a servo appropriate value.
692
693 Many boards are identical to other boards for servo's purposes.
694 This function makes that mapping.
695
696 @param afe_board string board name received from AFE.
697 @return board we expect servo to have.
698
699 """
700 KNOWN_SUFFIXES = ['-freon', '_freon', '_moblab', '-cheets']
701 BOARD_MAP = {'gizmo': 'panther'}
702 mapped_board = afe_board
703 if afe_board in BOARD_MAP:
704 mapped_board = BOARD_MAP[afe_board]
705 else:
706 for suffix in KNOWN_SUFFIXES:
707 if afe_board.endswith(suffix):
708 mapped_board = afe_board[0:-len(suffix)]
709 break
710 if mapped_board != afe_board:
711 logging.info('Mapping AFE board=%s to %s', afe_board, mapped_board)
712 return mapped_board
713
714
Richard Barnetteea3e4602016-06-10 12:36:41 -0700715def _get_standard_servo_args(dut_host):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700716 """Return servo data associated with a given DUT.
Richard Barnetteea3e4602016-06-10 12:36:41 -0700717
718 This checks for the presence of servo host and port attached to the
719 given `dut_host`. This data should be stored in the
Kevin Cheng05ae2a42016-06-06 10:12:48 -0700720 `_afe_host.attributes` field in the provided `dut_host` parameter.
Richard Barnetteea3e4602016-06-10 12:36:41 -0700721
722 @param dut_host Instance of `Host` on which to find the servo
723 attributes.
724 @return A tuple of `servo_args` dict with host and an option port,
725 plus an `is_in_lab` flag indicating whether this in the CrOS
726 test lab, or some different environment.
727 """
728 servo_args = None
729 is_in_lab = False
730 is_ssp_moblab = False
731 if utils.is_in_container():
732 is_moblab = _CONFIG.get_config_value(
733 'SSP', 'is_moblab', type=bool, default=False)
734 is_ssp_moblab = is_moblab
735 else:
736 is_moblab = utils.is_moblab()
Kevin Cheng05ae2a42016-06-06 10:12:48 -0700737 attrs = dut_host._afe_host.attributes
Richard Barnetteea3e4602016-06-10 12:36:41 -0700738 if attrs and SERVO_HOST_ATTR in attrs:
739 servo_host = attrs[SERVO_HOST_ATTR]
740 if (is_ssp_moblab and servo_host in ['localhost', '127.0.0.1']):
741 servo_host = _CONFIG.get_config_value(
742 'SSP', 'host_container_ip', type=str, default=None)
743 servo_args = {SERVO_HOST_ATTR: servo_host}
744 if SERVO_PORT_ATTR in attrs:
Kevin Cheng692e5292016-08-14 00:23:24 -0700745 try:
746 servo_port = attrs[SERVO_PORT_ATTR]
747 servo_args[SERVO_PORT_ATTR] = int(servo_port)
748 except ValueError:
749 logging.error('servo port is not an int: %s', servo_port)
750 # Let's set the servo args to None since we're not creating
751 # the ServoHost object with the proper port now.
752 servo_args = None
Kevin Cheng643ce8a2016-09-15 15:42:12 -0700753 if SERVO_SERIAL_ATTR in attrs:
754 servo_args[SERVO_SERIAL_ATTR] = attrs[SERVO_SERIAL_ATTR]
Richard Barnetteea3e4602016-06-10 12:36:41 -0700755 is_in_lab = (not is_moblab
756 and utils.host_is_in_lab_zone(servo_host))
757
758 # TODO(jrbarnette): This test to use the default lab servo hostname
759 # is a legacy that we need only until every host in the DB has
760 # proper attributes.
761 elif (not is_moblab and
762 not dnsname_mangler.is_ip_address(dut_host.hostname)):
763 servo_host = make_servo_hostname(dut_host.hostname)
764 is_in_lab = utils.host_is_in_lab_zone(servo_host)
765 if is_in_lab:
766 servo_args = {SERVO_HOST_ATTR: servo_host}
Richard Barnette9a26ad62016-06-10 12:03:08 -0700767 if servo_args is not None:
768 servo_board = afe_utils.get_board(dut_host)
769 if servo_board is not None:
770 servo_board = _map_afe_board_to_servo_board(servo_board)
771 servo_args[SERVO_BOARD_ATTR] = servo_board
Richard Barnetteea3e4602016-06-10 12:36:41 -0700772 return servo_args, is_in_lab
773
774
Dan Shi023aae32016-05-25 11:13:01 -0700775def create_servo_host(dut, servo_args, try_lab_servo=False,
Richard Barnette9a26ad62016-06-10 12:03:08 -0700776 try_servo_repair=False):
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700777 """Create a ServoHost object for a given DUT, if appropriate.
Dan Shi4d478522014-02-14 13:46:32 -0800778
Richard Barnette9a26ad62016-06-10 12:03:08 -0700779 This function attempts to create and verify or repair a `ServoHost`
780 object for a servo connected to the given `dut`, subject to various
781 constraints imposed by the parameters:
782 * When the `servo_args` parameter is not `None`, a servo
783 host must be created, and must be checked with `repair()`.
784 * Otherwise, if a servo exists in the lab and `try_lab_servo` is
785 true:
786 * If `try_servo_repair` is true, then create a servo host and
787 check it with `repair()`.
788 * Otherwise, if the servo responds to `ping` then create a
789 servo host and check it with `verify()`.
Fang Denge545abb2014-12-30 18:43:47 -0800790
Richard Barnette9a26ad62016-06-10 12:03:08 -0700791 In cases where `servo_args` was not `None`, repair failure
792 exceptions are passed back to the caller; otherwise, exceptions
Richard Barnette07c2e1d2016-10-26 14:24:28 -0700793 are logged and then discarded. Note that this only happens in cases
794 where we're called from a test (not special task) control file that
795 has an explicit dependency on servo. In that case, we require that
796 repair not write to `status.log`, so as to avoid polluting test
797 results.
798
799 TODO(jrbarnette): The special handling for servo in test control
800 files is a thorn in my flesh; I dearly hope to see it cut out before
801 my retirement.
Richard Barnette9a26ad62016-06-10 12:03:08 -0700802
803 Parameters for a servo host consist of a host name, port number, and
804 DUT board, and are determined from one of these sources, in order of
805 priority:
Richard Barnetteea3e4602016-06-10 12:36:41 -0700806 * Servo attributes from the `dut` parameter take precedence over
807 all other sources of information.
808 * If a DNS entry for the servo based on the DUT hostname exists in
809 the CrOS lab network, that hostname is used with the default
Richard Barnette9a26ad62016-06-10 12:03:08 -0700810 port and the DUT's board.
Richard Barnetteea3e4602016-06-10 12:36:41 -0700811 * If no other options are found, the parameters will be taken
Richard Barnette9a26ad62016-06-10 12:03:08 -0700812 from the `servo_args` dict passed in from the caller.
Richard Barnetteea3e4602016-06-10 12:36:41 -0700813
814 @param dut An instance of `Host` from which to take
815 servo parameters (if available).
816 @param servo_args A dictionary with servo parameters to use if
817 they can't be found from `dut`. If this
818 argument is supplied, unrepaired exceptions
819 from `verify()` will be passed back to the
820 caller.
821 @param try_lab_servo If not true, servo host creation will be
822 skipped unless otherwise required by the
823 caller.
Richard Barnette9a26ad62016-06-10 12:03:08 -0700824 @param try_servo_repair If true, check a servo host with
825 `repair()` instead of `verify()`.
Dan Shi4d478522014-02-14 13:46:32 -0800826
827 @returns: A ServoHost object or None. See comments above.
828
829 """
Richard Barnette07c2e1d2016-10-26 14:24:28 -0700830 servo_dependency = servo_args is not None
Richard Barnetteea3e4602016-06-10 12:36:41 -0700831 is_in_lab = False
Richard Barnette07c2e1d2016-10-26 14:24:28 -0700832 if dut is not None and (try_lab_servo or servo_dependency):
Richard Barnetteea3e4602016-06-10 12:36:41 -0700833 servo_args_override, is_in_lab = _get_standard_servo_args(dut)
834 if servo_args_override is not None:
835 servo_args = servo_args_override
836 if servo_args is None:
837 return None
Richard Barnette07c2e1d2016-10-26 14:24:28 -0700838 if (not servo_dependency and not try_servo_repair and
Richard Barnette9a26ad62016-06-10 12:03:08 -0700839 not servo_host_is_up(servo_args[SERVO_HOST_ATTR])):
Dan Shibbb0cb62014-03-24 17:50:57 -0700840 return None
Richard Barnette9a26ad62016-06-10 12:03:08 -0700841 newhost = ServoHost(is_in_lab=is_in_lab, **servo_args)
842 # Note that the logic of repair() includes everything done
843 # by verify(). It's sufficient to call one or the other;
844 # we don't need both.
Richard Barnette07c2e1d2016-10-26 14:24:28 -0700845 if servo_dependency:
846 newhost.repair(silent=True)
Richard Barnette9a26ad62016-06-10 12:03:08 -0700847 else:
848 try:
849 if try_servo_repair:
850 newhost.repair()
851 else:
852 newhost.verify()
Kevin Cheng5f2ba6c2016-09-28 10:20:05 -0700853 except Exception:
Richard Barnette9a26ad62016-06-10 12:03:08 -0700854 operation = 'repair' if try_servo_repair else 'verification'
855 logging.exception('Servo %s failed for %s',
856 operation, newhost.hostname)
857 return newhost