blob: efaa6b3db1c2e616c2f4f0b6f400fcb4f0781c9c [file] [log] [blame]
maruel@chromium.org9c72d4e2012-09-28 19:20:25 +00001#!/usr/bin/env python
maruelea586f32016-04-05 11:11:33 -07002# Copyright 2012 The LUCI Authors. All rights reserved.
maruelf1f5e2a2016-05-25 17:10:39 -07003# Use of this source code is governed under the Apache License, Version 2.0
4# that can be found in the LICENSE file.
maruel@chromium.org9c72d4e2012-09-28 19:20:25 +00005
nodir55be77b2016-05-03 09:39:57 -07006"""Runs a command with optional isolated input/output.
maruel@chromium.org9c72d4e2012-09-28 19:20:25 +00007
Marc-Antoine Rueleed2f3a2019-03-14 00:00:40 +00008run_isolated takes cares of setting up a temporary environment, running a
9command, and tearing it down.
nodir55be77b2016-05-03 09:39:57 -070010
Marc-Antoine Rueleed2f3a2019-03-14 00:00:40 +000011It handles downloading and uploading isolated files, mapping CIPD packages and
12reusing stateful named caches.
13
14The isolated files, CIPD packages and named caches are kept as a global LRU
15cache.
Marc-Antoine Ruel2283ad12014-02-09 11:14:57 -050016
Roberto Carrillo71ade6d2018-10-08 22:30:24 +000017Any ${EXECUTABLE_SUFFIX} on the command line or the environment variables passed
18with the --env option will be replaced with ".exe" string on Windows and "" on
19other platforms.
nodirbe642ff2016-06-09 15:51:51 -070020
Roberto Carrillo71ade6d2018-10-08 22:30:24 +000021Any ${ISOLATED_OUTDIR} on the command line or the environment variables passed
22with the --env option will be replaced by the location of a temporary directory
23upon execution of the command specified in the .isolated file. All content
24written to this directory will be uploaded upon termination and the .isolated
25file describing this directory will be printed to stdout.
bpastene447c1992016-06-20 15:21:47 -070026
Marc-Antoine Rueleed2f3a2019-03-14 00:00:40 +000027Any ${SWARMING_BOT_FILE} on the command line or the environment variables passed
28with the --env option will be replaced by the value of the --bot-file parameter.
29This file is used by a swarming bot to communicate state of the host to tasks.
30It is written to by the swarming bot's on_before_task() hook in the swarming
31server's custom bot_config.py.
32
33See
34https://chromium.googlesource.com/infra/luci/luci-py.git/+/master/appengine/swarming/doc/Magic-Values.md
35for all the variables.
36
37See
38https://chromium.googlesource.com/infra/luci/luci-py.git/+/master/appengine/swarming/swarming_bot/config/bot_config.py
39for more information about bot_config.py.
maruel@chromium.org9c72d4e2012-09-28 19:20:25 +000040"""
41
Marc-Antoine Ruelf899c482019-10-10 23:32:06 +000042from __future__ import print_function
43
44__version__ = '1.0.1'
maruel@chromium.orgdedbf492013-09-12 20:42:11 +000045
aludwin7556e0c2016-10-26 08:46:10 -070046import argparse
maruel064c0a32016-04-05 11:47:15 -070047import base64
iannucci96fcccc2016-08-30 15:52:22 -070048import collections
vadimsh232f5a82017-01-20 19:23:44 -080049import contextlib
Ye Kuangfff1e502020-07-13 13:21:57 +000050import distutils
Sadaf Matinkhoo10743a62018-03-29 16:28:58 -040051import errno
aludwin7556e0c2016-10-26 08:46:10 -070052import json
maruel@chromium.org9c72d4e2012-09-28 19:20:25 +000053import logging
54import optparse
55import os
Takuto Ikuta5c59a842020-01-24 03:05:24 +000056import platform
Marc-Antoine Ruel8b11dbd2018-05-18 14:31:22 -040057import re
maruel@chromium.org9c72d4e2012-09-28 19:20:25 +000058import sys
59import tempfile
maruel064c0a32016-04-05 11:47:15 -070060import time
maruel@chromium.org9c72d4e2012-09-28 19:20:25 +000061
Marc-Antoine Ruel016c7602019-04-02 18:31:13 +000062from utils import tools
63tools.force_local_third_party()
vadimsh@chromium.orga4326472013-08-24 02:05:41 +000064
Marc-Antoine Ruel016c7602019-04-02 18:31:13 +000065# third_party/
66from depot_tools import fix_encoding
Takuto Ikuta6e2ff962019-10-29 12:35:27 +000067import six
Marc-Antoine Ruel016c7602019-04-02 18:31:13 +000068
69# pylint: disable=ungrouped-imports
70import auth
71import cipd
72import isolate_storage
73import isolateserver
74import local_caching
75from libs import luci_context
Vadim Shtayura6b555c12014-07-23 16:22:18 -070076from utils import file_path
maruel12e30012015-10-09 11:55:35 -070077from utils import fs
maruel064c0a32016-04-05 11:47:15 -070078from utils import large
Marc-Antoine Ruelf74cffe2015-07-15 15:21:34 -040079from utils import logging_utils
Ye Kuang2dd17442020-04-22 08:45:52 +000080from utils import net
Marc-Antoine Ruelcfb60852014-07-02 15:22:00 -040081from utils import on_error
Marc-Antoine Ruelc44f5722015-01-08 16:10:01 -050082from utils import subprocess42
maruel@chromium.orgdedbf492013-09-12 20:42:11 +000083
vadimsh@chromium.orga4326472013-08-24 02:05:41 +000084
maruele2f2cb82016-07-13 14:41:03 -070085# Magic variables that can be found in the isolate task command line.
86ISOLATED_OUTDIR_PARAMETER = '${ISOLATED_OUTDIR}'
87EXECUTABLE_SUFFIX_PARAMETER = '${EXECUTABLE_SUFFIX}'
88SWARMING_BOT_FILE_PARAMETER = '${SWARMING_BOT_FILE}'
89
90
csharp@chromium.orgff2a4662012-11-21 20:49:32 +000091# The name of the log file to use.
92RUN_ISOLATED_LOG_FILE = 'run_isolated.log'
93
maruele2f2cb82016-07-13 14:41:03 -070094
csharp@chromium.orge217f302012-11-22 16:51:53 +000095# The name of the log to use for the run_test_cases.py command
vadimsh@chromium.org8b9d56b2013-08-21 22:24:35 +000096RUN_TEST_CASES_LOG = 'run_test_cases.log'
csharp@chromium.orge217f302012-11-22 16:51:53 +000097
vadimsh@chromium.org87d63262013-04-04 19:34:21 +000098
maruele2f2cb82016-07-13 14:41:03 -070099# Use short names for temporary directories. This is driven by Windows, which
100# imposes a relatively short maximum path length of 260 characters, often
101# referred to as MAX_PATH. It is relatively easy to create files with longer
Marc-Antoine Ruel793bff32019-04-18 17:50:48 +0000102# path length. A use case is with recursive dependency trees like npm packages.
maruele2f2cb82016-07-13 14:41:03 -0700103#
104# It is recommended to start the script with a `root_dir` as short as
105# possible.
106# - ir stands for isolated_run
107# - io stands for isolated_out
108# - it stands for isolated_tmp
Takuto Ikutab7ce0e32019-11-27 23:26:18 +0000109# - ic stands for isolated_client
maruele2f2cb82016-07-13 14:41:03 -0700110ISOLATED_RUN_DIR = u'ir'
111ISOLATED_OUT_DIR = u'io'
112ISOLATED_TMP_DIR = u'it'
Takuto Ikutab7ce0e32019-11-27 23:26:18 +0000113ISOLATED_CLIENT_DIR = u'ic'
maruele2f2cb82016-07-13 14:41:03 -0700114
Takuto Ikuta02edca22019-11-29 10:04:51 +0000115# TODO(tikuta): take these parameter from luci-config?
Takuto Ikutac8c92e62020-04-01 07:07:29 +0000116# Update tag by `./client/update_isolated.sh`.
117# Or take revision from
Takuto Ikutab7ce0e32019-11-27 23:26:18 +0000118# https://ci.chromium.org/p/infra-internal/g/infra-packagers/console
Takuto Ikuta02edca22019-11-29 10:04:51 +0000119ISOLATED_PACKAGE = 'infra/tools/luci/isolated/${platform}'
Takuto Ikutafd59b822020-06-24 07:43:05 +0000120ISOLATED_REVISION = 'git_revision:58b9c29f4c1aedddc1da81481be3a5e08b18234e'
maruele2f2cb82016-07-13 14:41:03 -0700121
Marc-Antoine Ruel8b11dbd2018-05-18 14:31:22 -0400122# Keep synced with task_request.py
Lei Leife202df2019-06-11 17:33:34 +0000123CACHE_NAME_RE = re.compile(r'^[a-z0-9_]{1,4096}$')
Marc-Antoine Ruel8b11dbd2018-05-18 14:31:22 -0400124
125
marueld928c862017-06-08 08:20:04 -0700126OUTLIVING_ZOMBIE_MSG = """\
127*** Swarming tried multiple times to delete the %s directory and failed ***
128*** Hard failing the task ***
129
130Swarming detected that your testing script ran an executable, which may have
131started a child executable, and the main script returned early, leaving the
132children executables playing around unguided.
133
134You don't want to leave children processes outliving the task on the Swarming
135bot, do you? The Swarming bot doesn't.
136
137How to fix?
138- For any process that starts children processes, make sure all children
139 processes terminated properly before each parent process exits. This is
140 especially important in very deep process trees.
141 - This must be done properly both in normal successful task and in case of
142 task failure. Cleanup is very important.
143- The Swarming bot sends a SIGTERM in case of timeout.
144 - You have %s seconds to comply after the signal was sent to the process
145 before the process is forcibly killed.
146- To achieve not leaking children processes in case of signals on timeout, you
147 MUST handle signals in each executable / python script and propagate them to
148 children processes.
149 - When your test script (python or binary) receives a signal like SIGTERM or
150 CTRL_BREAK_EVENT on Windows), send it to all children processes and wait for
151 them to terminate before quitting.
152
153See
Marc-Antoine Ruelc7243592018-05-24 17:04:04 -0400154https://chromium.googlesource.com/infra/luci/luci-py.git/+/master/appengine/swarming/doc/Bot.md#Graceful-termination_aka-the-SIGTERM-and-SIGKILL-dance
marueld928c862017-06-08 08:20:04 -0700155for more information.
156
157*** May the SIGKILL force be with you ***
158"""
159
160
Marc-Antoine Ruel5d7606b2018-06-15 19:06:12 +0000161# Currently hardcoded. Eventually could be exposed as a flag once there's value.
162# 3 weeks
163MAX_AGE_SECS = 21*24*60*60
164
165
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -0500166TaskData = collections.namedtuple(
Marc-Antoine Ruel03c6fd12019-04-30 12:12:55 +0000167 'TaskData',
168 [
Takuto Ikuta9a319502019-11-26 07:40:14 +0000169 # List of strings; the command line to use, independent of what was
170 # specified in the isolated file.
171 'command',
172 # Relative directory to start command into.
173 'relative_cwd',
174 # List of strings; the arguments to add to the command specified in the
175 # isolated file.
176 'extra_args',
177 # Hash of the .isolated file that must be retrieved to recreate the tree
178 # of files to run the target executable. The command specified in the
179 # .isolated is executed. Mutually exclusive with command argument.
180 'isolated_hash',
181 # isolateserver.Storage instance to retrieve remote objects. This object
182 # has a reference to an isolateserver.StorageApi, which does the actual
183 # I/O.
184 'storage',
185 # isolateserver.LocalCache instance to keep from retrieving the same
186 # objects constantly by caching the objects retrieved. Can be on-disk or
187 # in-memory.
188 'isolate_cache',
189 # List of paths relative to root_dir to put into the output isolated
190 # bundle upon task completion (see link_outputs_to_outdir).
191 'outputs',
192 # Function (run_dir) => context manager that installs named caches into
193 # |run_dir|.
194 'install_named_caches',
195 # If True, the temporary directory will be deliberately leaked for later
196 # examination.
197 'leak_temp_dir',
198 # Path to the directory to use to create the temporary directory. If not
199 # specified, a random temporary directory is created.
200 'root_dir',
201 # Kills the process if it lasts more than this amount of seconds.
202 'hard_timeout',
203 # Number of seconds to wait between SIGTERM and SIGKILL.
204 'grace_period',
205 # Path to a file with bot state, used in place of ${SWARMING_BOT_FILE}
206 # task command line argument.
207 'bot_file',
208 # Logical account to switch LUCI_CONTEXT into.
209 'switch_to_account',
210 # Context manager dir => CipdInfo, see install_client_and_packages.
211 'install_packages_fn',
Takuto Ikutad03ffcc2019-12-02 01:04:23 +0000212 # Use go isolated client.
213 'use_go_isolated',
Takuto Ikuta057c5342019-12-03 04:05:05 +0000214 # Cache directory for go isolated client.
215 'go_cache_dir',
Takuto Ikuta879788c2020-01-10 08:00:26 +0000216 # Parameters passed to go isolated client.
217 'go_cache_policies',
Takuto Ikuta9a319502019-11-26 07:40:14 +0000218 # Environment variables to set.
219 'env',
220 # Environment variables to mutate with relative directories.
221 # Example: {"ENV_KEY": ['relative', 'paths', 'to', 'prepend']}
222 'env_prefix',
223 # Lowers the task process priority.
224 'lower_priority',
225 # subprocess42.Containment instance. Can be None.
226 'containment',
Marc-Antoine Ruel03c6fd12019-04-30 12:12:55 +0000227 ])
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -0500228
229
Marc-Antoine Ruelee6ca622017-11-29 11:19:16 -0500230def _to_str(s):
231 """Downgrades a unicode instance to str. Pass str through as-is."""
232 if isinstance(s, str):
233 return s
234 # This is technically incorrect, especially on Windows. In theory
235 # sys.getfilesystemencoding() should be used to use the right 'ANSI code
236 # page' on Windows, but that causes other problems, as the character set
237 # is very limited.
238 return s.encode('utf-8')
239
240
Marc-Antoine Ruel7a68f712017-12-01 18:45:18 -0500241def _to_unicode(s):
242 """Upgrades a str instance to unicode. Pass unicode through as-is."""
Takuto Ikuta95459dd2019-10-29 12:39:47 +0000243 if isinstance(s, six.text_type) or s is None:
Marc-Antoine Ruel7a68f712017-12-01 18:45:18 -0500244 return s
245 return s.decode('utf-8')
246
247
maruel03e11842016-07-14 10:50:16 -0700248def make_temp_dir(prefix, root_dir):
249 """Returns a new unique temporary directory."""
Takuto Ikuta6e2ff962019-10-29 12:35:27 +0000250 return six.text_type(tempfile.mkdtemp(prefix=prefix, dir=root_dir))
maruel@chromium.org9c72d4e2012-09-28 19:20:25 +0000251
252
vadimsh9c54b2c2017-07-25 14:08:29 -0700253@contextlib.contextmanager
254def set_luci_context_account(account, tmp_dir):
255 """Sets LUCI_CONTEXT account to be used by the task.
256
257 If 'account' is None or '', does nothing at all. This happens when
258 run_isolated.py is called without '--switch-to-account' flag. In this case,
259 if run_isolated.py is running in some LUCI_CONTEXT environment, the task will
Takuto Ikuta33e2ff32019-09-30 12:44:03 +0000260 just inherit whatever account is already set. This may happen if users invoke
vadimsh9c54b2c2017-07-25 14:08:29 -0700261 run_isolated.py explicitly from their code.
262
263 If the requested account is not defined in the context, switches to
264 non-authenticated access. This happens for Swarming tasks that don't use
265 'task' service accounts.
266
267 If not using LUCI_CONTEXT-based auth, does nothing.
268 If already running as requested account, does nothing.
269 """
270 if not account:
271 # Not actually switching.
272 yield
273 return
274
275 local_auth = luci_context.read('local_auth')
276 if not local_auth:
277 # Not using LUCI_CONTEXT auth at all.
278 yield
279 return
280
281 # See LUCI_CONTEXT.md for the format of 'local_auth'.
282 if local_auth.get('default_account_id') == account:
283 # Already set, no need to switch.
284 yield
285 return
286
287 available = {a['id'] for a in local_auth.get('accounts') or []}
288 if account in available:
289 logging.info('Switching default LUCI_CONTEXT account to %r', account)
290 local_auth['default_account_id'] = account
291 else:
292 logging.warning(
293 'Requested LUCI_CONTEXT account %r is not available (have only %r), '
294 'disabling authentication', account, sorted(available))
295 local_auth.pop('default_account_id', None)
296
297 with luci_context.write(_tmpdir=tmp_dir, local_auth=local_auth):
298 yield
299
300
nodir90bc8dc2016-06-15 13:35:21 -0700301def process_command(command, out_dir, bot_file):
Roberto Carrillo71ade6d2018-10-08 22:30:24 +0000302 """Replaces parameters in a command line.
nodirbe642ff2016-06-09 15:51:51 -0700303
304 Raises:
305 ValueError if a parameter is requested in |command| but its value is not
306 provided.
307 """
Roberto Carrillo71ade6d2018-10-08 22:30:24 +0000308 return [replace_parameters(arg, out_dir, bot_file) for arg in command]
309
310
311def replace_parameters(arg, out_dir, bot_file):
312 """Replaces parameter tokens with appropriate values in a string.
313
314 Raises:
315 ValueError if a parameter is requested in |arg| but its value is not
316 provided.
317 """
318 arg = arg.replace(EXECUTABLE_SUFFIX_PARAMETER, cipd.EXECUTABLE_SUFFIX)
319 replace_slash = False
320 if ISOLATED_OUTDIR_PARAMETER in arg:
321 if not out_dir:
322 raise ValueError(
323 'output directory is requested in command or env var, but not '
324 'provided; please specify one')
325 arg = arg.replace(ISOLATED_OUTDIR_PARAMETER, out_dir)
326 replace_slash = True
327 if SWARMING_BOT_FILE_PARAMETER in arg:
328 if bot_file:
329 arg = arg.replace(SWARMING_BOT_FILE_PARAMETER, bot_file)
nodirbe642ff2016-06-09 15:51:51 -0700330 replace_slash = True
Roberto Carrillo71ade6d2018-10-08 22:30:24 +0000331 else:
332 logging.warning('SWARMING_BOT_FILE_PARAMETER found in command or env '
333 'var, but no bot_file specified. Leaving parameter '
334 'unchanged.')
335 if replace_slash:
336 # Replace slashes only if parameters are present
337 # because of arguments like '${ISOLATED_OUTDIR}/foo/bar'
338 arg = arg.replace('/', os.sep)
339 return arg
maruela9cfd6f2015-09-15 11:03:15 -0700340
341
Roberto Carrillo71ade6d2018-10-08 22:30:24 +0000342
343def get_command_env(tmp_dir, cipd_info, run_dir, env, env_prefixes, out_dir,
344 bot_file):
vadimsh232f5a82017-01-20 19:23:44 -0800345 """Returns full OS environment to run a command in.
346
Robert Iannuccibf5f84c2017-11-22 12:56:50 -0800347 Sets up TEMP, puts directory with cipd binary in front of PATH, exposes
348 CIPD_CACHE_DIR env var, and installs all env_prefixes.
vadimsh232f5a82017-01-20 19:23:44 -0800349
350 Args:
351 tmp_dir: temp directory.
352 cipd_info: CipdInfo object is cipd client is used, None if not.
Marc-Antoine Ruel9ec1e9f2017-12-20 16:36:54 -0500353 run_dir: The root directory the isolated tree is mapped in.
Marc-Antoine Ruel19dd8872017-11-28 18:33:39 -0500354 env: environment variables to use
Robert Iannuccibf5f84c2017-11-22 12:56:50 -0800355 env_prefixes: {"ENV_KEY": ['cwd', 'relative', 'paths', 'to', 'prepend']}
Roberto Carrillo71ade6d2018-10-08 22:30:24 +0000356 out_dir: Isolated output directory. Required to be != None if any of the
357 env vars contain ISOLATED_OUTDIR_PARAMETER.
358 bot_file: Required to be != None if any of the env vars contain
359 SWARMING_BOT_FILE_PARAMETER.
vadimsh232f5a82017-01-20 19:23:44 -0800360 """
Marc-Antoine Ruel19dd8872017-11-28 18:33:39 -0500361 out = os.environ.copy()
Marc-Antoine Ruel04903a32019-10-09 21:09:25 +0000362 for k, v in env.items():
Marc-Antoine Ruel19dd8872017-11-28 18:33:39 -0500363 if not v:
Marc-Antoine Ruel9ec1e9f2017-12-20 16:36:54 -0500364 out.pop(k, None)
Marc-Antoine Ruel19dd8872017-11-28 18:33:39 -0500365 else:
Roberto Carrillo71ade6d2018-10-08 22:30:24 +0000366 out[k] = replace_parameters(v, out_dir, bot_file)
Marc-Antoine Ruel19dd8872017-11-28 18:33:39 -0500367
368 if cipd_info:
369 bin_dir = os.path.dirname(cipd_info.client.binary_path)
Marc-Antoine Ruelee6ca622017-11-29 11:19:16 -0500370 out['PATH'] = '%s%s%s' % (_to_str(bin_dir), os.pathsep, out['PATH'])
371 out['CIPD_CACHE_DIR'] = _to_str(cipd_info.cache_dir)
Marc-Antoine Ruel19dd8872017-11-28 18:33:39 -0500372
Marc-Antoine Ruel04903a32019-10-09 21:09:25 +0000373 for key, paths in env_prefixes.items():
Marc-Antoine Ruel9ec1e9f2017-12-20 16:36:54 -0500374 assert isinstance(paths, list), paths
375 paths = [os.path.normpath(os.path.join(run_dir, p)) for p in paths]
Marc-Antoine Ruel19dd8872017-11-28 18:33:39 -0500376 cur = out.get(key)
377 if cur:
378 paths.append(cur)
Marc-Antoine Ruelee6ca622017-11-29 11:19:16 -0500379 out[key] = _to_str(os.path.pathsep.join(paths))
vadimsh232f5a82017-01-20 19:23:44 -0800380
Marc-Antoine Ruelefb30b12018-07-25 18:34:36 +0000381 tmp_dir = _to_str(tmp_dir)
382 # pylint: disable=line-too-long
383 # * python respects $TMPDIR, $TEMP, and $TMP in this order, regardless of
384 # platform. So $TMPDIR must be set on all platforms.
385 # https://github.com/python/cpython/blob/2.7/Lib/tempfile.py#L155
386 out['TMPDIR'] = tmp_dir
387 if sys.platform == 'win32':
388 # * chromium's base utils uses GetTempPath().
389 # https://cs.chromium.org/chromium/src/base/files/file_util_win.cc?q=GetTempPath
390 # * Go uses GetTempPath().
391 # * GetTempDir() uses %TMP%, then %TEMP%, then other stuff. So %TMP% must be
392 # set.
393 # https://docs.microsoft.com/en-us/windows/desktop/api/fileapi/nf-fileapi-gettemppathw
394 out['TMP'] = tmp_dir
395 # https://blogs.msdn.microsoft.com/oldnewthing/20150417-00/?p=44213
396 out['TEMP'] = tmp_dir
397 elif sys.platform == 'darwin':
398 # * Chromium uses an hack on macOS before calling into
399 # NSTemporaryDirectory().
400 # https://cs.chromium.org/chromium/src/base/files/file_util_mac.mm?q=GetTempDir
401 # https://developer.apple.com/documentation/foundation/1409211-nstemporarydirectory
402 out['MAC_CHROMIUM_TMPDIR'] = tmp_dir
403 else:
404 # TMPDIR is specified as the POSIX standard envvar for the temp directory.
405 # http://pubs.opengroup.org/onlinepubs/9699919799/basedefs/V1_chap08.html
406 # * mktemp on linux respects $TMPDIR.
407 # * Chromium respects $TMPDIR on linux.
408 # https://cs.chromium.org/chromium/src/base/files/file_util_posix.cc?q=GetTempDir
409 # * Go uses $TMPDIR.
410 # https://go.googlesource.com/go/+/go1.10.3/src/os/file_unix.go#307
411 pass
Marc-Antoine Ruel19dd8872017-11-28 18:33:39 -0500412 return out
vadimsh232f5a82017-01-20 19:23:44 -0800413
414
Marc-Antoine Ruel1b65f4e2019-05-02 21:56:58 +0000415def run_command(
416 command, cwd, env, hard_timeout, grace_period, lower_priority, containment):
maruel6be7f9e2015-10-01 12:25:30 -0700417 """Runs the command.
418
419 Returns:
420 tuple(process exit code, bool if had a hard timeout)
421 """
Marc-Antoine Ruel03c6fd12019-04-30 12:12:55 +0000422 logging.info(
Marc-Antoine Ruel1b65f4e2019-05-02 21:56:58 +0000423 'run_command(%s, %s, %s, %s, %s, %s)',
424 command, cwd, hard_timeout, grace_period, lower_priority, containment)
marueleb5fbee2015-09-17 13:01:36 -0700425
maruel6be7f9e2015-10-01 12:25:30 -0700426 exit_code = None
427 had_hard_timeout = False
maruela9cfd6f2015-09-15 11:03:15 -0700428 with tools.Profiler('RunTest'):
maruel6be7f9e2015-10-01 12:25:30 -0700429 proc = None
430 had_signal = []
maruela9cfd6f2015-09-15 11:03:15 -0700431 try:
maruel6be7f9e2015-10-01 12:25:30 -0700432 # TODO(maruel): This code is imperfect. It doesn't handle well signals
433 # during the download phase and there's short windows were things can go
434 # wrong.
435 def handler(signum, _frame):
436 if proc and not had_signal:
437 logging.info('Received signal %d', signum)
438 had_signal.append(True)
maruel556d9052015-10-05 11:12:44 -0700439 raise subprocess42.TimeoutExpired(command, None)
maruel6be7f9e2015-10-01 12:25:30 -0700440
Marc-Antoine Ruel30b80fe2019-02-08 13:51:31 +0000441 proc = subprocess42.Popen(
Marc-Antoine Ruel03c6fd12019-04-30 12:12:55 +0000442 command, cwd=cwd, env=env, detached=True, close_fds=True,
Marc-Antoine Ruel1b65f4e2019-05-02 21:56:58 +0000443 lower_priority=lower_priority, containment=containment)
maruel6be7f9e2015-10-01 12:25:30 -0700444 with subprocess42.set_signal_handler(subprocess42.STOP_SIGNALS, handler):
445 try:
John Budorickc398f092019-06-10 22:49:44 +0000446 exit_code = proc.wait(hard_timeout or None)
maruel6be7f9e2015-10-01 12:25:30 -0700447 except subprocess42.TimeoutExpired:
448 if not had_signal:
449 logging.warning('Hard timeout')
450 had_hard_timeout = True
451 logging.warning('Sending SIGTERM')
452 proc.terminate()
453
454 # Ignore signals in grace period. Forcibly give the grace period to the
455 # child process.
456 if exit_code is None:
457 ignore = lambda *_: None
458 with subprocess42.set_signal_handler(subprocess42.STOP_SIGNALS, ignore):
459 try:
460 exit_code = proc.wait(grace_period or None)
461 except subprocess42.TimeoutExpired:
462 # Now kill for real. The user can distinguish between the
463 # following states:
464 # - signal but process exited within grace period,
465 # hard_timed_out will be set but the process exit code will be
466 # script provided.
467 # - processed exited late, exit code will be -9 on posix.
468 logging.warning('Grace exhausted; sending SIGKILL')
469 proc.kill()
martiniss5c8043e2017-08-01 17:09:43 -0700470 logging.info('Waiting for process exit')
maruel6be7f9e2015-10-01 12:25:30 -0700471 exit_code = proc.wait()
Takuto Ikutaeccf0862020-03-19 03:05:55 +0000472 except OSError as e:
maruela9cfd6f2015-09-15 11:03:15 -0700473 # This is not considered to be an internal error. The executable simply
474 # does not exit.
maruela72f46e2016-02-24 11:05:45 -0800475 sys.stderr.write(
tikuta2d678212019-09-23 23:12:08 +0000476 '<The executable does not exist, a dependent library is missing or '
477 'the command line is too long>\n'
478 '<Check for missing .so/.dll in the .isolate or GN file or length of '
479 'command line args>\n'
Takuto Ikutaeccf0862020-03-19 03:05:55 +0000480 '<Command: %s, Exception: %s>\n' % (command, e))
maruela72f46e2016-02-24 11:05:45 -0800481 if os.environ.get('SWARMING_TASK_ID'):
482 # Give an additional hint when running as a swarming task.
483 sys.stderr.write(
484 '<See the task\'s page for commands to help diagnose this issue '
485 'by reproducing the task locally>\n')
maruela9cfd6f2015-09-15 11:03:15 -0700486 exit_code = 1
487 logging.info(
488 'Command finished with exit code %d (%s)',
489 exit_code, hex(0xffffffff & exit_code))
maruel6be7f9e2015-10-01 12:25:30 -0700490 return exit_code, had_hard_timeout
maruela9cfd6f2015-09-15 11:03:15 -0700491
492
Ye Kuangc0cf9ca2020-07-16 08:56:51 +0000493def _run_go_isolated_and_wait(cmd):
494 """
495 Runs a Go `isolated` command and wait for its completion.
496
497 While this is a generic function to launch a subprocess, it has logic that
498 is specific to Go `isolated` for waiting and logging.
499
500 Returns:
501 The subprocess object
502 """
503 proc = subprocess42.Popen(cmd)
504 cmd_str = ' '.join(cmd)
505
506 exceeded_max_timeout = True
507 check_period_sec = 30
508 max_checks = 100
509 # max timeout = max_checks * check_period_sec = 50 minutes
510 for i in range(max_checks):
511 # This is to prevent I/O timeout error during isolated setup.
512 try:
513 retcode = proc.wait(check_period_sec)
514 if retcode != 0:
515 raise ValueError("retcode is not 0: %s (cmd=%s)" % (retcode, cmd_str))
516 exceeded_max_timeout = False
517 break
518 except subprocess42.TimeoutExpired:
519 print('still running isolated (after %d seconds)' %
520 ((i + 1) * check_period_sec))
521
522 if exceeded_max_timeout:
523 proc.terminate()
524 try:
525 proc.wait(check_period_sec)
526 except subprocess42.TimeoutExpired:
527 logging.exception(
528 "failed to terminate? timeout happened after %d seconds",
529 check_period_sec)
530 proc.kill()
531 proc.wait()
532 # Raise unconditionally, because |proc| was forcefully terminated.
533 raise ValueError("timedout after %d seconds (cmd=%s)" %
534 (check_period_sec * max_checks, cmd_str))
535
536 return proc
537
538
Takuto Ikuta879788c2020-01-10 08:00:26 +0000539def _fetch_and_map_with_go(isolated_hash, storage, outdir, go_cache_dir,
540 policies, isolated_client):
Takuto Ikutad03ffcc2019-12-02 01:04:23 +0000541 """
542 Fetches an isolated tree using go client, create the tree and returns
543 (bundle, stats).
544 """
545 start = time.time()
546 server_ref = storage.server_ref
Takuto Ikutad03ffcc2019-12-02 01:04:23 +0000547 result_json_handle, result_json_path = tempfile.mkstemp(
548 prefix=u'fetch-and-map-result-', suffix=u'.json')
549 os.close(result_json_handle)
550 try:
Ye Kuanga98764c2020-04-09 03:17:37 +0000551 cmd = [
Takuto Ikutad03ffcc2019-12-02 01:04:23 +0000552 isolated_client,
553 'download',
554 '-isolate-server',
555 server_ref.url,
556 '-namespace',
557 server_ref.namespace,
558 '-isolated',
559 isolated_hash,
560
561 # flags for cache
562 '-cache-dir',
Takuto Ikuta057c5342019-12-03 04:05:05 +0000563 go_cache_dir,
Takuto Ikutad03ffcc2019-12-02 01:04:23 +0000564 '-cache-max-items',
Takuto Ikuta50bc0552019-12-03 03:26:46 +0000565 str(policies.max_items),
Takuto Ikutad03ffcc2019-12-02 01:04:23 +0000566 '-cache-max-size',
Takuto Ikuta50bc0552019-12-03 03:26:46 +0000567 str(policies.max_cache_size),
Takuto Ikutad03ffcc2019-12-02 01:04:23 +0000568 '-cache-min-free-space',
Takuto Ikuta50bc0552019-12-03 03:26:46 +0000569 str(policies.min_free_space),
Takuto Ikutad03ffcc2019-12-02 01:04:23 +0000570
571 # flags for output
572 '-output-dir',
573 outdir,
574 '-fetch-and-map-result-json',
575 result_json_path,
Ye Kuanga98764c2020-04-09 03:17:37 +0000576 ]
Ye Kuangc0cf9ca2020-07-16 08:56:51 +0000577 _run_go_isolated_and_wait(cmd)
Takuto Ikuta3153e3b2020-02-18 06:11:47 +0000578
Takuto Ikutad03ffcc2019-12-02 01:04:23 +0000579 with open(result_json_path) as json_file:
580 result_json = json.load(json_file)
581
582 isolated = result_json['isolated']
583 bundle = isolateserver.IsolatedBundle(filter_cb=None)
584 # Only following properties are used in caller.
585 bundle.command = isolated.get('command')
Takuto Ikutad03ffcc2019-12-02 01:04:23 +0000586 bundle.relative_cwd = isolated.get('relative_cwd')
587
588 return bundle, {
589 'duration': time.time() - start,
590 'items_cold': result_json['items_cold'],
591 'items_hot': result_json['items_hot'],
592 }
593 finally:
594 fs.remove(result_json_path)
595
596
597# TODO(crbug.com/932396): remove this function.
Takuto Ikuta16fac4b2019-12-09 04:57:18 +0000598def fetch_and_map(isolated_hash, storage, cache, outdir):
maruel4409e302016-07-19 14:25:51 -0700599 """Fetches an isolated tree, create the tree and returns (bundle, stats)."""
nodir6f801882016-04-29 14:41:50 -0700600 start = time.time()
601 bundle = isolateserver.fetch_isolated(
602 isolated_hash=isolated_hash,
603 storage=storage,
604 cache=cache,
maruel4409e302016-07-19 14:25:51 -0700605 outdir=outdir,
Takuto Ikuta16fac4b2019-12-09 04:57:18 +0000606 use_symlinks=False)
Takuto Ikuta2b9640e2019-06-19 00:53:23 +0000607 hot = (collections.Counter(cache.used) -
608 collections.Counter(cache.added)).elements()
nodir6f801882016-04-29 14:41:50 -0700609 return bundle, {
Takuto Ikuta630f99d2020-07-02 12:59:35 +0000610 'duration': time.time() - start,
611 'items_cold': base64.b64encode(large.pack(sorted(cache.added))).decode(),
612 'items_hot': base64.b64encode(large.pack(sorted(hot))).decode(),
nodir6f801882016-04-29 14:41:50 -0700613 }
614
615
aludwin0a8e17d2016-10-27 15:57:39 -0700616def link_outputs_to_outdir(run_dir, out_dir, outputs):
617 """Links any named outputs to out_dir so they can be uploaded.
618
619 Raises an error if the file already exists in that directory.
620 """
621 if not outputs:
622 return
623 isolateserver.create_directories(out_dir, outputs)
624 for o in outputs:
Sadaf Matinkhoo10743a62018-03-29 16:28:58 -0400625 copy_recursively(os.path.join(run_dir, o), os.path.join(out_dir, o))
626
627
628def copy_recursively(src, dst):
629 """Efficiently copies a file or directory from src_dir to dst_dir.
630
631 `item` may be a file, directory, or a symlink to a file or directory.
632 All symlinks are replaced with their targets, so the resulting
633 directory structure in dst_dir will never have any symlinks.
634
635 To increase speed, copy_recursively hardlinks individual files into the
636 (newly created) directory structure if possible, unlike Python's
637 shutil.copytree().
638 """
639 orig_src = src
640 try:
641 # Replace symlinks with their final target.
642 while fs.islink(src):
643 res = fs.readlink(src)
644 src = os.path.join(os.path.dirname(src), res)
645 # TODO(sadafm): Explicitly handle cyclic symlinks.
646
647 # Note that fs.isfile (which is a wrapper around os.path.isfile) throws
648 # an exception if src does not exist. A warning will be logged in that case.
649 if fs.isfile(src):
650 file_path.link_file(dst, src, file_path.HARDLINK_WITH_FALLBACK)
651 return
652
653 if not fs.exists(dst):
654 os.makedirs(dst)
655
656 for child in fs.listdir(src):
657 copy_recursively(os.path.join(src, child), os.path.join(dst, child))
658
659 except OSError as e:
660 if e.errno == errno.ENOENT:
661 logging.warning('Path %s does not exist or %s is a broken symlink',
662 src, orig_src)
663 else:
664 logging.info("Couldn't collect output file %s: %s", src, e)
aludwin0a8e17d2016-10-27 15:57:39 -0700665
666
Takuto Ikutab4aa8662019-09-17 05:54:36 +0000667def upload_then_delete(storage, out_dir, leak_temp_dir):
maruela9cfd6f2015-09-15 11:03:15 -0700668 """Deletes the temporary run directory and uploads results back.
669
670 Returns:
nodir6f801882016-04-29 14:41:50 -0700671 tuple(outputs_ref, success, stats)
maruel064c0a32016-04-05 11:47:15 -0700672 - outputs_ref: a dict referring to the results archived back to the isolated
673 server, if applicable.
674 - success: False if something occurred that means that the task must
675 forcibly be considered a failure, e.g. zombie processes were left
676 behind.
nodir6f801882016-04-29 14:41:50 -0700677 - stats: uploading stats.
maruela9cfd6f2015-09-15 11:03:15 -0700678 """
maruela9cfd6f2015-09-15 11:03:15 -0700679 # Upload out_dir and generate a .isolated file out of this directory. It is
680 # only done if files were written in the directory.
681 outputs_ref = None
maruel064c0a32016-04-05 11:47:15 -0700682 cold = []
683 hot = []
nodir6f801882016-04-29 14:41:50 -0700684 start = time.time()
685
maruel12e30012015-10-09 11:55:35 -0700686 if fs.isdir(out_dir) and fs.listdir(out_dir):
maruela9cfd6f2015-09-15 11:03:15 -0700687 with tools.Profiler('ArchiveOutput'):
688 try:
maruel064c0a32016-04-05 11:47:15 -0700689 results, f_cold, f_hot = isolateserver.archive_files_to_storage(
Takuto Ikuta26980872020-04-09 06:56:37 +0000690 storage, [out_dir], None, verify_push=True)
maruela9cfd6f2015-09-15 11:03:15 -0700691 outputs_ref = {
Takuto Ikuta630f99d2020-07-02 12:59:35 +0000692 'isolated': list(results.values())[0],
693 'isolatedserver': storage.server_ref.url,
694 'namespace': storage.server_ref.namespace,
maruela9cfd6f2015-09-15 11:03:15 -0700695 }
maruel064c0a32016-04-05 11:47:15 -0700696 cold = sorted(i.size for i in f_cold)
697 hot = sorted(i.size for i in f_hot)
maruela9cfd6f2015-09-15 11:03:15 -0700698 except isolateserver.Aborted:
699 # This happens when a signal SIGTERM was received while uploading data.
700 # There is 2 causes:
701 # - The task was too slow and was about to be killed anyway due to
702 # exceeding the hard timeout.
703 # - The amount of data uploaded back is very large and took too much
704 # time to archive.
705 sys.stderr.write('Received SIGTERM while uploading')
706 # Re-raise, so it will be treated as an internal failure.
707 raise
nodir6f801882016-04-29 14:41:50 -0700708
709 success = False
maruela9cfd6f2015-09-15 11:03:15 -0700710 try:
maruel12e30012015-10-09 11:55:35 -0700711 if (not leak_temp_dir and fs.isdir(out_dir) and
maruel6eeea7d2015-09-16 12:17:42 -0700712 not file_path.rmtree(out_dir)):
maruela9cfd6f2015-09-15 11:03:15 -0700713 logging.error('Had difficulties removing out_dir %s', out_dir)
nodir6f801882016-04-29 14:41:50 -0700714 else:
715 success = True
maruela9cfd6f2015-09-15 11:03:15 -0700716 except OSError as e:
717 # When this happens, it means there's a process error.
maruel12e30012015-10-09 11:55:35 -0700718 logging.exception('Had difficulties removing out_dir %s: %s', out_dir, e)
nodir6f801882016-04-29 14:41:50 -0700719 stats = {
Takuto Ikuta630f99d2020-07-02 12:59:35 +0000720 'duration': time.time() - start,
721 'items_cold': base64.b64encode(large.pack(cold)).decode(),
722 'items_hot': base64.b64encode(large.pack(hot)).decode(),
nodir6f801882016-04-29 14:41:50 -0700723 }
724 return outputs_ref, success, stats
maruela9cfd6f2015-09-15 11:03:15 -0700725
726
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -0500727def map_and_run(data, constant_run_path):
nodir55be77b2016-05-03 09:39:57 -0700728 """Runs a command with optional isolated input/output.
729
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -0500730 Arguments:
731 - data: TaskData instance.
732 - constant_run_path: TODO
nodir55be77b2016-05-03 09:39:57 -0700733
734 Returns metadata about the result.
735 """
Takuto Ikuta00cf8fc2020-01-14 01:36:00 +0000736
737 if data.isolate_cache:
738 download_stats = {
739 #'duration': 0.,
740 'initial_number_items': len(data.isolate_cache),
741 'initial_size': data.isolate_cache.total_size,
742 #'items_cold': '<large.pack()>',
743 #'items_hot': '<large.pack()>',
744 }
745 else:
746 # TODO(tikuta): take stats from state.json in this case too.
747 download_stats = {}
748
maruela9cfd6f2015-09-15 11:03:15 -0700749 result = {
Takuto Ikuta5ed62ad2019-09-26 09:16:00 +0000750 'duration': None,
751 'exit_code': None,
752 'had_hard_timeout': False,
753 'internal_failure': 'run_isolated did not complete properly',
754 'stats': {
755 #'cipd': {
756 # 'duration': 0.,
757 # 'get_client_duration': 0.,
758 #},
759 'isolated': {
Takuto Ikuta00cf8fc2020-01-14 01:36:00 +0000760 'download': download_stats,
Takuto Ikuta5ed62ad2019-09-26 09:16:00 +0000761 #'upload': {
762 # 'duration': 0.,
763 # 'items_cold': '<large.pack()>',
764 # 'items_hot': '<large.pack()>',
765 #},
766 },
Marc-Antoine Ruel5d7606b2018-06-15 19:06:12 +0000767 },
Takuto Ikuta5ed62ad2019-09-26 09:16:00 +0000768 #'cipd_pins': {
769 # 'packages': [
770 # {'package_name': ..., 'version': ..., 'path': ...},
771 # ...
772 # ],
773 # 'client_package': {'package_name': ..., 'version': ...},
774 #},
775 'outputs_ref': None,
776 'version': 5,
maruela9cfd6f2015-09-15 11:03:15 -0700777 }
nodirbe642ff2016-06-09 15:51:51 -0700778
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -0500779 if data.root_dir:
Lei Leife202df2019-06-11 17:33:34 +0000780 file_path.ensure_tree(data.root_dir, 0o700)
Takuto Ikuta00cf8fc2020-01-14 01:36:00 +0000781 elif data.use_go_isolated:
782 data = data._replace(root_dir=os.path.dirname(data.go_cache_dir))
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -0500783 elif data.isolate_cache.cache_dir:
784 data = data._replace(
785 root_dir=os.path.dirname(data.isolate_cache.cache_dir))
maruele2f2cb82016-07-13 14:41:03 -0700786 # See comment for these constants.
maruelcffa0542017-04-07 08:39:20 -0700787 # If root_dir is not specified, it is not constant.
788 # TODO(maruel): This is not obvious. Change this to become an error once we
789 # make the constant_run_path an exposed flag.
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -0500790 if constant_run_path and data.root_dir:
791 run_dir = os.path.join(data.root_dir, ISOLATED_RUN_DIR)
maruel5c4eed82017-05-26 05:33:40 -0700792 if os.path.isdir(run_dir):
793 file_path.rmtree(run_dir)
Lei Leife202df2019-06-11 17:33:34 +0000794 os.mkdir(run_dir, 0o700)
maruelcffa0542017-04-07 08:39:20 -0700795 else:
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -0500796 run_dir = make_temp_dir(ISOLATED_RUN_DIR, data.root_dir)
maruel03e11842016-07-14 10:50:16 -0700797 # storage should be normally set but don't crash if it is not. This can happen
798 # as Swarming task can run without an isolate server.
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -0500799 out_dir = make_temp_dir(
800 ISOLATED_OUT_DIR, data.root_dir) if data.storage else None
801 tmp_dir = make_temp_dir(ISOLATED_TMP_DIR, data.root_dir)
Takuto Ikutab7ce0e32019-11-27 23:26:18 +0000802 isolated_client_dir = make_temp_dir(ISOLATED_CLIENT_DIR, data.root_dir)
nodir55be77b2016-05-03 09:39:57 -0700803 cwd = run_dir
Marc-Antoine Ruel95068cf2017-12-07 21:35:05 -0500804 if data.relative_cwd:
805 cwd = os.path.normpath(os.path.join(cwd, data.relative_cwd))
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -0500806 command = data.command
nodir55be77b2016-05-03 09:39:57 -0700807 try:
Takuto Ikutab7ce0e32019-11-27 23:26:18 +0000808 with data.install_packages_fn(run_dir, isolated_client_dir) as cipd_info:
vadimsh232f5a82017-01-20 19:23:44 -0800809 if cipd_info:
810 result['stats']['cipd'] = cipd_info.stats
811 result['cipd_pins'] = cipd_info.pins
nodir90bc8dc2016-06-15 13:35:21 -0700812
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -0500813 if data.isolated_hash:
vadimsh232f5a82017-01-20 19:23:44 -0800814 isolated_stats = result['stats'].setdefault('isolated', {})
Takuto Ikutad03ffcc2019-12-02 01:04:23 +0000815 if data.use_go_isolated:
Takuto Ikuta90397ca2020-01-08 10:07:55 +0000816 bundle, stats = _fetch_and_map_with_go(
817 isolated_hash=data.isolated_hash,
818 storage=data.storage,
Takuto Ikuta90397ca2020-01-08 10:07:55 +0000819 outdir=run_dir,
820 go_cache_dir=data.go_cache_dir,
Takuto Ikuta879788c2020-01-10 08:00:26 +0000821 policies=data.go_cache_policies,
Takuto Ikuta90397ca2020-01-08 10:07:55 +0000822 isolated_client=os.path.join(isolated_client_dir,
823 'isolated' + cipd.EXECUTABLE_SUFFIX))
824 else:
Takuto Ikutad03ffcc2019-12-02 01:04:23 +0000825 bundle, stats = fetch_and_map(
826 isolated_hash=data.isolated_hash,
827 storage=data.storage,
828 cache=data.isolate_cache,
Takuto Ikuta16fac4b2019-12-09 04:57:18 +0000829 outdir=run_dir)
Marc-Antoine Ruel5d7606b2018-06-15 19:06:12 +0000830 isolated_stats['download'].update(stats)
Takuto Ikutab58dbd12020-06-05 09:29:14 +0000831
maruelabec63c2017-04-26 11:53:24 -0700832 # Inject the command
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -0500833 if not command and bundle.command:
834 command = bundle.command + data.extra_args
Marc-Antoine Rueld704a1f2017-10-31 10:51:23 -0400835 # Only set the relative directory if the isolated file specified a
836 # command, and no raw command was specified.
837 if bundle.relative_cwd:
838 cwd = os.path.normpath(os.path.join(cwd, bundle.relative_cwd))
maruelabec63c2017-04-26 11:53:24 -0700839
840 if not command:
841 # Handle this as a task failure, not an internal failure.
842 sys.stderr.write(
843 '<No command was specified!>\n'
844 '<Please secify a command when triggering your Swarming task>\n')
845 result['exit_code'] = 1
846 return result
nodirbe642ff2016-06-09 15:51:51 -0700847
Marc-Antoine Ruel95068cf2017-12-07 21:35:05 -0500848 if not cwd.startswith(run_dir):
849 # Handle this as a task failure, not an internal failure. This is a
850 # 'last chance' way to gate against directory escape.
851 sys.stderr.write('<Relative CWD is outside of run directory!>\n')
852 result['exit_code'] = 1
853 return result
854
855 if not os.path.isdir(cwd):
856 # Accepts relative_cwd that does not exist.
Lei Leife202df2019-06-11 17:33:34 +0000857 os.makedirs(cwd, 0o700)
Marc-Antoine Ruel95068cf2017-12-07 21:35:05 -0500858
vadimsh232f5a82017-01-20 19:23:44 -0800859 # If we have an explicit list of files to return, make sure their
860 # directories exist now.
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -0500861 if data.storage and data.outputs:
862 isolateserver.create_directories(run_dir, data.outputs)
aludwin0a8e17d2016-10-27 15:57:39 -0700863
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -0500864 with data.install_named_caches(run_dir):
nodird6160682017-02-02 13:03:35 -0800865 sys.stdout.flush()
866 start = time.time()
867 try:
vadimsh9c54b2c2017-07-25 14:08:29 -0700868 # Need to switch the default account before 'get_command_env' call,
869 # so it can grab correct value of LUCI_CONTEXT env var.
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -0500870 with set_luci_context_account(data.switch_to_account, tmp_dir):
871 env = get_command_env(
Roberto Carrillo71ade6d2018-10-08 22:30:24 +0000872 tmp_dir, cipd_info, run_dir, data.env, data.env_prefix, out_dir,
873 data.bot_file)
Brian Sheedy7a761172019-08-30 22:55:14 +0000874 command = tools.find_executable(command, env)
Robert Iannucci24ae76a2018-02-26 12:51:18 -0800875 command = process_command(command, out_dir, data.bot_file)
876 file_path.ensure_command_has_abs_path(command, cwd)
877
vadimsh9c54b2c2017-07-25 14:08:29 -0700878 result['exit_code'], result['had_hard_timeout'] = run_command(
Marc-Antoine Ruel03c6fd12019-04-30 12:12:55 +0000879 command, cwd, env, data.hard_timeout, data.grace_period,
Marc-Antoine Ruel1b65f4e2019-05-02 21:56:58 +0000880 data.lower_priority, data.containment)
nodird6160682017-02-02 13:03:35 -0800881 finally:
882 result['duration'] = max(time.time() - start, 0)
Seth Koehler49139812017-12-19 13:59:33 -0500883
884 # We successfully ran the command, set internal_failure back to
885 # None (even if the command failed, it's not an internal error).
886 result['internal_failure'] = None
maruela9cfd6f2015-09-15 11:03:15 -0700887 except Exception as e:
nodir90bc8dc2016-06-15 13:35:21 -0700888 # An internal error occurred. Report accordingly so the swarming task will
889 # be retried automatically.
maruel12e30012015-10-09 11:55:35 -0700890 logging.exception('internal failure: %s', e)
maruela9cfd6f2015-09-15 11:03:15 -0700891 result['internal_failure'] = str(e)
892 on_error.report(None)
aludwin0a8e17d2016-10-27 15:57:39 -0700893
894 # Clean up
maruela9cfd6f2015-09-15 11:03:15 -0700895 finally:
896 try:
aludwin0a8e17d2016-10-27 15:57:39 -0700897 # Try to link files to the output directory, if specified.
898 if out_dir:
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -0500899 link_outputs_to_outdir(run_dir, out_dir, data.outputs)
aludwin0a8e17d2016-10-27 15:57:39 -0700900
nodir32a1ec12016-10-26 18:34:07 -0700901 success = False
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -0500902 if data.leak_temp_dir:
nodir32a1ec12016-10-26 18:34:07 -0700903 success = True
maruela9cfd6f2015-09-15 11:03:15 -0700904 logging.warning(
905 'Deliberately leaking %s for later examination', run_dir)
marueleb5fbee2015-09-17 13:01:36 -0700906 else:
maruel84537cb2015-10-16 14:21:28 -0700907 # On Windows rmtree(run_dir) call above has a synchronization effect: it
908 # finishes only when all task child processes terminate (since a running
909 # process locks *.exe file). Examine out_dir only after that call
910 # completes (since child processes may write to out_dir too and we need
911 # to wait for them to finish).
Takuto Ikutab7ce0e32019-11-27 23:26:18 +0000912 for directory in (run_dir, tmp_dir, isolated_client_dir):
Takuto Ikuta69c0d662019-11-27 01:18:08 +0000913 if not fs.isdir(directory):
914 continue
maruel84537cb2015-10-16 14:21:28 -0700915 try:
Takuto Ikuta69c0d662019-11-27 01:18:08 +0000916 success = file_path.rmtree(directory)
maruel84537cb2015-10-16 14:21:28 -0700917 except OSError as e:
Takuto Ikuta69c0d662019-11-27 01:18:08 +0000918 logging.error('rmtree(%r) failed: %s', directory, e)
maruel84537cb2015-10-16 14:21:28 -0700919 success = False
920 if not success:
Takuto Ikuta69c0d662019-11-27 01:18:08 +0000921 sys.stderr.write(
922 OUTLIVING_ZOMBIE_MSG % (directory, data.grace_period))
maruel84537cb2015-10-16 14:21:28 -0700923 if result['exit_code'] == 0:
924 result['exit_code'] = 1
maruela9cfd6f2015-09-15 11:03:15 -0700925
marueleb5fbee2015-09-17 13:01:36 -0700926 # This deletes out_dir if leak_temp_dir is not set.
nodir9130f072016-05-27 13:59:08 -0700927 if out_dir:
nodir55715712016-06-03 12:28:19 -0700928 isolated_stats = result['stats'].setdefault('isolated', {})
929 result['outputs_ref'], success, isolated_stats['upload'] = (
Takuto Ikutab4aa8662019-09-17 05:54:36 +0000930 upload_then_delete(data.storage, out_dir, data.leak_temp_dir))
maruela9cfd6f2015-09-15 11:03:15 -0700931 if not success and result['exit_code'] == 0:
932 result['exit_code'] = 1
933 except Exception as e:
934 # Swallow any exception in the main finally clause.
nodir9130f072016-05-27 13:59:08 -0700935 if out_dir:
936 logging.exception('Leaking out_dir %s: %s', out_dir, e)
maruela9cfd6f2015-09-15 11:03:15 -0700937 result['internal_failure'] = str(e)
Takuto Ikutaa9a907b2020-04-17 08:50:50 +0000938 on_error.report(None)
maruela9cfd6f2015-09-15 11:03:15 -0700939 return result
Marc-Antoine Ruel2283ad12014-02-09 11:14:57 -0500940
941
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -0500942def run_tha_test(data, result_json):
nodir55be77b2016-05-03 09:39:57 -0700943 """Runs an executable and records execution metadata.
944
nodir55be77b2016-05-03 09:39:57 -0700945 If isolated_hash is specified, downloads the dependencies in the cache,
946 hardlinks them into a temporary directory and runs the command specified in
947 the .isolated.
Marc-Antoine Ruel2283ad12014-02-09 11:14:57 -0500948
949 A temporary directory is created to hold the output files. The content inside
950 this directory will be uploaded back to |storage| packaged as a .isolated
951 file.
952
953 Arguments:
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -0500954 - data: TaskData instance.
955 - result_json: File path to dump result metadata into. If set, the process
956 exit code is always 0 unless an internal error occurred.
maruela9cfd6f2015-09-15 11:03:15 -0700957
958 Returns:
959 Process exit code that should be used.
maruel@chromium.org9c72d4e2012-09-28 19:20:25 +0000960 """
maruela76b9ee2015-12-15 06:18:08 -0800961 if result_json:
962 # Write a json output file right away in case we get killed.
963 result = {
964 'exit_code': None,
965 'had_hard_timeout': False,
966 'internal_failure': 'Was terminated before completion',
967 'outputs_ref': None,
nodirbe642ff2016-06-09 15:51:51 -0700968 'version': 5,
maruela76b9ee2015-12-15 06:18:08 -0800969 }
970 tools.write_json(result_json, result, dense=True)
971
maruela9cfd6f2015-09-15 11:03:15 -0700972 # run_isolated exit code. Depends on if result_json is used or not.
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -0500973 result = map_and_run(data, True)
maruela9cfd6f2015-09-15 11:03:15 -0700974 logging.info('Result:\n%s', tools.format_json(result, dense=True))
bpastene3ae09522016-06-10 17:12:59 -0700975
maruela9cfd6f2015-09-15 11:03:15 -0700976 if result_json:
maruel05d5a882015-09-21 13:59:02 -0700977 # We've found tests to delete 'work' when quitting, causing an exception
978 # here. Try to recreate the directory if necessary.
nodire5028a92016-04-29 14:38:21 -0700979 file_path.ensure_tree(os.path.dirname(result_json))
maruela9cfd6f2015-09-15 11:03:15 -0700980 tools.write_json(result_json, result, dense=True)
981 # Only return 1 if there was an internal error.
982 return int(bool(result['internal_failure']))
maruel@chromium.org781ccf62013-09-17 19:39:47 +0000983
maruela9cfd6f2015-09-15 11:03:15 -0700984 # Marshall into old-style inline output.
985 if result['outputs_ref']:
Marc-Antoine Ruel793bff32019-04-18 17:50:48 +0000986 # pylint: disable=unsubscriptable-object
maruela9cfd6f2015-09-15 11:03:15 -0700987 data = {
Junji Watanabe38b28b02020-04-23 10:23:30 +0000988 'hash': result['outputs_ref']['isolated'],
989 'namespace': result['outputs_ref']['namespace'],
990 'storage': result['outputs_ref']['isolatedserver'],
maruela9cfd6f2015-09-15 11:03:15 -0700991 }
Marc-Antoine Ruelc44f5722015-01-08 16:10:01 -0500992 sys.stdout.flush()
Junji Watanabe38b28b02020-04-23 10:23:30 +0000993 print('[run_isolated_out_hack]%s[/run_isolated_out_hack]' %
994 tools.format_json(data, dense=True))
maruelb76604c2015-11-11 11:53:44 -0800995 sys.stdout.flush()
maruela9cfd6f2015-09-15 11:03:15 -0700996 return result['exit_code'] or int(bool(result['internal_failure']))
maruel@chromium.org9c72d4e2012-09-28 19:20:25 +0000997
998
iannuccib58d10d2017-03-18 02:00:25 -0700999# Yielded by 'install_client_and_packages'.
vadimsh232f5a82017-01-20 19:23:44 -08001000CipdInfo = collections.namedtuple('CipdInfo', [
1001 'client', # cipd.CipdClient object
1002 'cache_dir', # absolute path to bot-global cipd tag and instance cache
1003 'stats', # dict with stats to return to the server
1004 'pins', # dict with installed cipd pins to return to the server
1005])
1006
1007
1008@contextlib.contextmanager
Takuto Ikutab7ce0e32019-11-27 23:26:18 +00001009def noop_install_packages(_run_dir, _isolated_dir):
iannuccib58d10d2017-03-18 02:00:25 -07001010 """Placeholder for 'install_client_and_packages' if cipd is disabled."""
vadimsh232f5a82017-01-20 19:23:44 -08001011 yield None
1012
1013
Takuto Ikuta2efc7792019-11-27 14:33:34 +00001014def _install_packages(run_dir, cipd_cache_dir, client, packages):
iannuccib58d10d2017-03-18 02:00:25 -07001015 """Calls 'cipd ensure' for packages.
1016
1017 Args:
1018 run_dir (str): root of installation.
1019 cipd_cache_dir (str): the directory to use for the cipd package cache.
1020 client (CipdClient): the cipd client to use
1021 packages: packages to install, list [(path, package_name, version), ...].
iannuccib58d10d2017-03-18 02:00:25 -07001022
1023 Returns: list of pinned packages. Looks like [
1024 {
1025 'path': 'subdirectory',
1026 'package_name': 'resolved/package/name',
1027 'version': 'deadbeef...',
1028 },
1029 ...
1030 ]
1031 """
1032 package_pins = [None]*len(packages)
1033 def insert_pin(path, name, version, idx):
1034 package_pins[idx] = {
1035 'package_name': name,
1036 # swarming deals with 'root' as '.'
1037 'path': path or '.',
1038 'version': version,
1039 }
1040
1041 by_path = collections.defaultdict(list)
1042 for i, (path, name, version) in enumerate(packages):
1043 # cipd deals with 'root' as ''
1044 if path == '.':
1045 path = ''
1046 by_path[path].append((name, version, i))
1047
1048 pins = client.ensure(
Takuto Ikuta2efc7792019-11-27 14:33:34 +00001049 run_dir,
1050 {
1051 subdir: [(name, vers) for name, vers, _ in pkgs
1052 ] for subdir, pkgs in by_path.items()
1053 },
1054 cache_dir=cipd_cache_dir,
iannuccib58d10d2017-03-18 02:00:25 -07001055 )
1056
Marc-Antoine Ruel04903a32019-10-09 21:09:25 +00001057 for subdir, pin_list in sorted(pins.items()):
iannuccib58d10d2017-03-18 02:00:25 -07001058 this_subdir = by_path[subdir]
1059 for i, (name, version) in enumerate(pin_list):
1060 insert_pin(subdir, name, version, this_subdir[i][2])
1061
Robert Iannucci461b30d2017-12-13 11:34:03 -08001062 assert None not in package_pins, (packages, pins, package_pins)
iannuccib58d10d2017-03-18 02:00:25 -07001063
1064 return package_pins
1065
1066
vadimsh232f5a82017-01-20 19:23:44 -08001067@contextlib.contextmanager
Takuto Ikuta2efc7792019-11-27 14:33:34 +00001068def install_client_and_packages(run_dir, packages, service_url,
Takuto Ikutab7ce0e32019-11-27 23:26:18 +00001069 client_package_name, client_version, cache_dir,
1070 isolated_dir):
vadimsh902948e2017-01-20 15:57:32 -08001071 """Bootstraps CIPD client and installs CIPD packages.
iannucci96fcccc2016-08-30 15:52:22 -07001072
vadimsh232f5a82017-01-20 19:23:44 -08001073 Yields CipdClient, stats, client info and pins (as single CipdInfo object).
1074
1075 Pins and the CIPD client info are in the form of:
iannucci96fcccc2016-08-30 15:52:22 -07001076 [
1077 {
1078 "path": path, "package_name": package_name, "version": version,
1079 },
1080 ...
1081 ]
vadimsh902948e2017-01-20 15:57:32 -08001082 (the CIPD client info is a single dictionary instead of a list)
iannucci96fcccc2016-08-30 15:52:22 -07001083
1084 such that they correspond 1:1 to all input package arguments from the command
1085 line. These dictionaries make their all the way back to swarming, where they
1086 become the arguments of CipdPackage.
nodirbe642ff2016-06-09 15:51:51 -07001087
vadimsh902948e2017-01-20 15:57:32 -08001088 If 'packages' list is empty, will bootstrap CIPD client, but won't install
1089 any packages.
1090
1091 The bootstrapped client (regardless whether 'packages' list is empty or not),
vadimsh232f5a82017-01-20 19:23:44 -08001092 will be made available to the task via $PATH.
vadimsh902948e2017-01-20 15:57:32 -08001093
nodirbe642ff2016-06-09 15:51:51 -07001094 Args:
nodir90bc8dc2016-06-15 13:35:21 -07001095 run_dir (str): root of installation.
vadimsh902948e2017-01-20 15:57:32 -08001096 packages: packages to install, list [(path, package_name, version), ...].
nodirbe642ff2016-06-09 15:51:51 -07001097 service_url (str): CIPD server url, e.g.
1098 "https://chrome-infra-packages.appspot.com."
nodir90bc8dc2016-06-15 13:35:21 -07001099 client_package_name (str): CIPD package name of CIPD client.
1100 client_version (str): Version of CIPD client.
nodirbe642ff2016-06-09 15:51:51 -07001101 cache_dir (str): where to keep cache of cipd clients, packages and tags.
Takuto Ikutab7ce0e32019-11-27 23:26:18 +00001102 isolated_dir (str): where to download isolated client.
nodirbe642ff2016-06-09 15:51:51 -07001103 """
1104 assert cache_dir
nodir90bc8dc2016-06-15 13:35:21 -07001105
nodirbe642ff2016-06-09 15:51:51 -07001106 start = time.time()
nodirbe642ff2016-06-09 15:51:51 -07001107
vadimsh902948e2017-01-20 15:57:32 -08001108 cache_dir = os.path.abspath(cache_dir)
vadimsh232f5a82017-01-20 19:23:44 -08001109 cipd_cache_dir = os.path.join(cache_dir, 'cache') # tag and instance caches
nodir90bc8dc2016-06-15 13:35:21 -07001110 run_dir = os.path.abspath(run_dir)
vadimsh902948e2017-01-20 15:57:32 -08001111 packages = packages or []
nodir90bc8dc2016-06-15 13:35:21 -07001112
nodirbe642ff2016-06-09 15:51:51 -07001113 get_client_start = time.time()
Takuto Ikuta2efc7792019-11-27 14:33:34 +00001114 client_manager = cipd.get_client(service_url, client_package_name,
1115 client_version, cache_dir)
iannucci96fcccc2016-08-30 15:52:22 -07001116
nodirbe642ff2016-06-09 15:51:51 -07001117 with client_manager as client:
1118 get_client_duration = time.time() - get_client_start
nodir90bc8dc2016-06-15 13:35:21 -07001119
iannuccib58d10d2017-03-18 02:00:25 -07001120 package_pins = []
1121 if packages:
Takuto Ikuta2efc7792019-11-27 14:33:34 +00001122 package_pins = _install_packages(run_dir, cipd_cache_dir, client,
1123 packages)
iannuccib58d10d2017-03-18 02:00:25 -07001124
Takuto Ikutab7ce0e32019-11-27 23:26:18 +00001125 # Install isolated client to |isolated_dir|.
Takuto Ikuta02edca22019-11-29 10:04:51 +00001126 _install_packages(isolated_dir, cipd_cache_dir, client,
1127 [('', ISOLATED_PACKAGE, ISOLATED_REVISION)])
Takuto Ikutab7ce0e32019-11-27 23:26:18 +00001128
iannuccib58d10d2017-03-18 02:00:25 -07001129 file_path.make_tree_files_read_only(run_dir)
nodir90bc8dc2016-06-15 13:35:21 -07001130
vadimsh232f5a82017-01-20 19:23:44 -08001131 total_duration = time.time() - start
Junji Watanabe38b28b02020-04-23 10:23:30 +00001132 logging.info('Installing CIPD client and packages took %d seconds',
1133 total_duration)
nodir90bc8dc2016-06-15 13:35:21 -07001134
vadimsh232f5a82017-01-20 19:23:44 -08001135 yield CipdInfo(
Junji Watanabe38b28b02020-04-23 10:23:30 +00001136 client=client,
1137 cache_dir=cipd_cache_dir,
1138 stats={
1139 'duration': total_duration,
1140 'get_client_duration': get_client_duration,
iannuccib58d10d2017-03-18 02:00:25 -07001141 },
Junji Watanabe38b28b02020-04-23 10:23:30 +00001142 pins={
1143 'client_package': {
1144 'package_name': client.package_name,
1145 'version': client.instance_id,
1146 },
1147 'packages': package_pins,
1148 })
nodirbe642ff2016-06-09 15:51:51 -07001149
1150
1151def create_option_parser():
Marc-Antoine Ruelf74cffe2015-07-15 15:21:34 -04001152 parser = logging_utils.OptionParserWithLogging(
nodir55be77b2016-05-03 09:39:57 -07001153 usage='%prog <options> [command to run or extra args]',
maruel@chromium.orgdedbf492013-09-12 20:42:11 +00001154 version=__version__,
1155 log_file=RUN_ISOLATED_LOG_FILE)
maruela9cfd6f2015-09-15 11:03:15 -07001156 parser.add_option(
Junji Watanabe38b28b02020-04-23 10:23:30 +00001157 '--clean',
1158 action='store_true',
maruel36a963d2016-04-08 17:15:49 -07001159 help='Cleans the cache, trimming it necessary and remove corrupted items '
Junji Watanabe38b28b02020-04-23 10:23:30 +00001160 'and returns without executing anything; use with -v to know what '
1161 'was done')
maruel36a963d2016-04-08 17:15:49 -07001162 parser.add_option(
maruela9cfd6f2015-09-15 11:03:15 -07001163 '--json',
1164 help='dump output metadata to json file. When used, run_isolated returns '
Junji Watanabe38b28b02020-04-23 10:23:30 +00001165 'non-zero only on internal failure')
maruel6be7f9e2015-10-01 12:25:30 -07001166 parser.add_option(
maruel5c9e47b2015-12-18 13:02:30 -08001167 '--hard-timeout', type='float', help='Enforce hard timeout in execution')
maruel6be7f9e2015-10-01 12:25:30 -07001168 parser.add_option(
Junji Watanabe38b28b02020-04-23 10:23:30 +00001169 '--grace-period',
1170 type='float',
maruel6be7f9e2015-10-01 12:25:30 -07001171 help='Grace period between SIGTERM and SIGKILL')
bpastene3ae09522016-06-10 17:12:59 -07001172 parser.add_option(
Junji Watanabe38b28b02020-04-23 10:23:30 +00001173 '--raw-cmd',
1174 action='store_true',
Marc-Antoine Ruel49e347d2017-10-24 16:52:02 -07001175 help='Ignore the isolated command, use the one supplied at the command '
Junji Watanabe38b28b02020-04-23 10:23:30 +00001176 'line')
Marc-Antoine Ruel49e347d2017-10-24 16:52:02 -07001177 parser.add_option(
Marc-Antoine Ruel95068cf2017-12-07 21:35:05 -05001178 '--relative-cwd',
1179 help='Ignore the isolated \'relative_cwd\' and use this one instead; '
Junji Watanabe38b28b02020-04-23 10:23:30 +00001180 'requires --raw-cmd')
Marc-Antoine Ruel95068cf2017-12-07 21:35:05 -05001181 parser.add_option(
Junji Watanabe38b28b02020-04-23 10:23:30 +00001182 '--env',
1183 default=[],
1184 action='append',
Marc-Antoine Ruel19dd8872017-11-28 18:33:39 -05001185 help='Environment variables to set for the child process')
1186 parser.add_option(
Junji Watanabe38b28b02020-04-23 10:23:30 +00001187 '--env-prefix',
1188 default=[],
1189 action='append',
Robert Iannuccibf5f84c2017-11-22 12:56:50 -08001190 help='Specify a VAR=./path/fragment to put in the environment variable '
Junji Watanabe38b28b02020-04-23 10:23:30 +00001191 'before executing the command. The path fragment must be relative '
1192 'to the isolated run directory, and must not contain a `..` token. '
1193 'The path will be made absolute and prepended to the indicated '
1194 '$VAR using the OS\'s path separator. Multiple items for the same '
1195 '$VAR will be prepended in order.')
Robert Iannuccibf5f84c2017-11-22 12:56:50 -08001196 parser.add_option(
bpastene3ae09522016-06-10 17:12:59 -07001197 '--bot-file',
1198 help='Path to a file describing the state of the host. The content is '
Junji Watanabe38b28b02020-04-23 10:23:30 +00001199 'defined by on_before_task() in bot_config.')
aludwin7556e0c2016-10-26 08:46:10 -07001200 parser.add_option(
vadimsh9c54b2c2017-07-25 14:08:29 -07001201 '--switch-to-account',
1202 help='If given, switches LUCI_CONTEXT to given logical service account '
Junji Watanabe38b28b02020-04-23 10:23:30 +00001203 '(e.g. "task" or "system") before launching the isolated process.')
vadimsh9c54b2c2017-07-25 14:08:29 -07001204 parser.add_option(
Junji Watanabe38b28b02020-04-23 10:23:30 +00001205 '--output',
1206 action='append',
aludwin0a8e17d2016-10-27 15:57:39 -07001207 help='Specifies an output to return. If no outputs are specified, all '
Junji Watanabe38b28b02020-04-23 10:23:30 +00001208 'files located in $(ISOLATED_OUTDIR) will be returned; '
1209 'otherwise, outputs in both $(ISOLATED_OUTDIR) and those '
1210 'specified by --output option (there can be multiple) will be '
1211 'returned. Note that if a file in OUT_DIR has the same path '
1212 'as an --output option, the --output version will be returned.')
aludwin0a8e17d2016-10-27 15:57:39 -07001213 parser.add_option(
Junji Watanabe38b28b02020-04-23 10:23:30 +00001214 '-a',
1215 '--argsfile',
aludwin7556e0c2016-10-26 08:46:10 -07001216 # This is actually handled in parse_args; it's included here purely so it
1217 # can make it into the help text.
1218 help='Specify a file containing a JSON array of arguments to this '
Junji Watanabe38b28b02020-04-23 10:23:30 +00001219 'script. If --argsfile is provided, no other argument may be '
1220 'provided on the command line.')
Takuto Ikutad4be2f12020-05-12 02:15:25 +00001221 parser.add_option(
1222 '--report-on-exception',
1223 action='store_true',
1224 help='Whether report exception during execution to isolate server. '
1225 'This flag should only be used in swarming bot.')
Marc-Antoine Ruel1b65f4e2019-05-02 21:56:58 +00001226
1227 group = optparse.OptionGroup(parser, 'Data source')
1228 group.add_option(
Marc-Antoine Ruel185ded42015-01-28 20:49:18 -05001229 '-s', '--isolated',
nodir55be77b2016-05-03 09:39:57 -07001230 help='Hash of the .isolated to grab from the isolate server.')
Marc-Antoine Ruel1b65f4e2019-05-02 21:56:58 +00001231 isolateserver.add_isolate_server_options(group)
1232 parser.add_option_group(group)
maruel@chromium.org9c72d4e2012-09-28 19:20:25 +00001233
Marc-Antoine Ruela57d7db2014-10-15 20:31:19 -04001234 isolateserver.add_cache_options(parser)
nodirbe642ff2016-06-09 15:51:51 -07001235
1236 cipd.add_cipd_options(parser)
Marc-Antoine Ruel8b11dbd2018-05-18 14:31:22 -04001237
1238 group = optparse.OptionGroup(parser, 'Named caches')
1239 group.add_option(
1240 '--named-cache',
1241 dest='named_caches',
1242 action='append',
Marc-Antoine Ruelc7a704b2018-08-29 19:02:23 +00001243 nargs=3,
Marc-Antoine Ruel8b11dbd2018-05-18 14:31:22 -04001244 default=[],
Marc-Antoine Ruelc7a704b2018-08-29 19:02:23 +00001245 help='A named cache to request. Accepts 3 arguments: name, path, hint. '
Junji Watanabe38b28b02020-04-23 10:23:30 +00001246 'name identifies the cache, must match regex [a-z0-9_]{1,4096}. '
1247 'path is a path relative to the run dir where the cache directory '
1248 'must be put to. '
1249 'This option can be specified more than once.')
Marc-Antoine Ruel8b11dbd2018-05-18 14:31:22 -04001250 group.add_option(
Junji Watanabe38b28b02020-04-23 10:23:30 +00001251 '--named-cache-root',
1252 default='named_caches',
Marc-Antoine Ruel8b11dbd2018-05-18 14:31:22 -04001253 help='Cache root directory. Default=%default')
1254 parser.add_option_group(group)
maruel@chromium.org9c72d4e2012-09-28 19:20:25 +00001255
Marc-Antoine Ruel1b65f4e2019-05-02 21:56:58 +00001256 group = optparse.OptionGroup(parser, 'Process containment')
1257 parser.add_option(
1258 '--lower-priority', action='store_true',
1259 help='Lowers the child process priority')
1260 parser.add_option(
Junji Watanabe38b28b02020-04-23 10:23:30 +00001261 '--containment-type',
1262 choices=('NONE', 'AUTO', 'JOB_OBJECT'),
Marc-Antoine Ruel1b65f4e2019-05-02 21:56:58 +00001263 default='NONE',
1264 help='Type of container to use')
1265 parser.add_option(
Junji Watanabe38b28b02020-04-23 10:23:30 +00001266 '--limit-processes',
1267 type='int',
1268 default=0,
Marc-Antoine Ruel1b65f4e2019-05-02 21:56:58 +00001269 help='Maximum number of active processes in the containment')
1270 parser.add_option(
Junji Watanabe38b28b02020-04-23 10:23:30 +00001271 '--limit-total-committed-memory',
1272 type='int',
1273 default=0,
Marc-Antoine Ruel1b65f4e2019-05-02 21:56:58 +00001274 help='Maximum sum of committed memory in the containment')
1275 parser.add_option_group(group)
1276
1277 group = optparse.OptionGroup(parser, 'Debugging')
1278 group.add_option(
Kenneth Russell61d42352014-09-15 11:41:16 -07001279 '--leak-temp-dir',
1280 action='store_true',
nodirbe642ff2016-06-09 15:51:51 -07001281 help='Deliberately leak isolate\'s temp dir for later examination. '
Junji Watanabe38b28b02020-04-23 10:23:30 +00001282 'Default: %default')
1283 group.add_option('--root-dir', help='Use a directory instead of a random one')
Marc-Antoine Ruel1b65f4e2019-05-02 21:56:58 +00001284 parser.add_option_group(group)
Kenneth Russell61d42352014-09-15 11:41:16 -07001285
Vadim Shtayurae34e13a2014-02-02 11:23:26 -08001286 auth.add_auth_options(parser)
nodirbe642ff2016-06-09 15:51:51 -07001287
Ye Kuang1d096cb2020-06-26 08:38:21 +00001288 parser.set_defaults(cache='cache')
nodirbe642ff2016-06-09 15:51:51 -07001289 return parser
1290
1291
Marc-Antoine Ruel49f9f8d2018-05-24 15:57:06 -04001292def process_named_cache_options(parser, options, time_fn=None):
Marc-Antoine Ruel8b11dbd2018-05-18 14:31:22 -04001293 """Validates named cache options and returns a CacheManager."""
1294 if options.named_caches and not options.named_cache_root:
1295 parser.error('--named-cache is specified, but --named-cache-root is empty')
Marc-Antoine Ruelc7a704b2018-08-29 19:02:23 +00001296 for name, path, hint in options.named_caches:
Marc-Antoine Ruel8b11dbd2018-05-18 14:31:22 -04001297 if not CACHE_NAME_RE.match(name):
1298 parser.error(
1299 'cache name %r does not match %r' % (name, CACHE_NAME_RE.pattern))
1300 if not path:
1301 parser.error('cache path cannot be empty')
Marc-Antoine Ruelc7a704b2018-08-29 19:02:23 +00001302 try:
Takuto Ikuta630f99d2020-07-02 12:59:35 +00001303 int(hint)
Marc-Antoine Ruelc7a704b2018-08-29 19:02:23 +00001304 except ValueError:
1305 parser.error('cache hint must be a number')
Marc-Antoine Ruel8b11dbd2018-05-18 14:31:22 -04001306 if options.named_cache_root:
1307 # Make these configurable later if there is use case but for now it's fairly
1308 # safe values.
1309 # In practice, a fair chunk of bots are already recycled on a daily schedule
1310 # so this code doesn't have any effect to them, unless they are preloaded
1311 # with a really old cache.
1312 policies = local_caching.CachePolicies(
1313 # 1TiB.
1314 max_cache_size=1024*1024*1024*1024,
1315 min_free_space=options.min_free_space,
1316 max_items=50,
Marc-Antoine Ruel5d7606b2018-06-15 19:06:12 +00001317 max_age_secs=MAX_AGE_SECS)
Takuto Ikuta6e2ff962019-10-29 12:35:27 +00001318 root_dir = six.text_type(os.path.abspath(options.named_cache_root))
John Budorickc6186972020-02-26 00:58:14 +00001319 cache = local_caching.NamedCache(root_dir, policies, time_fn=time_fn)
1320 # Touch any named caches we're going to use to minimize thrashing
1321 # between tasks that request some (but not all) of the same named caches.
John Budorick0a4dab62020-03-02 22:23:35 +00001322 cache.touch(*[name for name, _, _ in options.named_caches])
John Budorickc6186972020-02-26 00:58:14 +00001323 return cache
Marc-Antoine Ruel8b11dbd2018-05-18 14:31:22 -04001324 return None
1325
1326
aludwin7556e0c2016-10-26 08:46:10 -07001327def parse_args(args):
1328 # Create a fake mini-parser just to get out the "-a" command. Note that
1329 # it's not documented here; instead, it's documented in create_option_parser
1330 # even though that parser will never actually get to parse it. This is
1331 # because --argsfile is exclusive with all other options and arguments.
1332 file_argparse = argparse.ArgumentParser(add_help=False)
1333 file_argparse.add_argument('-a', '--argsfile')
1334 (file_args, nonfile_args) = file_argparse.parse_known_args(args)
1335 if file_args.argsfile:
1336 if nonfile_args:
1337 file_argparse.error('Can\'t specify --argsfile with'
1338 'any other arguments (%s)' % nonfile_args)
1339 try:
1340 with open(file_args.argsfile, 'r') as f:
1341 args = json.load(f)
1342 except (IOError, OSError, ValueError) as e:
1343 # We don't need to error out here - "args" is now empty,
1344 # so the call below to parser.parse_args(args) will fail
1345 # and print the full help text.
Marc-Antoine Ruelf899c482019-10-10 23:32:06 +00001346 print('Couldn\'t read arguments: %s' % e, file=sys.stderr)
aludwin7556e0c2016-10-26 08:46:10 -07001347
1348 # Even if we failed to read the args, just call the normal parser now since it
1349 # will print the correct help message.
nodirbe642ff2016-06-09 15:51:51 -07001350 parser = create_option_parser()
Marc-Antoine Ruel90c98162013-12-18 15:11:57 -05001351 options, args = parser.parse_args(args)
Ye Kuangfff1e502020-07-13 13:21:57 +00001352 if not isinstance(options.cipd_enabled, (bool, int)):
1353 options.cipd_enabled = distutils.util.strtobool(options.cipd_enabled)
aludwin7556e0c2016-10-26 08:46:10 -07001354 return (parser, options, args)
1355
1356
Marc-Antoine Ruelc7a704b2018-08-29 19:02:23 +00001357def _calc_named_cache_hint(named_cache, named_caches):
1358 """Returns the expected size of the missing named caches."""
1359 present = named_cache.available
1360 size = 0
1361 for name, _, hint in named_caches:
1362 if name not in present:
Takuto Ikuta630f99d2020-07-02 12:59:35 +00001363 hint = int(hint)
Marc-Antoine Ruelc7a704b2018-08-29 19:02:23 +00001364 if hint > 0:
1365 size += hint
1366 return size
1367
1368
aludwin7556e0c2016-10-26 08:46:10 -07001369def main(args):
Marc-Antoine Ruelee6ca622017-11-29 11:19:16 -05001370 # Warning: when --argsfile is used, the strings are unicode instances, when
1371 # parsed normally, the strings are str instances.
aludwin7556e0c2016-10-26 08:46:10 -07001372 (parser, options, args) = parse_args(args)
maruel36a963d2016-04-08 17:15:49 -07001373
Takuto Ikutad4be2f12020-05-12 02:15:25 +00001374 if options.report_on_exception and options.isolate_server:
1375 on_error.report_on_exception_exit(options.isolate_server)
1376
Marc-Antoine Ruel5028ba22017-08-25 17:37:51 -04001377 if not file_path.enable_symlink():
Marc-Antoine Ruel5a024272019-01-15 20:11:16 +00001378 logging.warning('Symlink support is not enabled')
Marc-Antoine Ruel5028ba22017-08-25 17:37:51 -04001379
Marc-Antoine Ruelc7a704b2018-08-29 19:02:23 +00001380 named_cache = process_named_cache_options(parser, options)
Marc-Antoine Ruel0d8b0f62018-09-10 14:40:35 +00001381 # hint is 0 if there's no named cache.
Marc-Antoine Ruelc7a704b2018-08-29 19:02:23 +00001382 hint = _calc_named_cache_hint(named_cache, options.named_caches)
1383 if hint:
1384 # Increase the --min-free-space value by the hint, and recreate the
1385 # NamedCache instance so it gets the updated CachePolicy.
1386 options.min_free_space += hint
1387 named_cache = process_named_cache_options(parser, options)
1388
Takuto Ikuta5c59a842020-01-24 03:05:24 +00001389 # TODO(crbug.com/932396): Remove this.
Takuto Ikuta4a22c2c2020-06-05 02:02:23 +00001390 use_go_isolated = options.cipd_enabled
Takuto Ikuta5c59a842020-01-24 03:05:24 +00001391
Marc-Antoine Ruel7139d912018-06-15 20:04:42 +00001392 # TODO(maruel): CIPD caches should be defined at an higher level here too, so
1393 # they can be cleaned the same way.
Ye Kuang97849802020-06-29 13:17:09 +00001394 if use_go_isolated and not options.clean:
Takuto Ikuta00cf8fc2020-01-14 01:36:00 +00001395 isolate_cache = None
1396 else:
1397 isolate_cache = isolateserver.process_cache_options(options, trim=False)
1398
Marc-Antoine Ruel7139d912018-06-15 20:04:42 +00001399 caches = []
1400 if isolate_cache:
1401 caches.append(isolate_cache)
1402 if named_cache:
1403 caches.append(named_cache)
Takuto Ikuta6e2ff962019-10-29 12:35:27 +00001404 root = caches[0].cache_dir if caches else six.text_type(os.getcwd())
maruel36a963d2016-04-08 17:15:49 -07001405 if options.clean:
1406 if options.isolated:
1407 parser.error('Can\'t use --isolated with --clean.')
1408 if options.isolate_server:
1409 parser.error('Can\'t use --isolate-server with --clean.')
1410 if options.json:
1411 parser.error('Can\'t use --json with --clean.')
nodirf33b8d62016-10-26 22:34:58 -07001412 if options.named_caches:
1413 parser.error('Can\t use --named-cache with --clean.')
Marc-Antoine Ruel7139d912018-06-15 20:04:42 +00001414 # Trim first, then clean.
1415 local_caching.trim_caches(
1416 caches,
1417 root,
1418 min_free_space=options.min_free_space,
1419 max_age_secs=MAX_AGE_SECS)
1420 for c in caches:
Marc-Antoine Ruel87fc2222018-06-18 13:09:24 +00001421 c.cleanup()
maruel36a963d2016-04-08 17:15:49 -07001422 return 0
Marc-Antoine Ruelc7a704b2018-08-29 19:02:23 +00001423
1424 # Trim must still be done for the following case:
1425 # - named-cache was used
1426 # - some entries, with a large hint, where missing
1427 # - --min-free-space was increased accordingly, thus trimming is needed
1428 # Otherwise, this will have no effect, as bot_main calls run_isolated with
1429 # --clean after each task.
1430 if hint:
1431 logging.info('Additional trimming of %d bytes', hint)
Marc-Antoine Ruel0d8b0f62018-09-10 14:40:35 +00001432 local_caching.trim_caches(
1433 caches,
1434 root,
1435 min_free_space=options.min_free_space,
1436 max_age_secs=MAX_AGE_SECS)
maruel36a963d2016-04-08 17:15:49 -07001437
nodir55be77b2016-05-03 09:39:57 -07001438 if not options.isolated and not args:
1439 parser.error('--isolated or command to run is required.')
1440
Vadim Shtayura5d1efce2014-02-04 10:55:43 -08001441 auth.process_auth_options(parser, options)
nodir55be77b2016-05-03 09:39:57 -07001442
Takuto Ikutaae767b32020-05-11 01:22:19 +00001443 isolateserver.process_isolate_server_options(parser, options, False)
nodir55be77b2016-05-03 09:39:57 -07001444 if not options.isolate_server:
1445 if options.isolated:
1446 parser.error('--isolated requires --isolate-server')
1447 if ISOLATED_OUTDIR_PARAMETER in args:
1448 parser.error(
1449 '%s in args requires --isolate-server' % ISOLATED_OUTDIR_PARAMETER)
maruel@chromium.org9c72d4e2012-09-28 19:20:25 +00001450
nodir90bc8dc2016-06-15 13:35:21 -07001451 if options.root_dir:
Takuto Ikuta6e2ff962019-10-29 12:35:27 +00001452 options.root_dir = six.text_type(os.path.abspath(options.root_dir))
maruel12e30012015-10-09 11:55:35 -07001453 if options.json:
Takuto Ikuta6e2ff962019-10-29 12:35:27 +00001454 options.json = six.text_type(os.path.abspath(options.json))
nodir55be77b2016-05-03 09:39:57 -07001455
Marc-Antoine Ruel19dd8872017-11-28 18:33:39 -05001456 if any('=' not in i for i in options.env):
1457 parser.error(
1458 '--env required key=value form. value can be skipped to delete '
1459 'the variable')
Marc-Antoine Ruel7a68f712017-12-01 18:45:18 -05001460 options.env = dict(i.split('=', 1) for i in options.env)
Marc-Antoine Ruel19dd8872017-11-28 18:33:39 -05001461
1462 prefixes = {}
1463 cwd = os.path.realpath(os.getcwd())
1464 for item in options.env_prefix:
1465 if '=' not in item:
1466 parser.error(
1467 '--env-prefix %r is malformed, must be in the form `VAR=./path`'
1468 % item)
Marc-Antoine Ruel7a68f712017-12-01 18:45:18 -05001469 key, opath = item.split('=', 1)
Marc-Antoine Ruel19dd8872017-11-28 18:33:39 -05001470 if os.path.isabs(opath):
1471 parser.error('--env-prefix %r path is bad, must be relative.' % opath)
1472 opath = os.path.normpath(opath)
1473 if not os.path.realpath(os.path.join(cwd, opath)).startswith(cwd):
1474 parser.error(
Junji Watanabe38b28b02020-04-23 10:23:30 +00001475 '--env-prefix %r path is bad, must be relative and not contain `..`.'
1476 % opath)
Marc-Antoine Ruel19dd8872017-11-28 18:33:39 -05001477 prefixes.setdefault(key, []).append(opath)
1478 options.env_prefix = prefixes
Robert Iannuccibf5f84c2017-11-22 12:56:50 -08001479
nodirbe642ff2016-06-09 15:51:51 -07001480 cipd.validate_cipd_options(parser, options)
1481
vadimsh232f5a82017-01-20 19:23:44 -08001482 install_packages_fn = noop_install_packages
Ye Kuang1d096cb2020-06-26 08:38:21 +00001483 tmp_cipd_cache_dir = None
vadimsh902948e2017-01-20 15:57:32 -08001484 if options.cipd_enabled:
Ye Kuang1d096cb2020-06-26 08:38:21 +00001485 cache_dir = options.cipd_cache
1486 if not cache_dir:
1487 tmp_cipd_cache_dir = six.text_type(tempfile.mkdtemp())
1488 cache_dir = tmp_cipd_cache_dir
Takuto Ikutab7ce0e32019-11-27 23:26:18 +00001489 install_packages_fn = (
Ye Kuang1d096cb2020-06-26 08:38:21 +00001490 lambda run_dir, isolated_dir: install_client_and_packages(
1491 run_dir,
1492 cipd.parse_package_args(options.cipd_packages),
1493 options.cipd_server,
1494 options.cipd_client_package,
1495 options.cipd_client_version,
1496 cache_dir=cache_dir,
1497 isolated_dir=isolated_dir))
nodirbe642ff2016-06-09 15:51:51 -07001498
nodird6160682017-02-02 13:03:35 -08001499 @contextlib.contextmanager
nodir0ae98b32017-05-11 13:21:53 -07001500 def install_named_caches(run_dir):
nodird6160682017-02-02 13:03:35 -08001501 # WARNING: this function depends on "options" variable defined in the outer
1502 # function.
Takuto Ikuta6e2ff962019-10-29 12:35:27 +00001503 assert six.text_type(run_dir), repr(run_dir)
Marc-Antoine Ruel49f9f8d2018-05-24 15:57:06 -04001504 assert os.path.isabs(run_dir), run_dir
Takuto Ikuta6e2ff962019-10-29 12:35:27 +00001505 named_caches = [(os.path.join(run_dir, six.text_type(relpath)), name)
1506 for name, relpath, _ in options.named_caches]
Marc-Antoine Ruelc7a704b2018-08-29 19:02:23 +00001507 for path, name in named_caches:
Marc-Antoine Ruele79ddbf2018-06-13 18:33:07 +00001508 named_cache.install(path, name)
nodird6160682017-02-02 13:03:35 -08001509 try:
1510 yield
1511 finally:
dnje289d132017-07-07 11:16:44 -07001512 # Uninstall each named cache, returning it to the cache pool. If an
1513 # uninstall fails for a given cache, it will remain in the task's
1514 # temporary space, get cleaned up by the Swarming bot, and be lost.
1515 #
1516 # If the Swarming bot cannot clean up the cache, it will handle it like
1517 # any other bot file that could not be removed.
Marc-Antoine Ruelc7a704b2018-08-29 19:02:23 +00001518 for path, name in reversed(named_caches):
Marc-Antoine Ruele79ddbf2018-06-13 18:33:07 +00001519 try:
Marc-Antoine Ruele9558372018-08-03 03:41:22 +00001520 # uninstall() doesn't trim but does call save() implicitly. Trimming
1521 # *must* be done manually via periodic 'run_isolated.py --clean'.
Marc-Antoine Ruele79ddbf2018-06-13 18:33:07 +00001522 named_cache.uninstall(path, name)
1523 except local_caching.NamedCacheError:
1524 logging.exception('Error while removing named cache %r at %r. '
1525 'The cache will be lost.', path, name)
nodirf33b8d62016-10-26 22:34:58 -07001526
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -05001527 extra_args = []
1528 command = []
1529 if options.raw_cmd:
1530 command = args
Marc-Antoine Ruel95068cf2017-12-07 21:35:05 -05001531 if options.relative_cwd:
1532 a = os.path.normpath(os.path.abspath(options.relative_cwd))
1533 if not a.startswith(os.getcwd()):
1534 parser.error(
1535 '--relative-cwd must not try to escape the working directory')
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -05001536 else:
Marc-Antoine Ruel95068cf2017-12-07 21:35:05 -05001537 if options.relative_cwd:
1538 parser.error('--relative-cwd requires --raw-cmd')
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -05001539 extra_args = args
1540
Marc-Antoine Ruel1b65f4e2019-05-02 21:56:58 +00001541 containment_type = subprocess42.Containment.NONE
1542 if options.containment_type == 'AUTO':
1543 containment_type = subprocess42.Containment.AUTO
1544 if options.containment_type == 'JOB_OBJECT':
1545 containment_type = subprocess42.Containment.JOB_OBJECT
1546 containment = subprocess42.Containment(
1547 containment_type=containment_type,
1548 limit_processes=options.limit_processes,
1549 limit_total_committed_memory=options.limit_total_committed_memory)
1550
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -05001551 data = TaskData(
1552 command=command,
Marc-Antoine Ruel95068cf2017-12-07 21:35:05 -05001553 relative_cwd=options.relative_cwd,
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -05001554 extra_args=extra_args,
1555 isolated_hash=options.isolated,
1556 storage=None,
1557 isolate_cache=isolate_cache,
1558 outputs=options.output,
1559 install_named_caches=install_named_caches,
1560 leak_temp_dir=options.leak_temp_dir,
1561 root_dir=_to_unicode(options.root_dir),
1562 hard_timeout=options.hard_timeout,
1563 grace_period=options.grace_period,
1564 bot_file=options.bot_file,
1565 switch_to_account=options.switch_to_account,
1566 install_packages_fn=install_packages_fn,
Takuto Ikuta5c59a842020-01-24 03:05:24 +00001567 use_go_isolated=use_go_isolated,
Takuto Ikuta10cae642020-01-08 08:12:07 +00001568 go_cache_dir=options.cache,
Takuto Ikuta879788c2020-01-10 08:00:26 +00001569 go_cache_policies=local_caching.CachePolicies(
1570 max_cache_size=options.max_cache_size,
1571 min_free_space=options.min_free_space,
1572 max_items=options.max_items,
1573 max_age_secs=None,
1574 ),
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -05001575 env=options.env,
Marc-Antoine Ruel03c6fd12019-04-30 12:12:55 +00001576 env_prefix=options.env_prefix,
Marc-Antoine Ruel1b65f4e2019-05-02 21:56:58 +00001577 lower_priority=bool(options.lower_priority),
1578 containment=containment)
nodirbe642ff2016-06-09 15:51:51 -07001579 try:
nodir90bc8dc2016-06-15 13:35:21 -07001580 if options.isolate_server:
Marc-Antoine Ruelb8513132018-11-20 19:48:53 +00001581 server_ref = isolate_storage.ServerRef(
nodir90bc8dc2016-06-15 13:35:21 -07001582 options.isolate_server, options.namespace)
Marc-Antoine Ruelb8513132018-11-20 19:48:53 +00001583 storage = isolateserver.get_storage(server_ref)
nodir90bc8dc2016-06-15 13:35:21 -07001584 with storage:
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -05001585 data = data._replace(storage=storage)
nodirf33b8d62016-10-26 22:34:58 -07001586 # Hashing schemes used by |storage| and |isolate_cache| MUST match.
Marc-Antoine Ruelb8513132018-11-20 19:48:53 +00001587 assert storage.server_ref.hash_algo == server_ref.hash_algo
Marc-Antoine Ruel7de52592017-12-07 10:41:12 -05001588 return run_tha_test(data, options.json)
1589 return run_tha_test(data, options.json)
Junji Watanabe38b28b02020-04-23 10:23:30 +00001590 except (cipd.Error, local_caching.NamedCacheError,
1591 local_caching.NoMoreSpace) as ex:
Marc-Antoine Ruelf899c482019-10-10 23:32:06 +00001592 print(ex.message, file=sys.stderr)
nodirbe642ff2016-06-09 15:51:51 -07001593 return 1
Ye Kuang1d096cb2020-06-26 08:38:21 +00001594 finally:
1595 if tmp_cipd_cache_dir is not None:
1596 try:
1597 file_path.rmtree(tmp_cipd_cache_dir)
1598 except OSError:
1599 logging.exception('Remove tmp_cipd_cache_dir=%s failed',
1600 tmp_cipd_cache_dir)
1601 # Best effort clean up. Failed to do so doesn't affect the outcome.
maruel@chromium.org9c72d4e2012-09-28 19:20:25 +00001602
1603
1604if __name__ == '__main__':
maruel8e4e40c2016-05-30 06:21:07 -07001605 subprocess42.inhibit_os_error_reporting()
csharp@chromium.orgbfb98742013-03-26 20:28:36 +00001606 # Ensure that we are always running with the correct encoding.
vadimsh@chromium.orga4326472013-08-24 02:05:41 +00001607 fix_encoding.fix_encoding()
Ye Kuang2dd17442020-04-22 08:45:52 +00001608 net.set_user_agent('run_isolated.py/' + __version__)
Marc-Antoine Ruel90c98162013-12-18 15:11:57 -05001609 sys.exit(main(sys.argv[1:]))