blob: 12580438ca9bc5cd6ff65af5ccc8a3afb84fb78b [file] [log] [blame]
Derek Beckettf73baca2020-08-19 15:08:47 -07001# Lint as: python2, python3
Oleg Loskutoff75cc9b22019-10-16 17:28:12 -07002# Copyright (c) 2008 The Chromium OS Authors. All rights reserved.
3# Use of this source code is governed by a BSD-style license that can be
4# found in the LICENSE file.
5
Derek Beckettf73baca2020-08-19 15:08:47 -07006from __future__ import absolute_import
7from __future__ import division
8from __future__ import print_function
9
Dan Shid9be07a2017-08-18 09:51:45 -070010import os, time, socket, shutil, glob, logging, tempfile, re
xixuan6cf6d2f2016-01-29 15:29:00 -080011import shlex
Cheng-Yi Chianga155e7e2015-08-20 20:42:04 +080012import subprocess
13
Dan Shi4f8c0242017-07-07 15:34:49 -070014from autotest_lib.client.bin.result_tools import runner as result_tools_runner
Hidehiko Abe28422ed2017-06-21 10:50:44 +090015from autotest_lib.client.common_lib import error
Otabek Kasimovf5e0f102020-06-30 19:41:02 -070016from autotest_lib.client.common_lib import utils
Dan Shi9f92aa62017-07-27 17:07:05 -070017from autotest_lib.client.common_lib.cros.network import ping_runner
Hidehiko Abe28422ed2017-06-21 10:50:44 +090018from autotest_lib.client.common_lib.global_config import global_config
Congbin Guo306cdbc2021-02-08 13:37:41 -080019from autotest_lib.server import autoserv_parser
jadmanski31c49b72008-10-27 20:44:48 +000020from autotest_lib.server import utils, autotest
Prathmesh Prabhu8b5065d2017-01-10 17:13:01 -080021from autotest_lib.server.hosts import host_info
mblighe8b93af2009-01-30 00:45:53 +000022from autotest_lib.server.hosts import remote
Roshan Pius58e5dd32015-10-16 15:16:42 -070023from autotest_lib.server.hosts import rpc_server_tracker
Hidehiko Abe28422ed2017-06-21 10:50:44 +090024from autotest_lib.server.hosts import ssh_multiplex
Derek Beckettfccbb622021-02-08 16:44:53 -080025from autotest_lib.server.hosts.tls_client import exec_dut_command
Andrew Luo4be621d2020-03-21 07:01:13 -070026
Derek Beckettf73baca2020-08-19 15:08:47 -070027import six
28from six.moves import filter
jadmanskica7da372008-10-21 16:26:52 +000029
Otabek Kasimovf5e0f102020-06-30 19:41:02 -070030try:
Mike Frysinger714c5b02020-09-04 23:22:54 -040031 from autotest_lib.utils.frozen_chromite.lib import metrics
Otabek Kasimovf5e0f102020-06-30 19:41:02 -070032except ImportError:
33 metrics = utils.metrics_mock
34
Gwendal Grignou36b61702016-02-10 11:57:53 -080035# pylint: disable=C0111
jadmanskica7da372008-10-21 16:26:52 +000036
mblighb86bfa12010-02-12 20:22:21 +000037get_value = global_config.get_config_value
Derek Beckett189dfa42021-01-05 12:47:14 -080038enable_main_ssh = get_value('AUTOSERV',
39 'enable_main_ssh',
40 type=bool,
Derek Beckett4d102242020-12-01 14:24:37 -080041 default=False)
mblighefccc1b2010-01-11 19:08:42 +000042
Derek Beckett3b9974a2021-03-18 11:16:00 -070043ENABLE_EXEC_DUT_COMMAND = get_value('AUTOSERV',
44 'enable_tls',
45 type=bool,
46 default=False)
47
Dan Shi9f92aa62017-07-27 17:07:05 -070048# Number of seconds to use the cached up status.
49_DEFAULT_UP_STATUS_EXPIRATION_SECONDS = 300
Dean Liaoe3e75f62017-11-14 10:36:43 +080050_DEFAULT_SSH_PORT = 22
mblighefccc1b2010-01-11 19:08:42 +000051
Lutz Justen043e9c12017-10-27 12:40:47 +020052# Number of seconds to wait for the host to shut down in wait_down().
53_DEFAULT_WAIT_DOWN_TIME_SECONDS = 120
54
Philip Chen7ce1e392018-12-09 23:53:32 -080055# Number of seconds to wait for the host to boot up in wait_up().
56_DEFAULT_WAIT_UP_TIME_SECONDS = 120
57
58# Timeout in seconds for a single call of get_boot_id() in wait_down()
59# and a single ssh ping in wait_up().
Lutz Justen043e9c12017-10-27 12:40:47 +020060_DEFAULT_MAX_PING_TIMEOUT = 10
61
Derek Beckettfc04fbc2020-12-14 16:37:31 -080062# The client symlink directory.
63AUTOTEST_CLIENT_SYMLINK_END = 'client/autotest_lib'
64
65
Fang Deng96667ca2013-08-01 17:46:18 -070066class AbstractSSHHost(remote.RemoteHost):
mblighbc9402b2009-12-29 01:15:34 +000067 """
68 This class represents a generic implementation of most of the
jadmanskica7da372008-10-21 16:26:52 +000069 framework necessary for controlling a host via ssh. It implements
70 almost all of the abstract Host methods, except for the core
mblighbc9402b2009-12-29 01:15:34 +000071 Host.run method.
72 """
Simran Basi5ace6f22016-01-06 17:30:44 -080073 VERSION_PREFIX = ''
Derek Beckett4d102242020-12-01 14:24:37 -080074 # Timeout for main ssh connection setup, in seconds.
75 DEFAULT_START_MAIN_SSH_TIMEOUT_S = 5
jadmanskica7da372008-10-21 16:26:52 +000076
Andrew Luo4be621d2020-03-21 07:01:13 -070077 def _initialize(self,
78 hostname,
79 user="root",
80 port=_DEFAULT_SSH_PORT,
81 password="",
82 is_client_install_supported=True,
83 afe_host=None,
84 host_info_store=None,
85 connection_pool=None,
86 *args,
87 **dargs):
jadmanskif6562912008-10-21 17:59:01 +000088 super(AbstractSSHHost, self)._initialize(hostname=hostname,
89 *args, **dargs)
Kevin Cheng05ae2a42016-06-06 10:12:48 -070090 """
91 @param hostname: The hostname of the host.
92 @param user: The username to use when ssh'ing into the host.
93 @param password: The password to use when ssh'ing into the host.
94 @param port: The port to use for ssh.
95 @param is_client_install_supported: Boolean to indicate if we can
96 install autotest on the host.
97 @param afe_host: The host object attained from the AFE (get_hosts).
Prathmesh Prabhu8b5065d2017-01-10 17:13:01 -080098 @param host_info_store: Optional host_info.CachingHostInfoStore object
99 to obtain / update host information.
Hidehiko Abe06893302017-06-24 07:32:38 +0900100 @param connection_pool: ssh_multiplex.ConnectionPool instance to share
Derek Beckett4d102242020-12-01 14:24:37 -0800101 the main ssh connection across control scripts.
Kevin Cheng05ae2a42016-06-06 10:12:48 -0700102 """
Otabek Kasimovf5e0f102020-06-30 19:41:02 -0700103 self._track_class_usage()
Dan Shic07b8932014-12-11 15:22:30 -0800104 # IP address is retrieved only on demand. Otherwise the host
105 # initialization will fail for host is not online.
106 self._ip = None
jadmanskica7da372008-10-21 16:26:52 +0000107 self.user = user
108 self.port = port
109 self.password = password
Roshan Piusa58163a2015-10-14 13:36:29 -0700110 self._is_client_install_supported = is_client_install_supported
showard6eafb492010-01-15 20:29:06 +0000111 self._use_rsync = None
Fang Deng3af66202013-08-16 15:19:25 -0700112 self.known_hosts_file = tempfile.mkstemp()[1]
Roshan Pius58e5dd32015-10-16 15:16:42 -0700113 self._rpc_server_tracker = rpc_server_tracker.RpcServerTracker(self);
Derek Beckettfccbb622021-02-08 16:44:53 -0800114 self._tls_exec_dut_command_client = None
115 self._tls_unstable = False
jadmanskica7da372008-10-21 16:26:52 +0000116
Andrew Luo4be621d2020-03-21 07:01:13 -0700117 # Read the value of the use_icmp flag, setting to true if missing.
118 args_string = autoserv_parser.autoserv_parser.options.args
119 args_dict = utils.args_to_dict(
120 args_string.split() if args_string is not None else '')
121 value = args_dict.get('use_icmp', 'true').lower()
122 if value == 'true':
123 self._use_icmp = True
124 elif value == 'false':
125 self._use_icmp = False
126 else:
127 raise ValueError(
128 'use_icmp must be true or false: {}'.format(value))
mblighefccc1b2010-01-11 19:08:42 +0000129 """
Derek Beckett4d102242020-12-01 14:24:37 -0800130 Main SSH connection background job, socket temp directory and socket
131 control path option. If main-SSH is enabled, these fields will be
132 initialized by start_main_ssh when a new SSH connection is initiated.
mblighefccc1b2010-01-11 19:08:42 +0000133 """
Hidehiko Abe06893302017-06-24 07:32:38 +0900134 self._connection_pool = connection_pool
135 if connection_pool:
Derek Beckett4d102242020-12-01 14:24:37 -0800136 self._main_ssh = connection_pool.get(hostname, user, port)
Hidehiko Abe06893302017-06-24 07:32:38 +0900137 else:
Derek Beckett4d102242020-12-01 14:24:37 -0800138 self._main_ssh = ssh_multiplex.MainSsh(hostname, user, port)
Simran Basi3b858a22015-03-17 16:23:24 -0700139
Kevin Cheng05ae2a42016-06-06 10:12:48 -0700140 self._afe_host = afe_host or utils.EmptyAFEHost()
Prathmesh Prabhu8b5065d2017-01-10 17:13:01 -0800141 self.host_info_store = (host_info_store or
142 host_info.InMemoryHostInfoStore())
showard6eafb492010-01-15 20:29:06 +0000143
Dan Shi9f92aa62017-07-27 17:07:05 -0700144 # The cached status of whether the DUT responded to ping.
145 self._cached_up_status = None
146 # The timestamp when the value of _cached_up_status is set.
147 self._cached_up_status_updated = None
148
149
Dan Shic07b8932014-12-11 15:22:30 -0800150 @property
151 def ip(self):
152 """@return IP address of the host.
153 """
154 if not self._ip:
155 self._ip = socket.getaddrinfo(self.hostname, None)[0][4][0]
156 return self._ip
157
158
Roshan Piusa58163a2015-10-14 13:36:29 -0700159 @property
160 def is_client_install_supported(self):
161 """"
162 Returns True if the host supports autotest client installs, False
163 otherwise.
164 """
165 return self._is_client_install_supported
166
167
Roshan Pius58e5dd32015-10-16 15:16:42 -0700168 @property
169 def rpc_server_tracker(self):
170 """"
171 @return The RPC server tracker associated with this host.
172 """
173 return self._rpc_server_tracker
174
175
Dean Liaoe3e75f62017-11-14 10:36:43 +0800176 @property
177 def is_default_port(self):
Garry Wang81bea592020-08-27 17:25:25 -0700178 """Returns True if its port is default SSH port."""
179 return self.port == _DEFAULT_SSH_PORT
Dean Liaoe3e75f62017-11-14 10:36:43 +0800180
181 @property
182 def host_port(self):
183 """Returns hostname if port is default. Otherwise, hostname:port.
184 """
185 if self.is_default_port:
186 return self.hostname
187 else:
188 return '%s:%d' % (self.hostname, self.port)
189
Andrew Luo4be621d2020-03-21 07:01:13 -0700190 @property
191 def use_icmp(self):
192 """Returns True if icmp pings are allowed."""
193 return self._use_icmp
194
Dean Liaoe3e75f62017-11-14 10:36:43 +0800195
196 # Though it doesn't use self here, it is not declared as staticmethod
197 # because its subclass may use self to access member variables.
198 def make_ssh_command(self, user="root", port=_DEFAULT_SSH_PORT, opts='',
199 hosts_file='/dev/null', connect_timeout=30,
200 alive_interval=300, alive_count_max=3,
201 connection_attempts=1):
202 ssh_options = " ".join([
203 opts,
204 self.make_ssh_options(
205 hosts_file=hosts_file, connect_timeout=connect_timeout,
206 alive_interval=alive_interval, alive_count_max=alive_count_max,
207 connection_attempts=connection_attempts)])
208 return "/usr/bin/ssh -a -x %s -l %s -p %d" % (ssh_options, user, port)
209
210
211 @staticmethod
212 def make_ssh_options(hosts_file='/dev/null', connect_timeout=30,
213 alive_interval=300, alive_count_max=3,
214 connection_attempts=1):
215 """Composes SSH -o options."""
Derek Beckettf73baca2020-08-19 15:08:47 -0700216 assert isinstance(connect_timeout, six.integer_types)
Fang Deng96667ca2013-08-01 17:46:18 -0700217 assert connect_timeout > 0 # can't disable the timeout
Dean Liaoe3e75f62017-11-14 10:36:43 +0800218
219 options = [("StrictHostKeyChecking", "no"),
220 ("UserKnownHostsFile", hosts_file),
221 ("BatchMode", "yes"),
222 ("ConnectTimeout", str(connect_timeout)),
223 ("ServerAliveInterval", str(alive_interval)),
224 ("ServerAliveCountMax", str(alive_count_max)),
225 ("ConnectionAttempts", str(connection_attempts))]
226 return " ".join("-o %s=%s" % kv for kv in options)
Fang Deng96667ca2013-08-01 17:46:18 -0700227
228
showard6eafb492010-01-15 20:29:06 +0000229 def use_rsync(self):
230 if self._use_rsync is not None:
231 return self._use_rsync
232
mblighc9892c02010-01-06 19:02:16 +0000233 # Check if rsync is available on the remote host. If it's not,
234 # don't try to use it for any future file transfers.
Gwendal Grignou03286f02017-03-24 10:50:59 -0700235 self._use_rsync = self.check_rsync()
showard6eafb492010-01-15 20:29:06 +0000236 if not self._use_rsync:
Ilja H. Friedel04be2bd2014-05-07 21:29:59 -0700237 logging.warning("rsync not available on remote host %s -- disabled",
Dean Liaoe3e75f62017-11-14 10:36:43 +0800238 self.host_port)
Eric Lie0493a42010-11-15 13:05:43 -0800239 return self._use_rsync
mblighc9892c02010-01-06 19:02:16 +0000240
241
Gwendal Grignou03286f02017-03-24 10:50:59 -0700242 def check_rsync(self):
mblighc9892c02010-01-06 19:02:16 +0000243 """
244 Check if rsync is available on the remote host.
245 """
246 try:
Allen Liad719c12017-06-27 23:48:04 +0000247 self.run("rsync --version", stdout_tee=None, stderr_tee=None)
mblighc9892c02010-01-06 19:02:16 +0000248 except error.AutoservRunError:
249 return False
250 return True
251
jadmanskica7da372008-10-21 16:26:52 +0000252
Gwendal Grignou36b61702016-02-10 11:57:53 -0800253 def _encode_remote_paths(self, paths, escape=True, use_scp=False):
mblighbc9402b2009-12-29 01:15:34 +0000254 """
255 Given a list of file paths, encodes it as a single remote path, in
256 the style used by rsync and scp.
Gwendal Grignou36b61702016-02-10 11:57:53 -0800257 escape: add \\ to protect special characters.
258 use_scp: encode for scp if true, rsync if false.
mblighbc9402b2009-12-29 01:15:34 +0000259 """
showard56176ec2009-10-28 19:52:30 +0000260 if escape:
261 paths = [utils.scp_remote_escape(path) for path in paths]
Marc Herbert21eb6492015-11-13 15:48:53 -0800262
263 remote = self.hostname
264
265 # rsync and scp require IPv6 brackets, even when there isn't any
266 # trailing port number (ssh doesn't support IPv6 brackets).
267 # In the Python >= 3.3 future, 'import ipaddress' will parse addresses.
268 if re.search(r':.*:', remote):
269 remote = '[%s]' % remote
270
Gwendal Grignou36b61702016-02-10 11:57:53 -0800271 if use_scp:
272 return '%s@%s:"%s"' % (self.user, remote, " ".join(paths))
273 else:
274 return '%s@%s:%s' % (
275 self.user, remote,
276 " :".join('"%s"' % p for p in paths))
jadmanskica7da372008-10-21 16:26:52 +0000277
Gwendal Grignou36b61702016-02-10 11:57:53 -0800278 def _encode_local_paths(self, paths, escape=True):
279 """
280 Given a list of file paths, encodes it as a single local path.
281 escape: add \\ to protect special characters.
282 """
283 if escape:
284 paths = [utils.sh_escape(path) for path in paths]
285
286 return " ".join('"%s"' % p for p in paths)
jadmanskica7da372008-10-21 16:26:52 +0000287
Dean Liaoe3e75f62017-11-14 10:36:43 +0800288
289 def rsync_options(self, delete_dest=False, preserve_symlinks=False,
290 safe_symlinks=False, excludes=None):
291 """Obtains rsync options for the remote."""
Fang Deng96667ca2013-08-01 17:46:18 -0700292 ssh_cmd = self.make_ssh_command(user=self.user, port=self.port,
Derek Beckett4d102242020-12-01 14:24:37 -0800293 opts=self._main_ssh.ssh_option,
Fang Deng96667ca2013-08-01 17:46:18 -0700294 hosts_file=self.known_hosts_file)
jadmanskid7b79ed2009-01-07 17:19:48 +0000295 if delete_dest:
296 delete_flag = "--delete"
297 else:
298 delete_flag = ""
Luigi Semenzato9b083072016-12-19 16:50:40 -0800299 if safe_symlinks:
300 symlink_flag = "-l --safe-links"
301 elif preserve_symlinks:
302 symlink_flag = "-l"
mbligh45561782009-05-11 21:14:34 +0000303 else:
304 symlink_flag = "-L"
Dan Shi92c34c92017-07-14 15:28:56 -0700305 exclude_args = ''
306 if excludes:
307 exclude_args = ' '.join(
308 ["--exclude '%s'" % exclude for exclude in excludes])
Dean Liaoe3e75f62017-11-14 10:36:43 +0800309 return "%s %s --timeout=1800 --rsh='%s' -az --no-o --no-g %s" % (
310 symlink_flag, delete_flag, ssh_cmd, exclude_args)
311
312
313 def _make_rsync_cmd(self, sources, dest, delete_dest,
314 preserve_symlinks, safe_symlinks, excludes=None):
315 """
316 Given a string of source paths and a destination path, produces the
317 appropriate rsync command for copying them. Remote paths must be
318 pre-encoded.
319 """
320 rsync_options = self.rsync_options(
321 delete_dest=delete_dest, preserve_symlinks=preserve_symlinks,
322 safe_symlinks=safe_symlinks, excludes=excludes)
323 return 'rsync %s %s "%s"' % (rsync_options, sources, dest)
jadmanskid7b79ed2009-01-07 17:19:48 +0000324
325
Eric Li861b2d52011-02-04 14:50:35 -0800326 def _make_ssh_cmd(self, cmd):
327 """
328 Create a base ssh command string for the host which can be used
329 to run commands directly on the machine
330 """
Fang Deng96667ca2013-08-01 17:46:18 -0700331 base_cmd = self.make_ssh_command(user=self.user, port=self.port,
Derek Beckett4d102242020-12-01 14:24:37 -0800332 opts=self._main_ssh.ssh_option,
Fang Deng96667ca2013-08-01 17:46:18 -0700333 hosts_file=self.known_hosts_file)
Eric Li861b2d52011-02-04 14:50:35 -0800334
335 return '%s %s "%s"' % (base_cmd, self.hostname, utils.sh_escape(cmd))
336
jadmanskid7b79ed2009-01-07 17:19:48 +0000337 def _make_scp_cmd(self, sources, dest):
mblighbc9402b2009-12-29 01:15:34 +0000338 """
Gwendal Grignou36b61702016-02-10 11:57:53 -0800339 Given a string of source paths and a destination path, produces the
jadmanskid7b79ed2009-01-07 17:19:48 +0000340 appropriate scp command for encoding it. Remote paths must be
mblighbc9402b2009-12-29 01:15:34 +0000341 pre-encoded.
342 """
mblighc0649d62010-01-15 18:15:58 +0000343 command = ("scp -rq %s -o StrictHostKeyChecking=no "
lmraf676f32010-02-04 03:36:26 +0000344 "-o UserKnownHostsFile=%s -P %d %s '%s'")
Derek Beckett4d102242020-12-01 14:24:37 -0800345 return command % (self._main_ssh.ssh_option, self.known_hosts_file,
Gwendal Grignou36b61702016-02-10 11:57:53 -0800346 self.port, sources, dest)
jadmanskid7b79ed2009-01-07 17:19:48 +0000347
348
349 def _make_rsync_compatible_globs(self, path, is_local):
mblighbc9402b2009-12-29 01:15:34 +0000350 """
351 Given an rsync-style path, returns a list of globbed paths
jadmanskid7b79ed2009-01-07 17:19:48 +0000352 that will hopefully provide equivalent behaviour for scp. Does not
353 support the full range of rsync pattern matching behaviour, only that
354 exposed in the get/send_file interface (trailing slashes).
355
356 The is_local param is flag indicating if the paths should be
mblighbc9402b2009-12-29 01:15:34 +0000357 interpreted as local or remote paths.
358 """
jadmanskid7b79ed2009-01-07 17:19:48 +0000359
360 # non-trailing slash paths should just work
361 if len(path) == 0 or path[-1] != "/":
362 return [path]
363
364 # make a function to test if a pattern matches any files
365 if is_local:
showard56176ec2009-10-28 19:52:30 +0000366 def glob_matches_files(path, pattern):
367 return len(glob.glob(path + pattern)) > 0
jadmanskid7b79ed2009-01-07 17:19:48 +0000368 else:
showard56176ec2009-10-28 19:52:30 +0000369 def glob_matches_files(path, pattern):
370 result = self.run("ls \"%s\"%s" % (utils.sh_escape(path),
371 pattern),
372 stdout_tee=None, ignore_status=True)
jadmanskid7b79ed2009-01-07 17:19:48 +0000373 return result.exit_status == 0
374
375 # take a set of globs that cover all files, and see which are needed
376 patterns = ["*", ".[!.]*"]
showard56176ec2009-10-28 19:52:30 +0000377 patterns = [p for p in patterns if glob_matches_files(path, p)]
jadmanskid7b79ed2009-01-07 17:19:48 +0000378
379 # convert them into a set of paths suitable for the commandline
jadmanskid7b79ed2009-01-07 17:19:48 +0000380 if is_local:
showard56176ec2009-10-28 19:52:30 +0000381 return ["\"%s\"%s" % (utils.sh_escape(path), pattern)
382 for pattern in patterns]
jadmanskid7b79ed2009-01-07 17:19:48 +0000383 else:
showard56176ec2009-10-28 19:52:30 +0000384 return [utils.scp_remote_escape(path) + pattern
385 for pattern in patterns]
jadmanskid7b79ed2009-01-07 17:19:48 +0000386
387
388 def _make_rsync_compatible_source(self, source, is_local):
mblighbc9402b2009-12-29 01:15:34 +0000389 """
390 Applies the same logic as _make_rsync_compatible_globs, but
jadmanskid7b79ed2009-01-07 17:19:48 +0000391 applies it to an entire list of sources, producing a new list of
mblighbc9402b2009-12-29 01:15:34 +0000392 sources, properly quoted.
393 """
jadmanskid7b79ed2009-01-07 17:19:48 +0000394 return sum((self._make_rsync_compatible_globs(path, is_local)
395 for path in source), [])
jadmanskica7da372008-10-21 16:26:52 +0000396
397
mblighfeac0102009-04-28 18:31:12 +0000398 def _set_umask_perms(self, dest):
mblighbc9402b2009-12-29 01:15:34 +0000399 """
400 Given a destination file/dir (recursively) set the permissions on
401 all the files and directories to the max allowed by running umask.
402 """
mblighfeac0102009-04-28 18:31:12 +0000403
404 # now this looks strange but I haven't found a way in Python to _just_
405 # get the umask, apparently the only option is to try to set it
406 umask = os.umask(0)
407 os.umask(umask)
408
Derek Beckettf73baca2020-08-19 15:08:47 -0700409 max_privs = 0o777 & ~umask
mblighfeac0102009-04-28 18:31:12 +0000410
411 def set_file_privs(filename):
Chris Masone567d0d92011-12-19 09:38:30 -0800412 """Sets mode of |filename|. Assumes |filename| exists."""
413 file_stat = os.stat(filename)
mblighfeac0102009-04-28 18:31:12 +0000414
415 file_privs = max_privs
416 # if the original file permissions do not have at least one
417 # executable bit then do not set it anywhere
Derek Beckettf73baca2020-08-19 15:08:47 -0700418 if not file_stat.st_mode & 0o111:
419 file_privs &= ~0o111
mblighfeac0102009-04-28 18:31:12 +0000420
421 os.chmod(filename, file_privs)
422
423 # try a bottom-up walk so changes on directory permissions won't cut
424 # our access to the files/directories inside it
425 for root, dirs, files in os.walk(dest, topdown=False):
426 # when setting the privileges we emulate the chmod "X" behaviour
427 # that sets to execute only if it is a directory or any of the
428 # owner/group/other already has execute right
429 for dirname in dirs:
430 os.chmod(os.path.join(root, dirname), max_privs)
431
Chris Masone567d0d92011-12-19 09:38:30 -0800432 # Filter out broken symlinks as we go.
433 for filename in filter(os.path.exists, files):
mblighfeac0102009-04-28 18:31:12 +0000434 set_file_privs(os.path.join(root, filename))
435
436
437 # now set privs for the dest itself
438 if os.path.isdir(dest):
439 os.chmod(dest, max_privs)
440 else:
441 set_file_privs(dest)
442
443
mbligh45561782009-05-11 21:14:34 +0000444 def get_file(self, source, dest, delete_dest=False, preserve_perm=True,
Dana Goyette4d864e12019-09-19 11:05:44 -0700445 preserve_symlinks=False, retry=True, safe_symlinks=False,
446 try_rsync=True):
jadmanskica7da372008-10-21 16:26:52 +0000447 """
448 Copy files from the remote host to a local path.
449
450 Directories will be copied recursively.
451 If a source component is a directory with a trailing slash,
452 the content of the directory will be copied, otherwise, the
453 directory itself and its content will be copied. This
454 behavior is similar to that of the program 'rsync'.
455
456 Args:
457 source: either
458 1) a single file or directory, as a string
459 2) a list of one or more (possibly mixed)
460 files or directories
461 dest: a file or a directory (if source contains a
462 directory or more than one element, you must
463 supply a directory dest)
mbligh89e258d2008-10-24 13:58:08 +0000464 delete_dest: if this is true, the command will also clear
465 out any old files at dest that are not in the
466 source
mblighfeac0102009-04-28 18:31:12 +0000467 preserve_perm: tells get_file() to try to preserve the sources
468 permissions on files and dirs
mbligh45561782009-05-11 21:14:34 +0000469 preserve_symlinks: try to preserve symlinks instead of
470 transforming them into files/dirs on copy
Luigi Semenzato9b083072016-12-19 16:50:40 -0800471 safe_symlinks: same as preserve_symlinks, but discard links
472 that may point outside the copied tree
Dana Goyette4d864e12019-09-19 11:05:44 -0700473 try_rsync: set to False to skip directly to using scp
jadmanskica7da372008-10-21 16:26:52 +0000474 Raises:
475 AutoservRunError: the scp command failed
476 """
Simran Basi882f15b2013-10-29 14:59:34 -0700477 logging.debug('get_file. source: %s, dest: %s, delete_dest: %s,'
478 'preserve_perm: %s, preserve_symlinks:%s', source, dest,
479 delete_dest, preserve_perm, preserve_symlinks)
Dan Shi4f8c0242017-07-07 15:34:49 -0700480
Derek Beckett4d102242020-12-01 14:24:37 -0800481 # Start a main SSH connection if necessary.
482 self.start_main_ssh()
mblighefccc1b2010-01-11 19:08:42 +0000483
Derek Beckettf73baca2020-08-19 15:08:47 -0700484 if isinstance(source, six.string_types):
jadmanskica7da372008-10-21 16:26:52 +0000485 source = [source]
jadmanskid7b79ed2009-01-07 17:19:48 +0000486 dest = os.path.abspath(dest)
jadmanskica7da372008-10-21 16:26:52 +0000487
mblighc9892c02010-01-06 19:02:16 +0000488 # If rsync is disabled or fails, try scp.
showard6eafb492010-01-15 20:29:06 +0000489 try_scp = True
Dana Goyette4d864e12019-09-19 11:05:44 -0700490 if try_rsync and self.use_rsync():
Simran Basi882f15b2013-10-29 14:59:34 -0700491 logging.debug('Using Rsync.')
mblighc9892c02010-01-06 19:02:16 +0000492 try:
493 remote_source = self._encode_remote_paths(source)
494 local_dest = utils.sh_escape(dest)
Gwendal Grignou36b61702016-02-10 11:57:53 -0800495 rsync = self._make_rsync_cmd(remote_source, local_dest,
Luigi Semenzato9b083072016-12-19 16:50:40 -0800496 delete_dest, preserve_symlinks,
497 safe_symlinks)
mblighc9892c02010-01-06 19:02:16 +0000498 utils.run(rsync)
showard6eafb492010-01-15 20:29:06 +0000499 try_scp = False
Derek Beckettf73baca2020-08-19 15:08:47 -0700500 except error.CmdError as e:
Luigi Semenzato7f9dff12016-11-21 14:01:20 -0800501 # retry on rsync exit values which may be caused by transient
502 # network problems:
503 #
504 # rc 10: Error in socket I/O
505 # rc 12: Error in rsync protocol data stream
506 # rc 23: Partial transfer due to error
507 # rc 255: Ssh error
508 #
509 # Note that rc 23 includes dangling symlinks. In this case
510 # retrying is useless, but not very damaging since rsync checks
511 # for those before starting the transfer (scp does not).
512 status = e.result_obj.exit_status
513 if status in [10, 12, 23, 255] and retry:
514 logging.warning('rsync status %d, retrying', status)
515 self.get_file(source, dest, delete_dest, preserve_perm,
516 preserve_symlinks, retry=False)
517 # The nested get_file() does all that's needed.
518 return
519 else:
520 logging.warning("trying scp, rsync failed: %s (%d)",
521 e, status)
mblighc9892c02010-01-06 19:02:16 +0000522
523 if try_scp:
Simran Basi882f15b2013-10-29 14:59:34 -0700524 logging.debug('Trying scp.')
jadmanskid7b79ed2009-01-07 17:19:48 +0000525 # scp has no equivalent to --delete, just drop the entire dest dir
526 if delete_dest and os.path.isdir(dest):
527 shutil.rmtree(dest)
528 os.mkdir(dest)
jadmanskica7da372008-10-21 16:26:52 +0000529
jadmanskid7b79ed2009-01-07 17:19:48 +0000530 remote_source = self._make_rsync_compatible_source(source, False)
531 if remote_source:
showard56176ec2009-10-28 19:52:30 +0000532 # _make_rsync_compatible_source() already did the escaping
Gwendal Grignou36b61702016-02-10 11:57:53 -0800533 remote_source = self._encode_remote_paths(
534 remote_source, escape=False, use_scp=True)
jadmanskid7b79ed2009-01-07 17:19:48 +0000535 local_dest = utils.sh_escape(dest)
Gwendal Grignou36b61702016-02-10 11:57:53 -0800536 scp = self._make_scp_cmd(remote_source, local_dest)
jadmanskid7b79ed2009-01-07 17:19:48 +0000537 try:
538 utils.run(scp)
Derek Beckettf73baca2020-08-19 15:08:47 -0700539 except error.CmdError as e:
Simran Basi882f15b2013-10-29 14:59:34 -0700540 logging.debug('scp failed: %s', e)
jadmanskid7b79ed2009-01-07 17:19:48 +0000541 raise error.AutoservRunError(e.args[0], e.args[1])
jadmanskica7da372008-10-21 16:26:52 +0000542
mblighfeac0102009-04-28 18:31:12 +0000543 if not preserve_perm:
544 # we have no way to tell scp to not try to preserve the
545 # permissions so set them after copy instead.
546 # for rsync we could use "--no-p --chmod=ugo=rwX" but those
547 # options are only in very recent rsync versions
548 self._set_umask_perms(dest)
549
jadmanskica7da372008-10-21 16:26:52 +0000550
mbligh45561782009-05-11 21:14:34 +0000551 def send_file(self, source, dest, delete_dest=False,
Dan Shi92c34c92017-07-14 15:28:56 -0700552 preserve_symlinks=False, excludes=None):
jadmanskica7da372008-10-21 16:26:52 +0000553 """
554 Copy files from a local path to the remote host.
555
556 Directories will be copied recursively.
557 If a source component is a directory with a trailing slash,
558 the content of the directory will be copied, otherwise, the
559 directory itself and its content will be copied. This
560 behavior is similar to that of the program 'rsync'.
561
562 Args:
563 source: either
564 1) a single file or directory, as a string
565 2) a list of one or more (possibly mixed)
566 files or directories
567 dest: a file or a directory (if source contains a
568 directory or more than one element, you must
569 supply a directory dest)
mbligh89e258d2008-10-24 13:58:08 +0000570 delete_dest: if this is true, the command will also clear
571 out any old files at dest that are not in the
572 source
mbligh45561782009-05-11 21:14:34 +0000573 preserve_symlinks: controls if symlinks on the source will be
574 copied as such on the destination or transformed into the
575 referenced file/directory
Dan Shi92c34c92017-07-14 15:28:56 -0700576 excludes: A list of file pattern that matches files not to be
577 sent. `send_file` will fail if exclude is set, since
578 local copy does not support --exclude, e.g., when
579 using scp to copy file.
jadmanskica7da372008-10-21 16:26:52 +0000580
581 Raises:
582 AutoservRunError: the scp command failed
583 """
Simran Basi882f15b2013-10-29 14:59:34 -0700584 logging.debug('send_file. source: %s, dest: %s, delete_dest: %s,'
585 'preserve_symlinks:%s', source, dest,
586 delete_dest, preserve_symlinks)
Derek Beckett4d102242020-12-01 14:24:37 -0800587 # Start a main SSH connection if necessary.
588 self.start_main_ssh()
mblighefccc1b2010-01-11 19:08:42 +0000589
Derek Beckettf73baca2020-08-19 15:08:47 -0700590 if isinstance(source, six.string_types):
jadmanskica7da372008-10-21 16:26:52 +0000591 source = [source]
592
Derek Beckett61525712021-02-11 08:13:46 -0800593 client_symlink = _client_symlink(source)
Derek Beckettbc3d5b62021-03-10 08:57:32 -0800594 # The client symlink *must* be preserved, and should not be sent with
595 # the main send_file in case scp is used, which does not support symlink
596 if client_symlink:
Derek Beckett61525712021-02-11 08:13:46 -0800597 source.remove(client_symlink)
Derek Beckett61525712021-02-11 08:13:46 -0800598
Gwendal Grignou36b61702016-02-10 11:57:53 -0800599 local_sources = self._encode_local_paths(source)
mukesh agrawal0d3616c2015-07-17 15:47:36 -0700600 if not local_sources:
Gwendal Grignou36b61702016-02-10 11:57:53 -0800601 raise error.TestError('source |%s| yielded an empty string' % (
mukesh agrawal0d3616c2015-07-17 15:47:36 -0700602 source))
Gwendal Grignou36b61702016-02-10 11:57:53 -0800603 if local_sources.find('\x00') != -1:
mukesh agrawal0d3616c2015-07-17 15:47:36 -0700604 raise error.TestError('one or more sources include NUL char')
605
Derek Beckettfc04fbc2020-12-14 16:37:31 -0800606 self._send_file(
607 dest=dest,
608 source=source,
609 local_sources=local_sources,
610 delete_dest=delete_dest,
611 excludes=excludes,
612 preserve_symlinks=preserve_symlinks)
Derek Beckettfc04fbc2020-12-14 16:37:31 -0800613
Derek Beckettbc3d5b62021-03-10 08:57:32 -0800614 # Send the client symlink after the rest of the autotest repo has been
615 # sent.
616 if client_symlink:
617 self._send_client_symlink(dest=dest,
618 source=[client_symlink],
619 local_sources=client_symlink,
620 delete_dest=delete_dest,
621 excludes=excludes,
622 preserve_symlinks=True)
623
624 def _send_client_symlink(self, dest, source, local_sources, delete_dest,
625 excludes, preserve_symlinks):
626 if self.use_rsync():
627 if self._send_using_rsync(dest=dest,
628 local_sources=local_sources,
629 delete_dest=delete_dest,
630 preserve_symlinks=preserve_symlinks,
631 excludes=excludes):
632 return
633 # Manually create the symlink if rsync is not available, or fails.
634 try:
635 self.run('mkdir {f} && touch {f}/__init__.py && cd {f} && '
636 'ln -s ../ client'.format(
637 f=os.path.join(dest, 'autotest_lib')))
638 except Exception as e:
639 raise error.AutotestHostRunError(
640 "Could not create client symlink on host: %s" % e)
641
Derek Beckettfc04fbc2020-12-14 16:37:31 -0800642 def _send_file(self, dest, source, local_sources, delete_dest, excludes,
643 preserve_symlinks):
644 """Send file(s), trying rsync first, then scp."""
showard6eafb492010-01-15 20:29:06 +0000645 if self.use_rsync():
Derek Beckettfc04fbc2020-12-14 16:37:31 -0800646 rsync_success = self._send_using_rsync(
647 dest=dest,
648 local_sources=local_sources,
649 delete_dest=delete_dest,
650 preserve_symlinks=preserve_symlinks,
651 excludes=excludes)
Derek Beckettb0821d82021-01-08 11:33:39 -0800652 if rsync_success:
653 return
Derek Beckettfc04fbc2020-12-14 16:37:31 -0800654
Derek Beckettb0821d82021-01-08 11:33:39 -0800655 # Send using scp if you cannot via rsync, or rsync fails.
656 self._send_using_scp(dest=dest,
657 source=source,
658 delete_dest=delete_dest,
659 excludes=excludes)
Derek Beckettfc04fbc2020-12-14 16:37:31 -0800660
661 def _send_using_rsync(self, dest, local_sources, delete_dest,
662 preserve_symlinks, excludes):
663 """Send using rsync.
664
665 Args:
666 dest: a file or a directory (if source contains a
667 directory or more than one element, you must
668 supply a directory dest)
669 local_sources: a string of files/dirs to send separated with spaces
670 delete_dest: if this is true, the command will also clear
671 out any old files at dest that are not in the
672 source
673 preserve_symlinks: controls if symlinks on the source will be
674 copied as such on the destination or transformed into the
675 referenced file/directory
676 excludes: A list of file pattern that matches files not to be
677 sent. `send_file` will fail if exclude is set, since
678 local copy does not support --exclude, e.g., when
679 using scp to copy file.
680 Returns:
681 bool: True if the cmd succeeded, else False
682
683 """
684 logging.debug('Using Rsync.')
685 remote_dest = self._encode_remote_paths([dest])
686 try:
687 rsync = self._make_rsync_cmd(local_sources,
688 remote_dest,
689 delete_dest,
690 preserve_symlinks,
691 False,
692 excludes=excludes)
693 utils.run(rsync)
694 return True
695 except error.CmdError as e:
696 logging.warning("trying scp, rsync failed: %s", e)
697 return False
698
699 def _send_using_scp(self, dest, source, delete_dest, excludes):
700 """Send using scp.
701
702 Args:
703 source: either
704 1) a single file or directory, as a string
705 2) a list of one or more (possibly mixed)
706 files or directories
707 dest: a file or a directory (if source contains a
708 directory or more than one element, you must
709 supply a directory dest)
710 delete_dest: if this is true, the command will also clear
711 out any old files at dest that are not in the
712 source
713 excludes: A list of file pattern that matches files not to be
714 sent. `send_file` will fail if exclude is set, since
715 local copy does not support --exclude, e.g., when
716 using scp to copy file.
717
718 Raises:
719 AutoservRunError: the scp command failed
720 """
721 logging.debug('Trying scp.')
722 if excludes:
723 raise error.AutotestHostRunError(
724 '--exclude is not supported in scp, try to use rsync. '
725 'excludes: %s' % ','.join(excludes), None)
726
727 # scp has no equivalent to --delete, just drop the entire dest dir
728 if delete_dest:
729 is_dir = self.run("ls -d %s/" % dest,
730 ignore_status=True).exit_status == 0
731 if is_dir:
732 cmd = "rm -rf %s && mkdir %s"
733 cmd %= (dest, dest)
734 self.run(cmd)
735
736 remote_dest = self._encode_remote_paths([dest], use_scp=True)
737 local_sources = self._make_rsync_compatible_source(source, True)
738 if local_sources:
739 sources = self._encode_local_paths(local_sources, escape=False)
740 scp = self._make_scp_cmd(sources, remote_dest)
mblighc9892c02010-01-06 19:02:16 +0000741 try:
Derek Beckettfc04fbc2020-12-14 16:37:31 -0800742 utils.run(scp)
Derek Beckettf73baca2020-08-19 15:08:47 -0700743 except error.CmdError as e:
Derek Beckettfc04fbc2020-12-14 16:37:31 -0800744 logging.debug('scp failed: %s', e)
745 raise error.AutoservRunError(e.args[0], e.args[1])
746 else:
747 logging.debug('skipping scp for empty source list')
jadmanskid7b79ed2009-01-07 17:19:48 +0000748
Simran Basi1621c632015-10-14 12:22:23 -0700749 def verify_ssh_user_access(self):
750 """Verify ssh access to this host.
751
752 @returns False if ssh_ping fails due to Permissions error, True
753 otherwise.
754 """
755 try:
756 self.ssh_ping()
757 except (error.AutoservSshPermissionDeniedError,
758 error.AutoservSshPingHostError):
759 return False
760 return True
761
762
Luigi Semenzato135574c2016-08-31 17:25:08 -0700763 def ssh_ping(self, timeout=60, connect_timeout=None, base_cmd='true'):
beepsadd66d32013-03-04 17:21:51 -0800764 """
765 Pings remote host via ssh.
766
Philip Chen7ce1e392018-12-09 23:53:32 -0800767 @param timeout: Command execution timeout in seconds.
beepsadd66d32013-03-04 17:21:51 -0800768 Defaults to 60 seconds.
Philip Chen7ce1e392018-12-09 23:53:32 -0800769 @param connect_timeout: ssh connection timeout in seconds.
beeps46dadc92013-11-07 14:07:10 -0800770 @param base_cmd: The base command to run with the ssh ping.
771 Defaults to true.
beepsadd66d32013-03-04 17:21:51 -0800772 @raise AutoservSSHTimeout: If the ssh ping times out.
773 @raise AutoservSshPermissionDeniedError: If ssh ping fails due to
774 permissions.
775 @raise AutoservSshPingHostError: For other AutoservRunErrors.
776 """
Luigi Semenzato135574c2016-08-31 17:25:08 -0700777 ctimeout = min(timeout, connect_timeout or timeout)
jadmanskica7da372008-10-21 16:26:52 +0000778 try:
Allen Liad719c12017-06-27 23:48:04 +0000779 self.run(base_cmd, timeout=timeout, connect_timeout=ctimeout,
780 ssh_failure_retry_ok=True)
jadmanskica7da372008-10-21 16:26:52 +0000781 except error.AutoservSSHTimeout:
mblighd0e94982009-07-11 00:15:18 +0000782 msg = "Host (ssh) verify timed out (timeout = %d)" % timeout
jadmanskica7da372008-10-21 16:26:52 +0000783 raise error.AutoservSSHTimeout(msg)
mbligh9d738d62009-03-09 21:17:10 +0000784 except error.AutoservSshPermissionDeniedError:
Allen Liad719c12017-06-27 23:48:04 +0000785 #let AutoservSshPermissionDeniedError be visible to the callers
mbligh9d738d62009-03-09 21:17:10 +0000786 raise
Derek Beckettf73baca2020-08-19 15:08:47 -0700787 except error.AutoservRunError as e:
mblighc971c5f2009-06-08 16:48:54 +0000788 # convert the generic AutoservRunError into something more
789 # specific for this context
790 raise error.AutoservSshPingHostError(e.description + '\n' +
791 repr(e.result_obj))
jadmanskica7da372008-10-21 16:26:52 +0000792
793
Luigi Semenzato135574c2016-08-31 17:25:08 -0700794 def is_up(self, timeout=60, connect_timeout=None, base_cmd='true'):
jadmanskica7da372008-10-21 16:26:52 +0000795 """
beeps46dadc92013-11-07 14:07:10 -0800796 Check if the remote host is up by ssh-ing and running a base command.
jadmanskica7da372008-10-21 16:26:52 +0000797
Philip Chen7ce1e392018-12-09 23:53:32 -0800798 @param timeout: command execution timeout in seconds.
799 @param connect_timeout: ssh connection timeout in seconds.
beeps46dadc92013-11-07 14:07:10 -0800800 @param base_cmd: a base command to run with ssh. The default is 'true'.
beepsadd66d32013-03-04 17:21:51 -0800801 @returns True if the remote host is up before the timeout expires,
802 False otherwise.
jadmanskica7da372008-10-21 16:26:52 +0000803 """
804 try:
Luigi Semenzato135574c2016-08-31 17:25:08 -0700805 self.ssh_ping(timeout=timeout,
806 connect_timeout=connect_timeout,
807 base_cmd=base_cmd)
jadmanskica7da372008-10-21 16:26:52 +0000808 except error.AutoservError:
809 return False
810 else:
811 return True
812
813
Otabek Kasimovd48389b2020-12-07 02:38:34 -0800814 def is_up_fast(self, count=1):
815 """Return True if the host can be pinged.
816
817 @param count How many time try to ping before decide that host is not
818 reachable by ping.
819 """
Andrew Luo4be621d2020-03-21 07:01:13 -0700820 if not self._use_icmp:
821 stack = self._get_server_stack_state(lowest_frames=1,
822 highest_frames=7)
823 logging.warning("is_up_fast called with icmp disabled from %s!",
824 stack)
825 return True
Otabek Kasimovd48389b2020-12-07 02:38:34 -0800826 ping_config = ping_runner.PingConfig(self.hostname,
Derek Beckettf7e3d882021-02-12 15:05:16 -0800827 count=1,
Otabek Kasimovd48389b2020-12-07 02:38:34 -0800828 ignore_result=True,
829 ignore_status=True)
Derek Beckettf7e3d882021-02-12 15:05:16 -0800830
831 # Run up to the amount specified, but also exit as soon as the first
832 # reply is found.
833 loops_remaining = count
834 while loops_remaining > 0:
835 loops_remaining -= 1
836 if ping_runner.PingRunner().ping(ping_config).received > 0:
837 return True
838 return False
Dan Shi9f92aa62017-07-27 17:07:05 -0700839
840
Jeremy Bettis657eb412021-02-17 10:40:43 -0700841 def wait_up(self,
842 timeout=_DEFAULT_WAIT_UP_TIME_SECONDS,
843 host_is_down=False):
jadmanskica7da372008-10-21 16:26:52 +0000844 """
845 Wait until the remote host is up or the timeout expires.
846
847 In fact, it will wait until an ssh connection to the remote
848 host can be established, and getty is running.
849
jadmanskic0354912010-01-12 15:57:29 +0000850 @param timeout time limit in seconds before returning even
851 if the host is not up.
Jeremy Bettis657eb412021-02-17 10:40:43 -0700852 @param host_is_down set to True if the host is known to be down before
853 wait_up.
jadmanskica7da372008-10-21 16:26:52 +0000854
beepsadd66d32013-03-04 17:21:51 -0800855 @returns True if the host was found to be up before the timeout expires,
856 False otherwise
jadmanskica7da372008-10-21 16:26:52 +0000857 """
Jeremy Bettis657eb412021-02-17 10:40:43 -0700858 if host_is_down:
859 # Since we expect the host to be down when this is called, if there is
860 # an existing ssh main connection close it.
Jeremy Bettisb4b17ce2021-03-05 11:11:02 -0700861 self.close_main_ssh()
Philip Chen7ce1e392018-12-09 23:53:32 -0800862 current_time = int(time.time())
863 end_time = current_time + timeout
jadmanskica7da372008-10-21 16:26:52 +0000864
Luigi Semenzato135574c2016-08-31 17:25:08 -0700865 autoserv_error_logged = False
Philip Chen7ce1e392018-12-09 23:53:32 -0800866 while current_time < end_time:
867 ping_timeout = min(_DEFAULT_MAX_PING_TIMEOUT,
868 end_time - current_time)
869 if self.is_up(timeout=ping_timeout, connect_timeout=ping_timeout):
jadmanskica7da372008-10-21 16:26:52 +0000870 try:
871 if self.are_wait_up_processes_up():
Dean Liaoe3e75f62017-11-14 10:36:43 +0800872 logging.debug('Host %s is now up', self.host_port)
jadmanskica7da372008-10-21 16:26:52 +0000873 return True
Luigi Semenzato135574c2016-08-31 17:25:08 -0700874 except error.AutoservError as e:
875 if not autoserv_error_logged:
876 logging.debug('Ignoring failure to reach %s: %s %s',
Dean Liaoe3e75f62017-11-14 10:36:43 +0800877 self.host_port, e,
Luigi Semenzato135574c2016-08-31 17:25:08 -0700878 '(and further similar failures)')
879 autoserv_error_logged = True
jadmanskica7da372008-10-21 16:26:52 +0000880 time.sleep(1)
beeps46dadc92013-11-07 14:07:10 -0800881 current_time = int(time.time())
jadmanskica7da372008-10-21 16:26:52 +0000882
jadmanski7ebac3d2010-06-17 16:06:31 +0000883 logging.debug('Host %s is still down after waiting %d seconds',
Dean Liaoe3e75f62017-11-14 10:36:43 +0800884 self.host_port, int(timeout + time.time() - end_time))
jadmanskica7da372008-10-21 16:26:52 +0000885 return False
886
887
Lutz Justen043e9c12017-10-27 12:40:47 +0200888 def wait_down(self, timeout=_DEFAULT_WAIT_DOWN_TIME_SECONDS,
889 warning_timer=None, old_boot_id=None,
890 max_ping_timeout=_DEFAULT_MAX_PING_TIMEOUT):
jadmanskica7da372008-10-21 16:26:52 +0000891 """
892 Wait until the remote host is down or the timeout expires.
893
Lutz Justen043e9c12017-10-27 12:40:47 +0200894 If old_boot_id is provided, waits until either the machine is
895 unpingable or self.get_boot_id() returns a value different from
jadmanskic0354912010-01-12 15:57:29 +0000896 old_boot_id. If the boot_id value has changed then the function
Lutz Justen043e9c12017-10-27 12:40:47 +0200897 returns True under the assumption that the machine has shut down
jadmanskic0354912010-01-12 15:57:29 +0000898 and has now already come back up.
jadmanskica7da372008-10-21 16:26:52 +0000899
jadmanskic0354912010-01-12 15:57:29 +0000900 If old_boot_id is None then until the machine becomes unreachable the
901 method assumes the machine has not yet shut down.
jadmanskica7da372008-10-21 16:26:52 +0000902
Lutz Justen043e9c12017-10-27 12:40:47 +0200903 @param timeout Time limit in seconds before returning even if the host
904 is still up.
905 @param warning_timer Time limit in seconds that will generate a warning
906 if the host is not down yet. Can be None for no warning.
jadmanskic0354912010-01-12 15:57:29 +0000907 @param old_boot_id A string containing the result of self.get_boot_id()
908 prior to the host being told to shut down. Can be None if this is
909 not available.
Lutz Justen043e9c12017-10-27 12:40:47 +0200910 @param max_ping_timeout Maximum timeout in seconds for each
911 self.get_boot_id() call. If this timeout is hit, it is assumed that
912 the host went down and became unreachable.
jadmanskic0354912010-01-12 15:57:29 +0000913
Lutz Justen043e9c12017-10-27 12:40:47 +0200914 @returns True if the host was found to be down (max_ping_timeout timeout
915 expired or boot_id changed if provided) and False if timeout
916 expired.
jadmanskica7da372008-10-21 16:26:52 +0000917 """
mblighe5e3cf22010-05-27 23:33:14 +0000918 #TODO: there is currently no way to distinguish between knowing
919 #TODO: boot_id was unsupported and not knowing the boot_id.
beeps46dadc92013-11-07 14:07:10 -0800920 current_time = int(time.time())
Lutz Justen043e9c12017-10-27 12:40:47 +0200921 end_time = current_time + timeout
jadmanskica7da372008-10-21 16:26:52 +0000922
mbligh2ed998f2009-04-08 21:03:47 +0000923 if warning_timer:
924 warn_time = current_time + warning_timer
925
jadmanskic0354912010-01-12 15:57:29 +0000926 if old_boot_id is not None:
927 logging.debug('Host %s pre-shutdown boot_id is %s',
Dean Liaoe3e75f62017-11-14 10:36:43 +0800928 self.host_port, old_boot_id)
jadmanskic0354912010-01-12 15:57:29 +0000929
beepsadd66d32013-03-04 17:21:51 -0800930 # Impose semi real-time deadline constraints, since some clients
931 # (eg: watchdog timer tests) expect strict checking of time elapsed.
932 # Each iteration of this loop is treated as though it atomically
933 # completes within current_time, this is needed because if we used
934 # inline time.time() calls instead then the following could happen:
935 #
Lutz Justen043e9c12017-10-27 12:40:47 +0200936 # while time.time() < end_time: [23 < 30]
beepsadd66d32013-03-04 17:21:51 -0800937 # some code. [takes 10 secs]
938 # try:
939 # new_boot_id = self.get_boot_id(timeout=end_time - time.time())
940 # [30 - 33]
941 # The last step will lead to a return True, when in fact the machine
942 # went down at 32 seconds (>30). Hence we need to pass get_boot_id
943 # the same time that allowed us into that iteration of the loop.
Lutz Justen043e9c12017-10-27 12:40:47 +0200944 while current_time < end_time:
945 ping_timeout = min(end_time - current_time, max_ping_timeout)
jadmanskic0354912010-01-12 15:57:29 +0000946 try:
Lutz Justen043e9c12017-10-27 12:40:47 +0200947 new_boot_id = self.get_boot_id(timeout=ping_timeout)
mblighdbc7e4a2010-01-15 20:34:20 +0000948 except error.AutoservError:
jadmanskic0354912010-01-12 15:57:29 +0000949 logging.debug('Host %s is now unreachable over ssh, is down',
Dean Liaoe3e75f62017-11-14 10:36:43 +0800950 self.host_port)
jadmanskica7da372008-10-21 16:26:52 +0000951 return True
jadmanskic0354912010-01-12 15:57:29 +0000952 else:
953 # if the machine is up but the boot_id value has changed from
954 # old boot id, then we can assume the machine has gone down
955 # and then already come back up
956 if old_boot_id is not None and old_boot_id != new_boot_id:
957 logging.debug('Host %s now has boot_id %s and so must '
Dean Liaoe3e75f62017-11-14 10:36:43 +0800958 'have rebooted', self.host_port, new_boot_id)
jadmanskic0354912010-01-12 15:57:29 +0000959 return True
mbligh2ed998f2009-04-08 21:03:47 +0000960
961 if warning_timer and current_time > warn_time:
Scott Zawalskic86fdeb2013-10-23 10:24:04 -0400962 self.record("INFO", None, "shutdown",
mbligh2ed998f2009-04-08 21:03:47 +0000963 "Shutdown took longer than %ds" % warning_timer)
964 # Print the warning only once.
965 warning_timer = None
mbligha4464402009-04-17 20:13:41 +0000966 # If a machine is stuck switching runlevels
967 # This may cause the machine to reboot.
968 self.run('kill -HUP 1', ignore_status=True)
mbligh2ed998f2009-04-08 21:03:47 +0000969
jadmanskica7da372008-10-21 16:26:52 +0000970 time.sleep(1)
beeps46dadc92013-11-07 14:07:10 -0800971 current_time = int(time.time())
jadmanskica7da372008-10-21 16:26:52 +0000972
973 return False
jadmanskif6562912008-10-21 17:59:01 +0000974
mbligha0a27592009-01-24 01:41:36 +0000975
jadmanskif6562912008-10-21 17:59:01 +0000976 # tunable constants for the verify & repair code
mblighb86bfa12010-02-12 20:22:21 +0000977 AUTOTEST_GB_DISKSPACE_REQUIRED = get_value("SERVER",
978 "gb_diskspace_required",
Fang Deng6b05f5b2013-03-20 13:42:11 -0700979 type=float,
980 default=20.0)
mbligha0a27592009-01-24 01:41:36 +0000981
jadmanskif6562912008-10-21 17:59:01 +0000982
showardca572982009-09-18 21:20:01 +0000983 def verify_connectivity(self):
984 super(AbstractSSHHost, self).verify_connectivity()
jadmanskif6562912008-10-21 17:59:01 +0000985
Andrew Luo4be621d2020-03-21 07:01:13 -0700986 logging.info('Pinging host %s', self.host_port)
jadmanskif6562912008-10-21 17:59:01 +0000987 self.ssh_ping()
Dean Liaoe3e75f62017-11-14 10:36:43 +0800988 logging.info("Host (ssh) %s is alive", self.host_port)
jadmanskif6562912008-10-21 17:59:01 +0000989
jadmanski80deb752009-01-21 17:14:16 +0000990 if self.is_shutting_down():
mblighc971c5f2009-06-08 16:48:54 +0000991 raise error.AutoservHostIsShuttingDownError("Host is shutting down")
jadmanski80deb752009-01-21 17:14:16 +0000992
mblighb49b5232009-02-12 21:54:49 +0000993
showardca572982009-09-18 21:20:01 +0000994 def verify_software(self):
995 super(AbstractSSHHost, self).verify_software()
jadmanskif6562912008-10-21 17:59:01 +0000996 try:
showardad812bf2009-10-20 23:49:56 +0000997 self.check_diskspace(autotest.Autotest.get_install_dir(self),
998 self.AUTOTEST_GB_DISKSPACE_REQUIRED)
Keith Haddow07f1d3e2017-08-03 17:40:41 -0700999 except error.AutoservDiskFullHostError:
1000 # only want to raise if it's a space issue
1001 raise
1002 except (error.AutoservHostError, autotest.AutodirNotFoundError):
Lutz Justen043e9c12017-10-27 12:40:47 +02001003 logging.exception('autodir space check exception, this is probably '
Keith Haddow07f1d3e2017-08-03 17:40:41 -07001004 'safe to ignore\n')
mblighefccc1b2010-01-11 19:08:42 +00001005
mblighefccc1b2010-01-11 19:08:42 +00001006 def close(self):
1007 super(AbstractSSHHost, self).close()
Godofredo Contreras773179e2016-05-24 10:17:48 -07001008 self.rpc_server_tracker.disconnect_all()
Hidehiko Abe06893302017-06-24 07:32:38 +09001009 if not self._connection_pool:
Derek Beckett4d102242020-12-01 14:24:37 -08001010 self._main_ssh.close()
xixuand6011f12016-12-08 15:01:58 -08001011 if os.path.exists(self.known_hosts_file):
1012 os.remove(self.known_hosts_file)
Derek Beckettfccbb622021-02-08 16:44:53 -08001013 self.tls_exec_dut_command = None
mblighefccc1b2010-01-11 19:08:42 +00001014
Jeremy Bettisb4b17ce2021-03-05 11:11:02 -07001015 def close_main_ssh(self):
1016 """Stop the ssh main connection.
1017
1018 Intended for situations when the host is known to be down and we don't
1019 need a ssh timeout to tell us it is down. For example, if you just
1020 instructed the host to shutdown or hibernate.
1021 """
1022 logging.debug("Stopping main ssh connection")
1023 self._main_ssh.close()
1024
Derek Beckett4d102242020-12-01 14:24:37 -08001025 def restart_main_ssh(self):
Luigi Semenzato3b95ede2016-12-09 11:51:01 -08001026 """
Derek Beckett4d102242020-12-01 14:24:37 -08001027 Stop and restart the ssh main connection. This is meant as a last
Luigi Semenzato3b95ede2016-12-09 11:51:01 -08001028 resort when ssh commands fail and we don't understand why.
1029 """
Congbin Guo306cdbc2021-02-08 13:37:41 -08001030 logging.debug("Restarting main ssh connection")
Derek Beckett4d102242020-12-01 14:24:37 -08001031 self._main_ssh.close()
1032 self._main_ssh.maybe_start(timeout=30)
Luigi Semenzato3b95ede2016-12-09 11:51:01 -08001033
Derek Beckett4d102242020-12-01 14:24:37 -08001034 def start_main_ssh(self, timeout=DEFAULT_START_MAIN_SSH_TIMEOUT_S):
mblighefccc1b2010-01-11 19:08:42 +00001035 """
Derek Beckett4d102242020-12-01 14:24:37 -08001036 Called whenever a non-main SSH connection needs to be initiated (e.g.,
1037 by run, rsync, scp). If main SSH support is enabled and a main SSH
mblighefccc1b2010-01-11 19:08:42 +00001038 connection is not active already, start a new one in the background.
Derek Beckett4d102242020-12-01 14:24:37 -08001039 Also, cleanup any zombie main SSH connections (e.g., dead due to
mblighefccc1b2010-01-11 19:08:42 +00001040 reboot).
Aviv Keshet0749a822013-10-17 09:53:26 -07001041
Derek Beckett4d102242020-12-01 14:24:37 -08001042 timeout: timeout in seconds (default 5) to wait for main ssh
Aviv Keshet0749a822013-10-17 09:53:26 -07001043 connection to be established. If timeout is reached, a
1044 warning message is logged, but no other action is taken.
mblighefccc1b2010-01-11 19:08:42 +00001045 """
Derek Beckett4d102242020-12-01 14:24:37 -08001046 if not enable_main_ssh:
mblighefccc1b2010-01-11 19:08:42 +00001047 return
Derek Beckett4d102242020-12-01 14:24:37 -08001048 self._main_ssh.maybe_start(timeout=timeout)
mbligh0a883702010-04-21 01:58:34 +00001049
Derek Beckettfccbb622021-02-08 16:44:53 -08001050 @property
1051 def tls_unstable(self):
1052 # A single test will rebuild remote many times. Its safe to assume if
1053 # TLS unstable for one try, it will be for others. If we check each,
1054 # it adds ~60 seconds per test (if its dead).
1055 if os.getenv('TLS_UNSTABLE'):
1056 return bool(os.getenv('TLS_UNSTABLE'))
1057 if self._tls_unstable is not None:
1058 return self._tls_unstable
Congbin Guo306cdbc2021-02-08 13:37:41 -08001059
Derek Beckettfccbb622021-02-08 16:44:53 -08001060 @tls_unstable.setter
1061 def tls_unstable(self, v):
1062 if not isinstance(v, bool):
1063 raise error.AutoservError('tls_stable setting must be bool, got %s'
1064 % (type(v)))
1065 os.environ['TLS_UNSTABLE'] = str(v)
1066 self._tls_unstable = v
Congbin Guo306cdbc2021-02-08 13:37:41 -08001067
Derek Beckettfccbb622021-02-08 16:44:53 -08001068 @property
1069 def tls_exec_dut_command_client(self):
1070 # If client is already initialized, return that.
Derek Beckett3b9974a2021-03-18 11:16:00 -07001071 if not ENABLE_EXEC_DUT_COMMAND:
1072 return None
Derek Beckettfccbb622021-02-08 16:44:53 -08001073 if self.tls_unstable:
1074 return None
1075 if self._tls_exec_dut_command_client is not None:
1076 return self._tls_exec_dut_command_client
1077 # If the TLS connection is alive, create a new client.
1078 if self.tls_connection is None:
1079 return None
1080 return exec_dut_command.TLSExecDutCommandClient(
1081 tlsconnection=self.tls_connection,
1082 hostname=self.hostname)
mbligh0a883702010-04-21 01:58:34 +00001083
1084 def clear_known_hosts(self):
1085 """Clears out the temporary ssh known_hosts file.
1086
1087 This is useful if the test SSHes to the machine, then reinstalls it,
1088 then SSHes to it again. It can be called after the reinstall to
1089 reduce the spam in the logs.
1090 """
1091 logging.info("Clearing known hosts for host '%s', file '%s'.",
Dean Liaoe3e75f62017-11-14 10:36:43 +08001092 self.host_port, self.known_hosts_file)
mbligh0a883702010-04-21 01:58:34 +00001093 # Clear out the file by opening it for writing and then closing.
Fang Deng3af66202013-08-16 15:19:25 -07001094 fh = open(self.known_hosts_file, "w")
mbligh0a883702010-04-21 01:58:34 +00001095 fh.close()
Prashanth B98509c72014-04-04 16:01:34 -07001096
1097
1098 def collect_logs(self, remote_src_dir, local_dest_dir, ignore_errors=True):
1099 """Copy log directories from a host to a local directory.
1100
1101 @param remote_src_dir: A destination directory on the host.
1102 @param local_dest_dir: A path to a local destination directory.
1103 If it doesn't exist it will be created.
1104 @param ignore_errors: If True, ignore exceptions.
1105
1106 @raises OSError: If there were problems creating the local_dest_dir and
1107 ignore_errors is False.
1108 @raises AutoservRunError, AutotestRunError: If something goes wrong
1109 while copying the directories and ignore_errors is False.
1110 """
Dan Shi9f92aa62017-07-27 17:07:05 -07001111 if not self.check_cached_up_status():
1112 logging.warning('Host %s did not answer to ping, skip collecting '
Dean Liaoe3e75f62017-11-14 10:36:43 +08001113 'logs.', self.host_port)
Dan Shi9f92aa62017-07-27 17:07:05 -07001114 return
1115
Prashanth B98509c72014-04-04 16:01:34 -07001116 locally_created_dest = False
1117 if (not os.path.exists(local_dest_dir)
1118 or not os.path.isdir(local_dest_dir)):
1119 try:
1120 os.makedirs(local_dest_dir)
1121 locally_created_dest = True
1122 except OSError as e:
1123 logging.warning('Unable to collect logs from host '
Dean Liaoe3e75f62017-11-14 10:36:43 +08001124 '%s: %s', self.host_port, e)
Prashanth B98509c72014-04-04 16:01:34 -07001125 if not ignore_errors:
1126 raise
1127 return
Dan Shi4f8c0242017-07-07 15:34:49 -07001128
1129 # Build test result directory summary
1130 try:
1131 result_tools_runner.run_on_client(self, remote_src_dir)
1132 except (error.AutotestRunError, error.AutoservRunError,
1133 error.AutoservSSHTimeout) as e:
1134 logging.exception(
1135 'Non-critical failure: Failed to collect and throttle '
Dean Liaoe3e75f62017-11-14 10:36:43 +08001136 'results at %s from host %s', remote_src_dir,
1137 self.host_port)
Dan Shi4f8c0242017-07-07 15:34:49 -07001138
Prashanth B98509c72014-04-04 16:01:34 -07001139 try:
Luigi Semenzato9b083072016-12-19 16:50:40 -08001140 self.get_file(remote_src_dir, local_dest_dir, safe_symlinks=True)
Prashanth B98509c72014-04-04 16:01:34 -07001141 except (error.AutotestRunError, error.AutoservRunError,
1142 error.AutoservSSHTimeout) as e:
1143 logging.warning('Collection of %s to local dir %s from host %s '
1144 'failed: %s', remote_src_dir, local_dest_dir,
Dean Liaoe3e75f62017-11-14 10:36:43 +08001145 self.host_port, e)
Prashanth B98509c72014-04-04 16:01:34 -07001146 if locally_created_dest:
1147 shutil.rmtree(local_dest_dir, ignore_errors=ignore_errors)
1148 if not ignore_errors:
1149 raise
Cheng-Yi Chianga155e7e2015-08-20 20:42:04 +08001150
Dan Shi4f8c0242017-07-07 15:34:49 -07001151 # Clean up directory summary file on the client side.
1152 try:
1153 result_tools_runner.run_on_client(self, remote_src_dir,
1154 cleanup_only=True)
1155 except (error.AutotestRunError, error.AutoservRunError,
1156 error.AutoservSSHTimeout) as e:
1157 logging.exception(
1158 'Non-critical failure: Failed to cleanup result summary '
Lutz Justen043e9c12017-10-27 12:40:47 +02001159 'files at %s in host %s', remote_src_dir, self.hostname)
Dan Shi4f8c0242017-07-07 15:34:49 -07001160
Cheng-Yi Chianga155e7e2015-08-20 20:42:04 +08001161
xixuan6cf6d2f2016-01-29 15:29:00 -08001162 def create_ssh_tunnel(self, port, local_port):
Cheng-Yi Chianga155e7e2015-08-20 20:42:04 +08001163 """Create an ssh tunnel from local_port to port.
1164
xixuan6cf6d2f2016-01-29 15:29:00 -08001165 This is used to forward a port securely through a tunnel process from
1166 the server to the DUT for RPC server connection.
1167
Cheng-Yi Chianga155e7e2015-08-20 20:42:04 +08001168 @param port: remote port on the host.
1169 @param local_port: local forwarding port.
1170
1171 @return: the tunnel process.
1172 """
1173 tunnel_options = '-n -N -q -L %d:localhost:%d' % (local_port, port)
Prathmesh Prabhu817b3f12017-07-31 17:08:41 -07001174 ssh_cmd = self.make_ssh_command(opts=tunnel_options, port=self.port)
Cheng-Yi Chianga155e7e2015-08-20 20:42:04 +08001175 tunnel_cmd = '%s %s' % (ssh_cmd, self.hostname)
1176 logging.debug('Full tunnel command: %s', tunnel_cmd)
xixuan6cf6d2f2016-01-29 15:29:00 -08001177 # Exec the ssh process directly here rather than using a shell.
1178 # Using a shell leaves a dangling ssh process, because we deliver
1179 # signals to the shell wrapping ssh, not the ssh process itself.
1180 args = shlex.split(tunnel_cmd)
Kuang-che Wu0ea03232019-08-31 10:52:31 +08001181 with open('/dev/null', 'w') as devnull:
1182 tunnel_proc = subprocess.Popen(args, stdout=devnull, stderr=devnull,
1183 close_fds=True)
Cheng-Yi Chianga155e7e2015-08-20 20:42:04 +08001184 logging.debug('Started ssh tunnel, local = %d'
1185 ' remote = %d, pid = %d',
1186 local_port, port, tunnel_proc.pid)
1187 return tunnel_proc
Gilad Arnolda76bef02015-09-29 13:55:15 -07001188
1189
Oleg Loskutoff1199bbb2019-10-21 12:27:13 -07001190 def disconnect_ssh_tunnel(self, tunnel_proc):
Roshan Pius58e5dd32015-10-16 15:16:42 -07001191 """
1192 Disconnects a previously forwarded port from the server to the DUT for
1193 RPC server connection.
1194
xixuan6cf6d2f2016-01-29 15:29:00 -08001195 @param tunnel_proc: a tunnel process returned from |create_ssh_tunnel|.
Roshan Pius58e5dd32015-10-16 15:16:42 -07001196 """
1197 if tunnel_proc.poll() is None:
1198 tunnel_proc.terminate()
1199 logging.debug('Terminated tunnel, pid %d', tunnel_proc.pid)
1200 else:
1201 logging.debug('Tunnel pid %d terminated early, status %d',
1202 tunnel_proc.pid, tunnel_proc.returncode)
1203
1204
Gilad Arnolda76bef02015-09-29 13:55:15 -07001205 def get_os_type(self):
1206 """Returns the host OS descriptor (to be implemented in subclasses).
1207
1208 @return A string describing the OS type.
1209 """
Gwendal Grignou36b61702016-02-10 11:57:53 -08001210 raise NotImplementedError
Dan Shi9f92aa62017-07-27 17:07:05 -07001211
1212
1213 def check_cached_up_status(
1214 self, expiration_seconds=_DEFAULT_UP_STATUS_EXPIRATION_SECONDS):
1215 """Check if the DUT responded to ping in the past `expiration_seconds`.
1216
1217 @param expiration_seconds: The number of seconds to keep the cached
1218 status of whether the DUT responded to ping.
1219 @return: True if the DUT has responded to ping during the past
1220 `expiration_seconds`.
1221 """
1222 # Refresh the up status if any of following conditions is true:
1223 # * cached status is never set
1224 # * cached status is False, so the method can check if the host is up
1225 # again.
1226 # * If the cached status is older than `expiration_seconds`
Andrew Luo4be621d2020-03-21 07:01:13 -07001227 # If we have icmp disabled, treat that as a cached ping.
1228 if not self._use_icmp:
1229 return True
Dan Shi9f92aa62017-07-27 17:07:05 -07001230 expire_time = time.time() - expiration_seconds
1231 if (self._cached_up_status_updated is None or
1232 not self._cached_up_status or
1233 self._cached_up_status_updated < expire_time):
1234 self._cached_up_status = self.is_up_fast()
1235 self._cached_up_status_updated = time.time()
1236 return self._cached_up_status
Otabek Kasimovf5e0f102020-06-30 19:41:02 -07001237
1238
1239 def _track_class_usage(self):
1240 """Tracking which class was used.
1241
1242 The idea to identify unused classes to be able clean them up.
1243 We skip names with dynamic created classes where the name is
1244 hostname of the device.
1245 """
1246 class_name = None
1247 if 'chrome' not in self.__class__.__name__:
1248 class_name = self.__class__.__name__
1249 else:
1250 for base in self.__class__.__bases__:
1251 if 'chrome' not in base.__name__:
1252 class_name = base.__name__
1253 break
1254 if class_name:
1255 data = {'host_class': class_name}
1256 metrics.Counter(
1257 'chromeos/autotest/used_hosts').increment(fields=data)
Garry Wang81bea592020-08-27 17:25:25 -07001258
1259 def is_file_exists(self, file_path):
1260 """Check whether a given file is exist on the host.
1261 """
1262 result = self.run('test -f ' + file_path,
1263 timeout=30,
1264 ignore_status=True)
1265 return result.exit_status == 0
Derek Beckettfc04fbc2020-12-14 16:37:31 -08001266
1267
1268def _client_symlink(sources):
1269 """Return the client symlink if in sources."""
1270 for source in sources:
1271 if source.endswith(AUTOTEST_CLIENT_SYMLINK_END):
1272 return source
1273 return None