maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 1 | #!/usr/bin/env python |
Marc-Antoine Ruel | 8add124 | 2013-11-05 17:28:27 -0500 | [diff] [blame] | 2 | # Copyright 2012 The Swarming Authors. All rights reserved. |
Marc-Antoine Ruel | e98b112 | 2013-11-05 20:27:57 -0500 | [diff] [blame] | 3 | # Use of this source code is governed under the Apache License, Version 2.0 that |
| 4 | # can be found in the LICENSE file. |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 5 | |
maruel@chromium.org | 0cd0b18 | 2012-10-22 13:34:15 +0000 | [diff] [blame] | 6 | """Reads a .isolated, creates a tree of hardlinks and runs the test. |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 7 | |
| 8 | Keeps a local cache. |
| 9 | """ |
| 10 | |
maruel@chromium.org | b7e79a2 | 2013-09-13 01:24:56 +0000 | [diff] [blame] | 11 | __version__ = '0.2' |
maruel@chromium.org | dedbf49 | 2013-09-12 20:42:11 +0000 | [diff] [blame] | 12 | |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 13 | import ctypes |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 14 | import logging |
| 15 | import optparse |
| 16 | import os |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 17 | import re |
| 18 | import shutil |
| 19 | import stat |
| 20 | import subprocess |
| 21 | import sys |
| 22 | import tempfile |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 23 | import time |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 24 | |
vadimsh@chromium.org | a432647 | 2013-08-24 02:05:41 +0000 | [diff] [blame] | 25 | from third_party.depot_tools import fix_encoding |
| 26 | |
vadimsh@chromium.org | 3e97deb | 2013-08-24 00:56:44 +0000 | [diff] [blame] | 27 | from utils import lru |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 28 | from utils import threading_utils |
vadimsh@chromium.org | a432647 | 2013-08-24 02:05:41 +0000 | [diff] [blame] | 29 | from utils import tools |
vadimsh@chromium.org | 3e97deb | 2013-08-24 00:56:44 +0000 | [diff] [blame] | 30 | from utils import zip_package |
vadimsh@chromium.org | 8b9d56b | 2013-08-21 22:24:35 +0000 | [diff] [blame] | 31 | |
maruel@chromium.org | dedbf49 | 2013-09-12 20:42:11 +0000 | [diff] [blame] | 32 | import isolateserver |
maruel@chromium.org | dedbf49 | 2013-09-12 20:42:11 +0000 | [diff] [blame] | 33 | |
vadimsh@chromium.org | a432647 | 2013-08-24 02:05:41 +0000 | [diff] [blame] | 34 | |
vadimsh@chromium.org | 8507106 | 2013-08-21 23:37:45 +0000 | [diff] [blame] | 35 | # Absolute path to this file (can be None if running from zip on Mac). |
| 36 | THIS_FILE_PATH = os.path.abspath(__file__) if __file__ else None |
vadimsh@chromium.org | 8b9d56b | 2013-08-21 22:24:35 +0000 | [diff] [blame] | 37 | |
| 38 | # Directory that contains this file (might be inside zip package). |
vadimsh@chromium.org | 8507106 | 2013-08-21 23:37:45 +0000 | [diff] [blame] | 39 | BASE_DIR = os.path.dirname(THIS_FILE_PATH) if __file__ else None |
vadimsh@chromium.org | 8b9d56b | 2013-08-21 22:24:35 +0000 | [diff] [blame] | 40 | |
| 41 | # Directory that contains currently running script file. |
maruel@chromium.org | 814d23f | 2013-10-01 19:08:00 +0000 | [diff] [blame] | 42 | if zip_package.get_main_script_path(): |
| 43 | MAIN_DIR = os.path.dirname( |
| 44 | os.path.abspath(zip_package.get_main_script_path())) |
| 45 | else: |
| 46 | # This happens when 'import run_isolated' is executed at the python |
| 47 | # interactive prompt, in that case __file__ is undefined. |
| 48 | MAIN_DIR = None |
vadimsh@chromium.org | 8b9d56b | 2013-08-21 22:24:35 +0000 | [diff] [blame] | 49 | |
maruel@chromium.org | 6b365dc | 2012-10-18 19:17:56 +0000 | [diff] [blame] | 50 | # Types of action accepted by link_file(). |
maruel@chromium.org | ba6489b | 2013-07-11 20:23:33 +0000 | [diff] [blame] | 51 | HARDLINK, HARDLINK_WITH_FALLBACK, SYMLINK, COPY = range(1, 5) |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 52 | |
csharp@chromium.org | ff2a466 | 2012-11-21 20:49:32 +0000 | [diff] [blame] | 53 | # The name of the log file to use. |
| 54 | RUN_ISOLATED_LOG_FILE = 'run_isolated.log' |
| 55 | |
csharp@chromium.org | e217f30 | 2012-11-22 16:51:53 +0000 | [diff] [blame] | 56 | # The name of the log to use for the run_test_cases.py command |
vadimsh@chromium.org | 8b9d56b | 2013-08-21 22:24:35 +0000 | [diff] [blame] | 57 | RUN_TEST_CASES_LOG = 'run_test_cases.log' |
csharp@chromium.org | e217f30 | 2012-11-22 16:51:53 +0000 | [diff] [blame] | 58 | |
vadimsh@chromium.org | 87d6326 | 2013-04-04 19:34:21 +0000 | [diff] [blame] | 59 | |
maruel@chromium.org | 9e9ceaa | 2013-04-05 15:42:42 +0000 | [diff] [blame] | 60 | # Used by get_flavor(). |
| 61 | FLAVOR_MAPPING = { |
| 62 | 'cygwin': 'win', |
| 63 | 'win32': 'win', |
| 64 | 'darwin': 'mac', |
| 65 | 'sunos5': 'solaris', |
| 66 | 'freebsd7': 'freebsd', |
| 67 | 'freebsd8': 'freebsd', |
| 68 | } |
| 69 | |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 70 | |
vadimsh@chromium.org | 8b9d56b | 2013-08-21 22:24:35 +0000 | [diff] [blame] | 71 | def get_as_zip_package(executable=True): |
| 72 | """Returns ZipPackage with this module and all its dependencies. |
| 73 | |
| 74 | If |executable| is True will store run_isolated.py as __main__.py so that |
| 75 | zip package is directly executable be python. |
| 76 | """ |
| 77 | # Building a zip package when running from another zip package is |
| 78 | # unsupported and probably unneeded. |
| 79 | assert not zip_package.is_zipped_module(sys.modules[__name__]) |
vadimsh@chromium.org | 8507106 | 2013-08-21 23:37:45 +0000 | [diff] [blame] | 80 | assert THIS_FILE_PATH |
| 81 | assert BASE_DIR |
vadimsh@chromium.org | 8b9d56b | 2013-08-21 22:24:35 +0000 | [diff] [blame] | 82 | package = zip_package.ZipPackage(root=BASE_DIR) |
| 83 | package.add_python_file(THIS_FILE_PATH, '__main__.py' if executable else None) |
maruel@chromium.org | dedbf49 | 2013-09-12 20:42:11 +0000 | [diff] [blame] | 84 | package.add_python_file(os.path.join(BASE_DIR, 'isolateserver.py')) |
vadimsh@chromium.org | 8b9d56b | 2013-08-21 22:24:35 +0000 | [diff] [blame] | 85 | package.add_directory(os.path.join(BASE_DIR, 'third_party')) |
| 86 | package.add_directory(os.path.join(BASE_DIR, 'utils')) |
| 87 | return package |
| 88 | |
| 89 | |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 90 | def get_flavor(): |
| 91 | """Returns the system default flavor. Copied from gyp/pylib/gyp/common.py.""" |
maruel@chromium.org | 9e9ceaa | 2013-04-05 15:42:42 +0000 | [diff] [blame] | 92 | return FLAVOR_MAPPING.get(sys.platform, 'linux') |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 93 | |
| 94 | |
Marc-Antoine Ruel | fb199cf | 2013-11-12 15:38:12 -0500 | [diff] [blame] | 95 | def hardlink(source, link_name): |
| 96 | """Hardlinks a file. |
| 97 | |
| 98 | Add support for os.link() on Windows. |
| 99 | """ |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 100 | if sys.platform == 'win32': |
| 101 | if not ctypes.windll.kernel32.CreateHardLinkW( |
| 102 | unicode(link_name), unicode(source), 0): |
| 103 | raise OSError() |
| 104 | else: |
| 105 | os.link(source, link_name) |
| 106 | |
| 107 | |
| 108 | def readable_copy(outfile, infile): |
| 109 | """Makes a copy of the file that is readable by everyone.""" |
csharp@chromium.org | 59d116d | 2013-07-05 18:04:08 +0000 | [diff] [blame] | 110 | shutil.copy2(infile, outfile) |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 111 | read_enabled_mode = (os.stat(outfile).st_mode | stat.S_IRUSR | |
| 112 | stat.S_IRGRP | stat.S_IROTH) |
| 113 | os.chmod(outfile, read_enabled_mode) |
| 114 | |
| 115 | |
| 116 | def link_file(outfile, infile, action): |
| 117 | """Links a file. The type of link depends on |action|.""" |
| 118 | logging.debug('Mapping %s to %s' % (infile, outfile)) |
maruel@chromium.org | ba6489b | 2013-07-11 20:23:33 +0000 | [diff] [blame] | 119 | if action not in (HARDLINK, HARDLINK_WITH_FALLBACK, SYMLINK, COPY): |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 120 | raise ValueError('Unknown mapping action %s' % action) |
| 121 | if not os.path.isfile(infile): |
maruel@chromium.org | 9958e4a | 2013-09-17 00:01:48 +0000 | [diff] [blame] | 122 | raise isolateserver.MappingError('%s is missing' % infile) |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 123 | if os.path.isfile(outfile): |
maruel@chromium.org | 9958e4a | 2013-09-17 00:01:48 +0000 | [diff] [blame] | 124 | raise isolateserver.MappingError( |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 125 | '%s already exist; insize:%d; outsize:%d' % |
| 126 | (outfile, os.stat(infile).st_size, os.stat(outfile).st_size)) |
| 127 | |
| 128 | if action == COPY: |
| 129 | readable_copy(outfile, infile) |
| 130 | elif action == SYMLINK and sys.platform != 'win32': |
| 131 | # On windows, symlink are converted to hardlink and fails over to copy. |
maruel@chromium.org | f43e68b | 2012-10-15 20:23:10 +0000 | [diff] [blame] | 132 | os.symlink(infile, outfile) # pylint: disable=E1101 |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 133 | else: |
| 134 | try: |
Marc-Antoine Ruel | fb199cf | 2013-11-12 15:38:12 -0500 | [diff] [blame] | 135 | hardlink(infile, outfile) |
maruel@chromium.org | ba6489b | 2013-07-11 20:23:33 +0000 | [diff] [blame] | 136 | except OSError as e: |
| 137 | if action == HARDLINK: |
maruel@chromium.org | 9958e4a | 2013-09-17 00:01:48 +0000 | [diff] [blame] | 138 | raise isolateserver.MappingError( |
maruel@chromium.org | ba6489b | 2013-07-11 20:23:33 +0000 | [diff] [blame] | 139 | 'Failed to hardlink %s to %s: %s' % (infile, outfile, e)) |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 140 | # Probably a different file system. |
maruel@chromium.org | 9e98e43 | 2013-05-31 17:06:51 +0000 | [diff] [blame] | 141 | logging.warning( |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 142 | 'Failed to hardlink, failing back to copy %s to %s' % ( |
| 143 | infile, outfile)) |
| 144 | readable_copy(outfile, infile) |
| 145 | |
| 146 | |
Marc-Antoine Ruel | d2d4d4f | 2013-11-10 14:32:38 -0500 | [diff] [blame] | 147 | def set_read_only(path, read_only): |
| 148 | """Sets or resets the write bit on a file or directory. |
| 149 | |
| 150 | Zaps out access to 'group' and 'others'. |
| 151 | """ |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 152 | mode = os.lstat(path).st_mode |
Marc-Antoine Ruel | 4727bd5 | 2013-11-22 14:44:56 -0500 | [diff] [blame] | 153 | # TODO(maruel): Stop removing GO bits. |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 154 | if read_only: |
| 155 | mode = mode & 0500 |
| 156 | else: |
| 157 | mode = mode | 0200 |
| 158 | if hasattr(os, 'lchmod'): |
| 159 | os.lchmod(path, mode) # pylint: disable=E1101 |
| 160 | else: |
| 161 | if stat.S_ISLNK(mode): |
| 162 | # Skip symlink without lchmod() support. |
Marc-Antoine Ruel | 45dc290 | 2013-12-05 14:54:20 -0500 | [diff] [blame] | 163 | logging.debug( |
| 164 | 'Can\'t change %sw bit on symlink %s', |
| 165 | '-' if read_only else '+', path) |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 166 | return |
| 167 | |
| 168 | # TODO(maruel): Implement proper DACL modification on Windows. |
| 169 | os.chmod(path, mode) |
| 170 | |
| 171 | |
Marc-Antoine Ruel | debee21 | 2013-11-11 14:51:03 -0500 | [diff] [blame] | 172 | def make_tree_read_only(root): |
| 173 | """Makes all the files in the directories read only. |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 174 | |
Marc-Antoine Ruel | debee21 | 2013-11-11 14:51:03 -0500 | [diff] [blame] | 175 | Also makes the directories read only, only if it makes sense on the platform. |
Marc-Antoine Ruel | ccafe0e | 2013-11-08 16:15:36 -0500 | [diff] [blame] | 176 | """ |
Marc-Antoine Ruel | fb199cf | 2013-11-12 15:38:12 -0500 | [diff] [blame] | 177 | logging.debug('make_tree_read_only(%s)', root) |
Marc-Antoine Ruel | ccafe0e | 2013-11-08 16:15:36 -0500 | [diff] [blame] | 178 | assert os.path.isabs(root), root |
Marc-Antoine Ruel | debee21 | 2013-11-11 14:51:03 -0500 | [diff] [blame] | 179 | for dirpath, dirnames, filenames in os.walk(root, topdown=True): |
| 180 | for filename in filenames: |
| 181 | set_read_only(os.path.join(dirpath, filename), True) |
| 182 | if sys.platform != 'win32': |
| 183 | # It must not be done on Windows. |
| 184 | for dirname in dirnames: |
| 185 | set_read_only(os.path.join(dirpath, dirname), True) |
Marc-Antoine Ruel | 4727bd5 | 2013-11-22 14:44:56 -0500 | [diff] [blame] | 186 | # TODO(maruel): Investigate if it makes sense. |
| 187 | #set_read_only(root, True) |
| 188 | |
| 189 | |
| 190 | def make_tree_writeable(root): |
| 191 | """Makes all the files in the directories writeable. |
| 192 | |
| 193 | Also makes the directories writeable, only if it makes sense on the platform. |
| 194 | |
| 195 | It is different from make_tree_deleteable() because it unconditionally affects |
| 196 | the files. |
| 197 | """ |
| 198 | logging.debug('make_tree_writeable(%s)', root) |
| 199 | assert os.path.isabs(root), root |
| 200 | if sys.platform != 'win32': |
| 201 | set_read_only(root, False) |
| 202 | for dirpath, dirnames, filenames in os.walk(root, topdown=True): |
| 203 | for filename in filenames: |
| 204 | set_read_only(os.path.join(dirpath, filename), False) |
| 205 | if sys.platform != 'win32': |
| 206 | # It must not be done on Windows. |
| 207 | for dirname in dirnames: |
| 208 | set_read_only(os.path.join(dirpath, dirname), False) |
Marc-Antoine Ruel | debee21 | 2013-11-11 14:51:03 -0500 | [diff] [blame] | 209 | |
| 210 | |
| 211 | def make_tree_deleteable(root): |
| 212 | """Changes the appropriate permissions so the files in the directories can be |
| 213 | deleted. |
| 214 | |
| 215 | On Windows, the files are modified. On other platforms, modify the directory. |
Marc-Antoine Ruel | 4727bd5 | 2013-11-22 14:44:56 -0500 | [diff] [blame] | 216 | It only does the minimum so the files can be deleted safely. |
Marc-Antoine Ruel | debee21 | 2013-11-11 14:51:03 -0500 | [diff] [blame] | 217 | |
| 218 | Warning on Windows: since file permission is modified, the file node is |
| 219 | modified. This means that for hard-linked files, every directory entry for the |
| 220 | file node has its file permission modified. |
| 221 | """ |
Marc-Antoine Ruel | fb199cf | 2013-11-12 15:38:12 -0500 | [diff] [blame] | 222 | logging.debug('make_tree_deleteable(%s)', root) |
Marc-Antoine Ruel | debee21 | 2013-11-11 14:51:03 -0500 | [diff] [blame] | 223 | assert os.path.isabs(root), root |
| 224 | if sys.platform != 'win32': |
| 225 | set_read_only(root, False) |
| 226 | for dirpath, dirnames, filenames in os.walk(root, topdown=True): |
| 227 | if sys.platform == 'win32': |
| 228 | for filename in filenames: |
| 229 | set_read_only(os.path.join(dirpath, filename), False) |
| 230 | else: |
| 231 | for dirname in dirnames: |
| 232 | set_read_only(os.path.join(dirpath, dirname), False) |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 233 | |
| 234 | |
| 235 | def rmtree(root): |
| 236 | """Wrapper around shutil.rmtree() to retry automatically on Windows.""" |
Marc-Antoine Ruel | debee21 | 2013-11-11 14:51:03 -0500 | [diff] [blame] | 237 | make_tree_deleteable(root) |
Marc-Antoine Ruel | fb199cf | 2013-11-12 15:38:12 -0500 | [diff] [blame] | 238 | logging.info('rmtree(%s)', root) |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 239 | if sys.platform == 'win32': |
| 240 | for i in range(3): |
| 241 | try: |
| 242 | shutil.rmtree(root) |
| 243 | break |
| 244 | except WindowsError: # pylint: disable=E0602 |
Marc-Antoine Ruel | debee21 | 2013-11-11 14:51:03 -0500 | [diff] [blame] | 245 | if i == 2: |
| 246 | raise |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 247 | delay = (i+1)*2 |
| 248 | print >> sys.stderr, ( |
Marc-Antoine Ruel | debee21 | 2013-11-11 14:51:03 -0500 | [diff] [blame] | 249 | 'Failed to delete %s. Maybe the test has subprocess outliving it.' |
| 250 | ' Sleep %d seconds.' % (root, delay)) |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 251 | time.sleep(delay) |
| 252 | else: |
| 253 | shutil.rmtree(root) |
| 254 | |
| 255 | |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 256 | def try_remove(filepath): |
| 257 | """Removes a file without crashing even if it doesn't exist.""" |
| 258 | try: |
Marc-Antoine Ruel | debee21 | 2013-11-11 14:51:03 -0500 | [diff] [blame] | 259 | # TODO(maruel): Not do it unless necessary since it slows this function |
| 260 | # down. |
| 261 | if sys.platform == 'win32': |
| 262 | # Deleting a read-only file will fail if it is read-only. |
| 263 | set_read_only(filepath, False) |
| 264 | else: |
| 265 | # Deleting a read-only file will fail if the directory is read-only. |
| 266 | set_read_only(os.path.dirname(filepath), False) |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 267 | os.remove(filepath) |
| 268 | except OSError: |
| 269 | pass |
| 270 | |
| 271 | |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 272 | def is_same_filesystem(path1, path2): |
| 273 | """Returns True if both paths are on the same filesystem. |
| 274 | |
| 275 | This is required to enable the use of hardlinks. |
| 276 | """ |
| 277 | assert os.path.isabs(path1), path1 |
| 278 | assert os.path.isabs(path2), path2 |
| 279 | if sys.platform == 'win32': |
| 280 | # If the drive letter mismatches, assume it's a separate partition. |
| 281 | # TODO(maruel): It should look at the underlying drive, a drive letter could |
| 282 | # be a mount point to a directory on another drive. |
| 283 | assert re.match(r'^[a-zA-Z]\:\\.*', path1), path1 |
| 284 | assert re.match(r'^[a-zA-Z]\:\\.*', path2), path2 |
| 285 | if path1[0].lower() != path2[0].lower(): |
| 286 | return False |
| 287 | return os.stat(path1).st_dev == os.stat(path2).st_dev |
| 288 | |
| 289 | |
| 290 | def get_free_space(path): |
| 291 | """Returns the number of free bytes.""" |
| 292 | if sys.platform == 'win32': |
| 293 | free_bytes = ctypes.c_ulonglong(0) |
| 294 | ctypes.windll.kernel32.GetDiskFreeSpaceExW( |
| 295 | ctypes.c_wchar_p(path), None, None, ctypes.pointer(free_bytes)) |
| 296 | return free_bytes.value |
maruel@chromium.org | f43e68b | 2012-10-15 20:23:10 +0000 | [diff] [blame] | 297 | # For OSes other than Windows. |
| 298 | f = os.statvfs(path) # pylint: disable=E1101 |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 299 | return f.f_bfree * f.f_frsize |
| 300 | |
| 301 | |
| 302 | def make_temp_dir(prefix, root_dir): |
| 303 | """Returns a temporary directory on the same file system as root_dir.""" |
| 304 | base_temp_dir = None |
| 305 | if not is_same_filesystem(root_dir, tempfile.gettempdir()): |
| 306 | base_temp_dir = os.path.dirname(root_dir) |
| 307 | return tempfile.mkdtemp(prefix=prefix, dir=base_temp_dir) |
| 308 | |
| 309 | |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 310 | class CachePolicies(object): |
| 311 | def __init__(self, max_cache_size, min_free_space, max_items): |
| 312 | """ |
| 313 | Arguments: |
| 314 | - max_cache_size: Trim if the cache gets larger than this value. If 0, the |
| 315 | cache is effectively a leak. |
| 316 | - min_free_space: Trim if disk free space becomes lower than this value. If |
| 317 | 0, it unconditionally fill the disk. |
| 318 | - max_items: Maximum number of items to keep in the cache. If 0, do not |
| 319 | enforce a limit. |
| 320 | """ |
| 321 | self.max_cache_size = max_cache_size |
| 322 | self.min_free_space = min_free_space |
| 323 | self.max_items = max_items |
| 324 | |
| 325 | |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 326 | class DiskCache(isolateserver.LocalCache): |
maruel@chromium.org | e45728d | 2013-09-16 23:23:22 +0000 | [diff] [blame] | 327 | """Stateful LRU cache in a flat hash table in a directory. |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 328 | |
| 329 | Saves its state as json file. |
| 330 | """ |
| 331 | STATE_FILE = 'state.json' |
| 332 | |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 333 | def __init__(self, cache_dir, policies, algo): |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 334 | """ |
| 335 | Arguments: |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 336 | cache_dir: directory where to place the cache. |
| 337 | policies: cache retention policies. |
| 338 | algo: hashing algorithm used. |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 339 | """ |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 340 | super(DiskCache, self).__init__() |
| 341 | self.algo = algo |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 342 | self.cache_dir = cache_dir |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 343 | self.policies = policies |
| 344 | self.state_file = os.path.join(cache_dir, self.STATE_FILE) |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 345 | |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 346 | # All protected methods (starting with '_') except _path should be called |
| 347 | # with this lock locked. |
| 348 | self._lock = threading_utils.LockWithAssert() |
| 349 | self._lru = lru.LRUDict() |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 350 | |
| 351 | # Profiling values. |
| 352 | self._added = [] |
| 353 | self._removed = [] |
| 354 | self._free_disk = 0 |
| 355 | |
vadimsh@chromium.org | a432647 | 2013-08-24 02:05:41 +0000 | [diff] [blame] | 356 | with tools.Profiler('Setup'): |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 357 | with self._lock: |
| 358 | self._load() |
maruel@chromium.org | 4f2ebe4 | 2013-09-19 13:09:08 +0000 | [diff] [blame] | 359 | |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 360 | def __enter__(self): |
| 361 | return self |
| 362 | |
| 363 | def __exit__(self, _exc_type, _exec_value, _traceback): |
vadimsh@chromium.org | a432647 | 2013-08-24 02:05:41 +0000 | [diff] [blame] | 364 | with tools.Profiler('CleanupTrimming'): |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 365 | with self._lock: |
| 366 | self._trim() |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 367 | |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 368 | logging.info( |
| 369 | '%5d (%8dkb) added', |
| 370 | len(self._added), sum(self._added) / 1024) |
| 371 | logging.info( |
| 372 | '%5d (%8dkb) current', |
| 373 | len(self._lru), |
| 374 | sum(self._lru.itervalues()) / 1024) |
| 375 | logging.info( |
| 376 | '%5d (%8dkb) removed', |
| 377 | len(self._removed), sum(self._removed) / 1024) |
| 378 | logging.info( |
| 379 | ' %8dkb free', |
| 380 | self._free_disk / 1024) |
maruel@chromium.org | 4160164 | 2013-09-18 19:40:46 +0000 | [diff] [blame] | 381 | return False |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 382 | |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 383 | def cached_set(self): |
| 384 | with self._lock: |
| 385 | return self._lru.keys_set() |
| 386 | |
| 387 | def touch(self, digest, size): |
Marc-Antoine Ruel | ccafe0e | 2013-11-08 16:15:36 -0500 | [diff] [blame] | 388 | """Verifies an actual file is valid. |
| 389 | |
| 390 | Note that is doesn't compute the hash so it could still be corrupted if the |
| 391 | file size didn't change. |
| 392 | |
| 393 | TODO(maruel): More stringent verification while keeping the check fast. |
| 394 | """ |
| 395 | # Do the check outside the lock. |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 396 | if not isolateserver.is_valid_file(self._path(digest), size): |
| 397 | return False |
| 398 | |
| 399 | # Update it's LRU position. |
| 400 | with self._lock: |
| 401 | if digest not in self._lru: |
| 402 | return False |
| 403 | self._lru.touch(digest) |
| 404 | return True |
| 405 | |
| 406 | def evict(self, digest): |
| 407 | with self._lock: |
| 408 | self._lru.pop(digest) |
| 409 | self._delete_file(digest, isolateserver.UNKNOWN_FILE_SIZE) |
| 410 | |
| 411 | def read(self, digest): |
| 412 | with open(self._path(digest), 'rb') as f: |
| 413 | return f.read() |
| 414 | |
| 415 | def write(self, digest, content): |
| 416 | path = self._path(digest) |
Marc-Antoine Ruel | ccafe0e | 2013-11-08 16:15:36 -0500 | [diff] [blame] | 417 | # A stale broken file may remain. It is possible for the file to have write |
| 418 | # access bit removed which would cause the file_write() call to fail to open |
| 419 | # in write mode. Take no chance here. |
| 420 | try_remove(path) |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 421 | try: |
| 422 | size = isolateserver.file_write(path, content) |
| 423 | except: |
| 424 | # There are two possible places were an exception can occur: |
| 425 | # 1) Inside |content| generator in case of network or unzipping errors. |
| 426 | # 2) Inside file_write itself in case of disk IO errors. |
| 427 | # In any case delete an incomplete file and propagate the exception to |
| 428 | # caller, it will be logged there. |
| 429 | try_remove(path) |
| 430 | raise |
Marc-Antoine Ruel | bcdf6d6 | 2013-11-11 16:45:14 -0500 | [diff] [blame] | 431 | # Make the file read-only in the cache. This has a few side-effects since |
| 432 | # the file node is modified, so every directory entries to this file becomes |
| 433 | # read-only. It's fine here because it is a new file. |
| 434 | set_read_only(path, True) |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 435 | with self._lock: |
| 436 | self._add(digest, size) |
| 437 | |
Marc-Antoine Ruel | fb199cf | 2013-11-12 15:38:12 -0500 | [diff] [blame] | 438 | def hardlink(self, digest, dest, file_mode): |
Marc-Antoine Ruel | ccafe0e | 2013-11-08 16:15:36 -0500 | [diff] [blame] | 439 | """Hardlinks the file to |dest|. |
| 440 | |
| 441 | Note that the file permission bits are on the file node, not the directory |
| 442 | entry, so changing the access bit on any of the directory entries for the |
| 443 | file node will affect them all. |
| 444 | """ |
| 445 | path = self._path(digest) |
| 446 | link_file(dest, path, HARDLINK) |
Marc-Antoine Ruel | fb199cf | 2013-11-12 15:38:12 -0500 | [diff] [blame] | 447 | if file_mode is not None: |
| 448 | # Ignores all other bits. |
| 449 | os.chmod(dest, file_mode & 0500) |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 450 | |
| 451 | def _load(self): |
| 452 | """Loads state of the cache from json file.""" |
| 453 | self._lock.assert_locked() |
| 454 | |
| 455 | if not os.path.isdir(self.cache_dir): |
| 456 | os.makedirs(self.cache_dir) |
Marc-Antoine Ruel | bcdf6d6 | 2013-11-11 16:45:14 -0500 | [diff] [blame] | 457 | else: |
| 458 | # Make sure the cache is read-only. |
| 459 | # TODO(maruel): Calculate the cost and optimize the performance |
| 460 | # accordingly. |
| 461 | make_tree_read_only(self.cache_dir) |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 462 | |
| 463 | # Load state of the cache. |
| 464 | if os.path.isfile(self.state_file): |
| 465 | try: |
| 466 | self._lru = lru.LRUDict.load(self.state_file) |
| 467 | except ValueError as err: |
| 468 | logging.error('Failed to load cache state: %s' % (err,)) |
| 469 | # Don't want to keep broken state file. |
Marc-Antoine Ruel | bcdf6d6 | 2013-11-11 16:45:14 -0500 | [diff] [blame] | 470 | try_remove(self.state_file) |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 471 | |
| 472 | # Ensure that all files listed in the state still exist and add new ones. |
| 473 | previous = self._lru.keys_set() |
| 474 | unknown = [] |
| 475 | for filename in os.listdir(self.cache_dir): |
| 476 | if filename == self.STATE_FILE: |
| 477 | continue |
| 478 | if filename in previous: |
| 479 | previous.remove(filename) |
| 480 | continue |
| 481 | # An untracked file. |
| 482 | if not isolateserver.is_valid_hash(filename, self.algo): |
| 483 | logging.warning('Removing unknown file %s from cache', filename) |
| 484 | try_remove(self._path(filename)) |
| 485 | continue |
| 486 | # File that's not referenced in 'state.json'. |
| 487 | # TODO(vadimsh): Verify its SHA1 matches file name. |
| 488 | logging.warning('Adding unknown file %s to cache', filename) |
| 489 | unknown.append(filename) |
| 490 | |
| 491 | if unknown: |
| 492 | # Add as oldest files. They will be deleted eventually if not accessed. |
| 493 | self._add_oldest_list(unknown) |
| 494 | logging.warning('Added back %d unknown files', len(unknown)) |
| 495 | |
| 496 | if previous: |
| 497 | # Filter out entries that were not found. |
| 498 | logging.warning('Removed %d lost files', len(previous)) |
| 499 | for filename in previous: |
| 500 | self._lru.pop(filename) |
| 501 | self._trim() |
| 502 | |
| 503 | def _save(self): |
| 504 | """Saves the LRU ordering.""" |
| 505 | self._lock.assert_locked() |
Marc-Antoine Ruel | bcdf6d6 | 2013-11-11 16:45:14 -0500 | [diff] [blame] | 506 | try: |
| 507 | set_read_only(self.state_file, False) |
| 508 | except OSError: |
| 509 | pass |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 510 | self._lru.save(self.state_file) |
| 511 | |
| 512 | def _trim(self): |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 513 | """Trims anything we don't know, make sure enough free space exists.""" |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 514 | self._lock.assert_locked() |
| 515 | |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 516 | # Ensure maximum cache size. |
vadimsh@chromium.org | 3e97deb | 2013-08-24 00:56:44 +0000 | [diff] [blame] | 517 | if self.policies.max_cache_size: |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 518 | total_size = sum(self._lru.itervalues()) |
vadimsh@chromium.org | 3e97deb | 2013-08-24 00:56:44 +0000 | [diff] [blame] | 519 | while total_size > self.policies.max_cache_size: |
maruel@chromium.org | e45728d | 2013-09-16 23:23:22 +0000 | [diff] [blame] | 520 | total_size -= self._remove_lru_file() |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 521 | |
| 522 | # Ensure maximum number of items in the cache. |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 523 | if self.policies.max_items and len(self._lru) > self.policies.max_items: |
| 524 | for _ in xrange(len(self._lru) - self.policies.max_items): |
maruel@chromium.org | e45728d | 2013-09-16 23:23:22 +0000 | [diff] [blame] | 525 | self._remove_lru_file() |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 526 | |
| 527 | # Ensure enough free space. |
| 528 | self._free_disk = get_free_space(self.cache_dir) |
maruel@chromium.org | 9e98e43 | 2013-05-31 17:06:51 +0000 | [diff] [blame] | 529 | trimmed_due_to_space = False |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 530 | while ( |
| 531 | self.policies.min_free_space and |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 532 | self._lru and |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 533 | self._free_disk < self.policies.min_free_space): |
maruel@chromium.org | 9e98e43 | 2013-05-31 17:06:51 +0000 | [diff] [blame] | 534 | trimmed_due_to_space = True |
maruel@chromium.org | e45728d | 2013-09-16 23:23:22 +0000 | [diff] [blame] | 535 | self._remove_lru_file() |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 536 | self._free_disk = get_free_space(self.cache_dir) |
maruel@chromium.org | 9e98e43 | 2013-05-31 17:06:51 +0000 | [diff] [blame] | 537 | if trimmed_due_to_space: |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 538 | total = sum(self._lru.itervalues()) |
maruel@chromium.org | 9e98e43 | 2013-05-31 17:06:51 +0000 | [diff] [blame] | 539 | logging.warning( |
| 540 | 'Trimmed due to not enough free disk space: %.1fkb free, %.1fkb ' |
| 541 | 'cache (%.1f%% of its maximum capacity)', |
| 542 | self._free_disk / 1024., |
| 543 | total / 1024., |
| 544 | 100. * self.policies.max_cache_size / float(total), |
| 545 | ) |
maruel@chromium.org | e45728d | 2013-09-16 23:23:22 +0000 | [diff] [blame] | 546 | self._save() |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 547 | |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 548 | def _path(self, digest): |
maruel@chromium.org | 4f2ebe4 | 2013-09-19 13:09:08 +0000 | [diff] [blame] | 549 | """Returns the path to one item.""" |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 550 | return os.path.join(self.cache_dir, digest) |
maruel@chromium.org | e45728d | 2013-09-16 23:23:22 +0000 | [diff] [blame] | 551 | |
| 552 | def _remove_lru_file(self): |
| 553 | """Removes the last recently used file and returns its size.""" |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 554 | self._lock.assert_locked() |
| 555 | digest, size = self._lru.pop_oldest() |
| 556 | self._delete_file(digest, size) |
maruel@chromium.org | e45728d | 2013-09-16 23:23:22 +0000 | [diff] [blame] | 557 | return size |
| 558 | |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 559 | def _add(self, digest, size=isolateserver.UNKNOWN_FILE_SIZE): |
vadimsh@chromium.org | 3e97deb | 2013-08-24 00:56:44 +0000 | [diff] [blame] | 560 | """Adds an item into LRU cache marking it as a newest one.""" |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 561 | self._lock.assert_locked() |
| 562 | if size == isolateserver.UNKNOWN_FILE_SIZE: |
| 563 | size = os.stat(self._path(digest)).st_size |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 564 | self._added.append(size) |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 565 | self._lru.add(digest, size) |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 566 | |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 567 | def _add_oldest_list(self, digests): |
vadimsh@chromium.org | 3e97deb | 2013-08-24 00:56:44 +0000 | [diff] [blame] | 568 | """Adds a bunch of items into LRU cache marking them as oldest ones.""" |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 569 | self._lock.assert_locked() |
vadimsh@chromium.org | 3e97deb | 2013-08-24 00:56:44 +0000 | [diff] [blame] | 570 | pairs = [] |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 571 | for digest in digests: |
| 572 | size = os.stat(self._path(digest)).st_size |
vadimsh@chromium.org | 3e97deb | 2013-08-24 00:56:44 +0000 | [diff] [blame] | 573 | self._added.append(size) |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 574 | pairs.append((digest, size)) |
| 575 | self._lru.batch_insert_oldest(pairs) |
vadimsh@chromium.org | 3e97deb | 2013-08-24 00:56:44 +0000 | [diff] [blame] | 576 | |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 577 | def _delete_file(self, digest, size=isolateserver.UNKNOWN_FILE_SIZE): |
vadimsh@chromium.org | 3e97deb | 2013-08-24 00:56:44 +0000 | [diff] [blame] | 578 | """Deletes cache file from the file system.""" |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 579 | self._lock.assert_locked() |
vadimsh@chromium.org | 3e97deb | 2013-08-24 00:56:44 +0000 | [diff] [blame] | 580 | try: |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 581 | if size == isolateserver.UNKNOWN_FILE_SIZE: |
| 582 | size = os.stat(self._path(digest)).st_size |
Marc-Antoine Ruel | bcdf6d6 | 2013-11-11 16:45:14 -0500 | [diff] [blame] | 583 | try_remove(self._path(digest)) |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 584 | self._removed.append(size) |
vadimsh@chromium.org | 3e97deb | 2013-08-24 00:56:44 +0000 | [diff] [blame] | 585 | except OSError as e: |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 586 | logging.error('Error attempting to delete a file %s:\n%s' % (digest, e)) |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 587 | |
| 588 | |
Marc-Antoine Ruel | def5b80 | 2014-01-08 20:57:12 -0500 | [diff] [blame^] | 589 | def run_tha_test(isolated_hash, storage, cache, algo, outdir, extra_args): |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 590 | """Downloads the dependencies in the cache, hardlinks them into a |outdir| |
| 591 | and runs the executable. |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 592 | """ |
Marc-Antoine Ruel | 3a96379 | 2013-12-11 11:33:49 -0500 | [diff] [blame] | 593 | result = 0 |
maruel@chromium.org | 781ccf6 | 2013-09-17 19:39:47 +0000 | [diff] [blame] | 594 | try: |
maruel@chromium.org | 4f2ebe4 | 2013-09-19 13:09:08 +0000 | [diff] [blame] | 595 | try: |
| 596 | settings = isolateserver.fetch_isolated( |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 597 | isolated_hash=isolated_hash, |
| 598 | storage=storage, |
| 599 | cache=cache, |
| 600 | algo=algo, |
| 601 | outdir=outdir, |
| 602 | os_flavor=get_flavor(), |
| 603 | require_command=True) |
maruel@chromium.org | 4f2ebe4 | 2013-09-19 13:09:08 +0000 | [diff] [blame] | 604 | except isolateserver.ConfigError as e: |
vadimsh@chromium.org | d908a54 | 2013-10-30 01:36:17 +0000 | [diff] [blame] | 605 | tools.report_error(e) |
Marc-Antoine Ruel | 3a96379 | 2013-12-11 11:33:49 -0500 | [diff] [blame] | 606 | result = 1 |
| 607 | return result |
maruel@chromium.org | 781ccf6 | 2013-09-17 19:39:47 +0000 | [diff] [blame] | 608 | |
| 609 | if settings.read_only: |
Marc-Antoine Ruel | ccafe0e | 2013-11-08 16:15:36 -0500 | [diff] [blame] | 610 | # Note that the files themselves are read only anyway. This only inhibits |
| 611 | # creating files or deleting files in the test directory. |
Marc-Antoine Ruel | debee21 | 2013-11-11 14:51:03 -0500 | [diff] [blame] | 612 | make_tree_read_only(outdir) |
Marc-Antoine Ruel | 4727bd5 | 2013-11-22 14:44:56 -0500 | [diff] [blame] | 613 | else: |
| 614 | # This code is safe to keep but DiskCache.touch() must be changed to |
| 615 | # verify the hash of the content of the files it is looking at, so that if |
| 616 | # a test modifies an input file, the file must be deleted. |
| 617 | make_tree_writeable(outdir) |
| 618 | |
maruel@chromium.org | 4f2ebe4 | 2013-09-19 13:09:08 +0000 | [diff] [blame] | 619 | cwd = os.path.normpath(os.path.join(outdir, settings.relative_cwd)) |
Marc-Antoine Ruel | def5b80 | 2014-01-08 20:57:12 -0500 | [diff] [blame^] | 620 | command = settings.command + extra_args |
| 621 | logging.info('Running %s, cwd=%s' % (command, cwd)) |
maruel@chromium.org | 781ccf6 | 2013-09-17 19:39:47 +0000 | [diff] [blame] | 622 | |
| 623 | # TODO(csharp): This should be specified somewhere else. |
| 624 | # TODO(vadimsh): Pass it via 'env_vars' in manifest. |
| 625 | # Add a rotating log file if one doesn't already exist. |
| 626 | env = os.environ.copy() |
maruel@chromium.org | 814d23f | 2013-10-01 19:08:00 +0000 | [diff] [blame] | 627 | if MAIN_DIR: |
| 628 | env.setdefault('RUN_TEST_CASES_LOG_FILE', |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 629 | os.path.join(MAIN_DIR, RUN_TEST_CASES_LOG)) |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 630 | try: |
maruel@chromium.org | 781ccf6 | 2013-09-17 19:39:47 +0000 | [diff] [blame] | 631 | with tools.Profiler('RunTest'): |
Marc-Antoine Ruel | def5b80 | 2014-01-08 20:57:12 -0500 | [diff] [blame^] | 632 | result = subprocess.call(command, cwd=cwd, env=env) |
maruel@chromium.org | 781ccf6 | 2013-09-17 19:39:47 +0000 | [diff] [blame] | 633 | except OSError: |
Marc-Antoine Ruel | def5b80 | 2014-01-08 20:57:12 -0500 | [diff] [blame^] | 634 | tools.report_error('Failed to run %s; cwd=%s' % (command, cwd)) |
Marc-Antoine Ruel | 3a96379 | 2013-12-11 11:33:49 -0500 | [diff] [blame] | 635 | result = 1 |
maruel@chromium.org | 781ccf6 | 2013-09-17 19:39:47 +0000 | [diff] [blame] | 636 | finally: |
| 637 | if outdir: |
Marc-Antoine Ruel | 3a96379 | 2013-12-11 11:33:49 -0500 | [diff] [blame] | 638 | try: |
| 639 | rmtree(outdir) |
| 640 | except OSError: |
| 641 | # Swallow the exception so it doesn't generate an infrastructure error. |
| 642 | # |
| 643 | # It usually happens on Windows when a child process is not properly |
| 644 | # terminated, usually because of a test case starting child processes |
| 645 | # that time out. This causes files to be locked and it becomes |
| 646 | # impossible to delete them. |
| 647 | # |
| 648 | # Only report an infrastructure error if the test didn't fail. This is |
| 649 | # because a swarming bot will likely not reboot. This situation will |
| 650 | # cause accumulation of temporary hardlink trees. |
| 651 | if result: |
| 652 | logging.warning('Leaking %s' % outdir) |
| 653 | else: |
| 654 | raise |
| 655 | return result |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 656 | |
| 657 | |
Marc-Antoine Ruel | 90c9816 | 2013-12-18 15:11:57 -0500 | [diff] [blame] | 658 | def main(args): |
vadimsh@chromium.org | a432647 | 2013-08-24 02:05:41 +0000 | [diff] [blame] | 659 | tools.disable_buffering() |
| 660 | parser = tools.OptionParserWithLogging( |
maruel@chromium.org | dedbf49 | 2013-09-12 20:42:11 +0000 | [diff] [blame] | 661 | usage='%prog <options>', |
| 662 | version=__version__, |
| 663 | log_file=RUN_ISOLATED_LOG_FILE) |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 664 | |
| 665 | group = optparse.OptionGroup(parser, 'Data source') |
| 666 | group.add_option( |
maruel@chromium.org | 0cd0b18 | 2012-10-22 13:34:15 +0000 | [diff] [blame] | 667 | '-s', '--isolated', |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 668 | metavar='FILE', |
| 669 | help='File/url describing what to map or run') |
| 670 | group.add_option( |
| 671 | '-H', '--hash', |
maruel@chromium.org | 0cd0b18 | 2012-10-22 13:34:15 +0000 | [diff] [blame] | 672 | help='Hash of the .isolated to grab from the hash table') |
maruel@chromium.org | b7e79a2 | 2013-09-13 01:24:56 +0000 | [diff] [blame] | 673 | group.add_option( |
maruel@chromium.org | e9403ab | 2013-09-20 18:03:49 +0000 | [diff] [blame] | 674 | '-I', '--isolate-server', |
| 675 | metavar='URL', default='', |
| 676 | help='Isolate server to use') |
maruel@chromium.org | b7e79a2 | 2013-09-13 01:24:56 +0000 | [diff] [blame] | 677 | group.add_option( |
| 678 | '-n', '--namespace', |
| 679 | default='default-gzip', |
| 680 | help='namespace to use when using isolateserver, default: %default') |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 681 | parser.add_option_group(group) |
| 682 | |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 683 | group = optparse.OptionGroup(parser, 'Cache management') |
| 684 | group.add_option( |
| 685 | '--cache', |
| 686 | default='cache', |
| 687 | metavar='DIR', |
| 688 | help='Cache directory, default=%default') |
| 689 | group.add_option( |
| 690 | '--max-cache-size', |
| 691 | type='int', |
| 692 | metavar='NNN', |
| 693 | default=20*1024*1024*1024, |
| 694 | help='Trim if the cache gets larger than this value, default=%default') |
| 695 | group.add_option( |
| 696 | '--min-free-space', |
| 697 | type='int', |
| 698 | metavar='NNN', |
maruel@chromium.org | 9e98e43 | 2013-05-31 17:06:51 +0000 | [diff] [blame] | 699 | default=2*1024*1024*1024, |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 700 | help='Trim if disk free space becomes lower than this value, ' |
| 701 | 'default=%default') |
| 702 | group.add_option( |
| 703 | '--max-items', |
| 704 | type='int', |
| 705 | metavar='NNN', |
| 706 | default=100000, |
| 707 | help='Trim if more than this number of items are in the cache ' |
| 708 | 'default=%default') |
| 709 | parser.add_option_group(group) |
| 710 | |
Marc-Antoine Ruel | 90c9816 | 2013-12-18 15:11:57 -0500 | [diff] [blame] | 711 | options, args = parser.parse_args(args) |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 712 | |
maruel@chromium.org | 0cd0b18 | 2012-10-22 13:34:15 +0000 | [diff] [blame] | 713 | if bool(options.isolated) == bool(options.hash): |
maruel@chromium.org | 5dd75dd | 2012-12-03 15:11:32 +0000 | [diff] [blame] | 714 | logging.debug('One and only one of --isolated or --hash is required.') |
maruel@chromium.org | 0cd0b18 | 2012-10-22 13:34:15 +0000 | [diff] [blame] | 715 | parser.error('One and only one of --isolated or --hash is required.') |
maruel@chromium.org | e9403ab | 2013-09-20 18:03:49 +0000 | [diff] [blame] | 716 | if not options.isolate_server: |
| 717 | parser.error('--isolate-server is required.') |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 718 | |
csharp@chromium.org | ffd8cf0 | 2013-01-09 21:57:38 +0000 | [diff] [blame] | 719 | options.cache = os.path.abspath(options.cache) |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 720 | policies = CachePolicies( |
| 721 | options.max_cache_size, options.min_free_space, options.max_items) |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 722 | algo = isolateserver.get_hash_algo(options.namespace) |
csharp@chromium.org | ffd8cf0 | 2013-01-09 21:57:38 +0000 | [diff] [blame] | 723 | |
maruel@chromium.org | 3e42ce8 | 2013-09-12 18:36:59 +0000 | [diff] [blame] | 724 | try: |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 725 | # |options.cache| may not exist until DiskCache() instance is created. |
| 726 | cache = DiskCache(options.cache, policies, algo) |
| 727 | outdir = make_temp_dir('run_tha_test', options.cache) |
Vadim Shtayura | 3172be5 | 2013-12-03 12:49:05 -0800 | [diff] [blame] | 728 | with isolateserver.get_storage( |
| 729 | options.isolate_server, options.namespace) as storage: |
| 730 | return run_tha_test( |
Marc-Antoine Ruel | def5b80 | 2014-01-08 20:57:12 -0500 | [diff] [blame^] | 731 | options.isolated or options.hash, storage, cache, algo, outdir, args) |
vadimsh@chromium.org | 7b5dae3 | 2013-10-03 16:59:59 +0000 | [diff] [blame] | 732 | except Exception as e: |
maruel@chromium.org | 3e42ce8 | 2013-09-12 18:36:59 +0000 | [diff] [blame] | 733 | # Make sure any exception is logged. |
vadimsh@chromium.org | d908a54 | 2013-10-30 01:36:17 +0000 | [diff] [blame] | 734 | tools.report_error(e) |
maruel@chromium.org | 3e42ce8 | 2013-09-12 18:36:59 +0000 | [diff] [blame] | 735 | logging.exception(e) |
| 736 | return 1 |
maruel@chromium.org | 9c72d4e | 2012-09-28 19:20:25 +0000 | [diff] [blame] | 737 | |
| 738 | |
| 739 | if __name__ == '__main__': |
csharp@chromium.org | bfb9874 | 2013-03-26 20:28:36 +0000 | [diff] [blame] | 740 | # Ensure that we are always running with the correct encoding. |
vadimsh@chromium.org | a432647 | 2013-08-24 02:05:41 +0000 | [diff] [blame] | 741 | fix_encoding.fix_encoding() |
Marc-Antoine Ruel | 90c9816 | 2013-12-18 15:11:57 -0500 | [diff] [blame] | 742 | sys.exit(main(sys.argv[1:])) |