Kuang-che Wu | 6e4beca | 2018-06-27 17:45:02 +0800 | [diff] [blame] | 1 | # -*- coding: utf-8 -*- |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 2 | # Copyright 2017 The Chromium OS Authors. All rights reserved. |
| 3 | # Use of this source code is governed by a BSD-style license that can be |
| 4 | # found in the LICENSE file. |
| 5 | """ChromeOS utility. |
| 6 | |
| 7 | Terminology used in this module. |
| 8 | short_version: ChromeOS version number without milestone, like "9876.0.0". |
| 9 | full_version: ChromeOS version number with milestone, like "R62-9876.0.0". |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 10 | snapshot_version: ChromeOS version number with milestone and snapshot id, |
| 11 | like "R62-9876.0.0-12345". |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 12 | version: if not specified, it could be in short or full format. |
| 13 | """ |
| 14 | |
| 15 | from __future__ import print_function |
Kuang-che Wu | b9705bd | 2018-06-28 17:59:18 +0800 | [diff] [blame] | 16 | import ast |
Kuang-che Wu | 72b5a57 | 2019-10-29 20:37:57 +0800 | [diff] [blame] | 17 | import calendar |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 18 | import datetime |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 19 | import errno |
| 20 | import json |
| 21 | import logging |
| 22 | import os |
| 23 | import re |
Kuang-che Wu | ad5ac7b | 2020-02-20 19:02:52 +0800 | [diff] [blame] | 24 | import shutil |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 25 | import subprocess |
Kuang-che Wu | d558a04 | 2020-06-06 02:11:00 +0800 | [diff] [blame] | 26 | import sys |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 27 | import time |
| 28 | |
Zheng-Jie Chang | 4fabff6 | 2019-12-08 21:54:35 +0800 | [diff] [blame] | 29 | from google.protobuf import json_format |
| 30 | |
Zheng-Jie Chang | 2b6d147 | 2019-11-13 12:40:17 +0800 | [diff] [blame] | 31 | from bisect_kit import buildbucket_util |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 32 | from bisect_kit import cli |
Kuang-che Wu | e4bae0b | 2018-07-19 12:10:14 +0800 | [diff] [blame] | 33 | from bisect_kit import codechange |
Kuang-che Wu | 3eb6b50 | 2018-06-06 16:15:18 +0800 | [diff] [blame] | 34 | from bisect_kit import cr_util |
Kuang-che Wu | e121fae | 2018-11-09 16:18:39 +0800 | [diff] [blame] | 35 | from bisect_kit import errors |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 36 | from bisect_kit import git_util |
Kuang-che Wu | fb55310 | 2018-10-02 18:14:29 +0800 | [diff] [blame] | 37 | from bisect_kit import locking |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 38 | from bisect_kit import repo_util |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 39 | from bisect_kit import util |
| 40 | |
| 41 | logger = logging.getLogger(__name__) |
| 42 | |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 43 | re_chromeos_full_version = r'^R\d+-\d+\.\d+\.\d+$' |
Kuang-che Wu | acb6efd | 2018-04-25 18:52:58 +0800 | [diff] [blame] | 44 | re_chromeos_localbuild_version = r'^\d+\.\d+\.\d{4}_\d\d_\d\d_\d{4}$' |
| 45 | re_chromeos_short_version = r'^\d+\.\d+\.\d+$' |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 46 | re_chromeos_snapshot_version = r'^R\d+-\d+\.\d+\.\d+-\d+$' |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 47 | |
| 48 | gs_archive_path = 'gs://chromeos-image-archive/{board}-release' |
| 49 | gs_release_path = ( |
Kuang-che Wu | 80bf6a5 | 2019-05-31 12:48:06 +0800 | [diff] [blame] | 50 | 'gs://chromeos-releases/{channel}-channel/{boardpath}/{short_version}') |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 51 | |
| 52 | # Assume gsutil is in PATH. |
| 53 | gsutil_bin = 'gsutil' |
Zheng-Jie Chang | b869704 | 2019-10-29 16:03:26 +0800 | [diff] [blame] | 54 | |
| 55 | # Since snapshots with version >= 12618.0.0 have android and chrome version |
| 56 | # info. |
| 57 | snapshot_cutover_version = '12618.0.0' |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 58 | |
Zheng-Jie Chang | 61a645a | 2020-04-29 10:12:13 +0800 | [diff] [blame] | 59 | # current earliest buildbucket buildable versions |
| 60 | # picked from https://crrev.com/c/2072618 |
| 61 | buildbucket_cutover_versions = [ |
| 62 | '12931.0.0', |
| 63 | '12871.26.0', # R81 |
| 64 | '12871.24.2', # stabilize-12871.24.B |
| 65 | '12812.10.0', # factory-excelsior-12812.B |
| 66 | '12768.14.0', # firmware-servo-12768.B |
| 67 | '12739.85.0', # R80 |
| 68 | '12739.67.1', # stabilize-excelsior-12739.67.B |
| 69 | '12692.36.0', # factory-hatch-12692.B |
| 70 | '12672.104.0', # firmware-hatch-12672.B |
| 71 | '12607.110.0', # R79 |
| 72 | '12607.83.2', # stabilize-quickfix-12607.83.B |
| 73 | '12587.59.0', # factory-kukui-12587.B |
| 74 | '12573.78.0', # firmware-kukui-12573.B |
| 75 | '12499.96.0', # R78 |
| 76 | '12422.33.0', # firmware-mistral-12422.B |
| 77 | '12371.190.0', # R77 |
| 78 | '12361.38.0', # factory-mistral-12361.B |
| 79 | '12200.65.0', # firmware-sarien-12200.B |
| 80 | '12105.128.0', # R75 |
| 81 | '12033.82.0', # factory-sarien-12033.B |
| 82 | ] |
| 83 | |
Kuang-che Wu | b9705bd | 2018-06-28 17:59:18 +0800 | [diff] [blame] | 84 | chromeos_root_inside_chroot = '/mnt/host/source' |
| 85 | # relative to chromeos_root |
Kuang-che Wu | 7f82c6f | 2019-08-12 14:29:28 +0800 | [diff] [blame] | 86 | prebuilt_autotest_dir = 'tmp/autotest-prebuilt' |
Kuang-che Wu | 28980b2 | 2019-07-31 19:51:45 +0800 | [diff] [blame] | 87 | # Relative to chromeos root. Images are cached_images_dir/$board/$image_name. |
| 88 | cached_images_dir = 'src/build/images' |
| 89 | test_image_filename = 'chromiumos_test_image.bin' |
Kuang-che Wu | b9705bd | 2018-06-28 17:59:18 +0800 | [diff] [blame] | 90 | |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 91 | VERSION_KEY_CROS_SHORT_VERSION = 'cros_short_version' |
| 92 | VERSION_KEY_CROS_FULL_VERSION = 'cros_full_version' |
| 93 | VERSION_KEY_MILESTONE = 'milestone' |
| 94 | VERSION_KEY_CR_VERSION = 'cr_version' |
Kuang-che Wu | 708310b | 2018-03-28 17:24:34 +0800 | [diff] [blame] | 95 | VERSION_KEY_ANDROID_BUILD_ID = 'android_build_id' |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 96 | VERSION_KEY_ANDROID_BRANCH = 'android_branch' |
Zheng-Jie Chang | 5f9ae4e | 2020-02-07 14:26:06 +0800 | [diff] [blame] | 97 | CROSLAND_URL_TEMPLATE = 'https://crosland.corp.google.com/log/%s..%s' |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 98 | |
| 99 | |
Kuang-che Wu | 9890ce8 | 2018-07-07 15:14:10 +0800 | [diff] [blame] | 100 | class NeedRecreateChrootException(Exception): |
| 101 | """Failed to build ChromeOS because of chroot mismatch or corruption""" |
| 102 | |
| 103 | |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 104 | def is_cros_short_version(s): |
Kuang-che Wu | acb6efd | 2018-04-25 18:52:58 +0800 | [diff] [blame] | 105 | """Determines if `s` is chromeos short version. |
| 106 | |
| 107 | This function doesn't accept version number of local build. |
| 108 | """ |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 109 | return bool(re.match(re_chromeos_short_version, s)) |
| 110 | |
| 111 | |
Kuang-che Wu | acb6efd | 2018-04-25 18:52:58 +0800 | [diff] [blame] | 112 | def is_cros_localbuild_version(s): |
| 113 | """Determines if `s` is chromeos local build version.""" |
| 114 | return bool(re.match(re_chromeos_localbuild_version, s)) |
| 115 | |
| 116 | |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 117 | def is_cros_full_version(s): |
Kuang-che Wu | acb6efd | 2018-04-25 18:52:58 +0800 | [diff] [blame] | 118 | """Determines if `s` is chromeos full version. |
| 119 | |
| 120 | This function doesn't accept version number of local build. |
| 121 | """ |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 122 | return bool(re.match(re_chromeos_full_version, s)) |
| 123 | |
| 124 | |
| 125 | def is_cros_version(s): |
| 126 | """Determines if `s` is chromeos version (either short or full)""" |
| 127 | return is_cros_short_version(s) or is_cros_full_version(s) |
| 128 | |
| 129 | |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 130 | def is_cros_snapshot_version(s): |
| 131 | """Determines if `s` is chromeos snapshot version""" |
| 132 | return bool(re.match(re_chromeos_snapshot_version, s)) |
| 133 | |
| 134 | |
Zheng-Jie Chang | aea4fba | 2020-02-17 17:12:09 +0800 | [diff] [blame] | 135 | def is_cros_version_lesseq(ver1, ver2): |
| 136 | """Determines if ver1 is less or equal to ver2. |
| 137 | |
| 138 | Args: |
| 139 | ver1: a Chrome OS version in short, full, or snapshot format. |
| 140 | ver2: a Chrome OS version in short, full, or snapshot format. |
| 141 | |
| 142 | Returns: |
Zheng-Jie Chang | 61a645a | 2020-04-29 10:12:13 +0800 | [diff] [blame] | 143 | True if ver1 is less or equal to ver2. |
Zheng-Jie Chang | aea4fba | 2020-02-17 17:12:09 +0800 | [diff] [blame] | 144 | """ |
| 145 | assert is_cros_version(ver1) or is_cros_snapshot_version(ver1) |
| 146 | assert is_cros_version(ver2) or is_cros_snapshot_version(ver2) |
| 147 | |
| 148 | ver1 = [int(x) for x in re.split(r'[.-]', ver1) if not x.startswith('R')] |
| 149 | ver2 = [int(x) for x in re.split(r'[.-]', ver2) if not x.startswith('R')] |
| 150 | return ver1 <= ver2 |
| 151 | |
| 152 | |
Zheng-Jie Chang | 61a645a | 2020-04-29 10:12:13 +0800 | [diff] [blame] | 153 | def is_buildbucket_buildable(version): |
| 154 | """Determines if a version is buildable on buildbucket.""" |
| 155 | short_version = version_to_short(version) |
| 156 | # If given version is child of any cutover, then it's buildable |
| 157 | return any([ |
| 158 | util.is_direct_relative_version(x, short_version) and |
| 159 | is_cros_version_lesseq(x, version) for x in buildbucket_cutover_versions |
| 160 | ]) |
| 161 | |
| 162 | |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 163 | def make_cros_full_version(milestone, short_version): |
| 164 | """Makes full_version from milestone and short_version""" |
Kuang-che Wu | acb6efd | 2018-04-25 18:52:58 +0800 | [diff] [blame] | 165 | assert milestone |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 166 | return 'R%s-%s' % (milestone, short_version) |
| 167 | |
| 168 | |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 169 | def make_cros_snapshot_version(milestone, short_version, snapshot_id): |
| 170 | """Makes snapshot version from milestone, short_version and snapshot id""" |
| 171 | return 'R%s-%s-%s' % (milestone, short_version, snapshot_id) |
| 172 | |
| 173 | |
| 174 | def version_split(version): |
| 175 | """Splits full_version or snapshot_version into milestone and short_version""" |
| 176 | assert is_cros_full_version(version) or is_cros_snapshot_version(version) |
| 177 | if is_cros_snapshot_version(version): |
| 178 | return snapshot_version_split(version)[0:2] |
| 179 | milestone, short_version = version.split('-') |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 180 | return milestone[1:], short_version |
| 181 | |
| 182 | |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 183 | def snapshot_version_split(snapshot_version): |
| 184 | """Splits snapshot_version into milestone, short_version and snapshot_id""" |
| 185 | assert is_cros_snapshot_version(snapshot_version) |
| 186 | milestone, shot_version, snapshot_id = snapshot_version.split('-') |
| 187 | return milestone[1:], shot_version, snapshot_id |
| 188 | |
| 189 | |
Zheng-Jie Chang | b869704 | 2019-10-29 16:03:26 +0800 | [diff] [blame] | 190 | def query_snapshot_buildbucket_id(board, snapshot_version): |
| 191 | """Query buildbucket id of a snapshot""" |
| 192 | assert is_cros_snapshot_version(snapshot_version) |
Zheng-Jie Chang | 5402083 | 2020-04-21 15:52:48 +0800 | [diff] [blame] | 193 | path = ('gs://chromeos-image-archive/{board}-snapshot' |
Zheng-Jie Chang | b869704 | 2019-10-29 16:03:26 +0800 | [diff] [blame] | 194 | '/{snapshot_version}-*/image.zip') |
| 195 | output = gsutil_ls( |
| 196 | '-d', |
| 197 | path.format(board=board, snapshot_version=snapshot_version), |
| 198 | ignore_errors=True) |
| 199 | for line in output: |
Zheng-Jie Chang | 5402083 | 2020-04-21 15:52:48 +0800 | [diff] [blame] | 200 | m = re.match(r'.*-snapshot/R\d+-\d+\.\d+\.\d+-\d+-(.+)/image\.zip', line) |
Zheng-Jie Chang | b869704 | 2019-10-29 16:03:26 +0800 | [diff] [blame] | 201 | if m: |
| 202 | return m.group(1) |
| 203 | return None |
| 204 | |
| 205 | |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 206 | def argtype_cros_version(s): |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 207 | if (not is_cros_version(s)) and (not is_cros_snapshot_version(s)): |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 208 | msg = 'invalid cros version' |
Kuang-che Wu | ce2f3be | 2019-10-28 19:44:54 +0800 | [diff] [blame] | 209 | raise cli.ArgTypeError(msg, '9876.0.0, R62-9876.0.0 or R77-12369.0.0-11681') |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 210 | return s |
| 211 | |
| 212 | |
| 213 | def query_dut_lsb_release(host): |
| 214 | """Query /etc/lsb-release of given DUT |
| 215 | |
| 216 | Args: |
| 217 | host: the DUT address |
| 218 | |
| 219 | Returns: |
Kuang-che Wu | 3eb6b50 | 2018-06-06 16:15:18 +0800 | [diff] [blame] | 220 | dict for keys and values of /etc/lsb-release. |
| 221 | |
| 222 | Raises: |
Kuang-che Wu | 4427814 | 2019-03-04 11:33:57 +0800 | [diff] [blame] | 223 | errors.SshConnectionError: cannot connect to host |
| 224 | errors.ExternalError: lsb-release file doesn't exist |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 225 | """ |
| 226 | try: |
Kuang-che Wu | 4427814 | 2019-03-04 11:33:57 +0800 | [diff] [blame] | 227 | output = util.ssh_cmd(host, 'cat', '/etc/lsb-release') |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 228 | except subprocess.CalledProcessError: |
Kuang-che Wu | 4427814 | 2019-03-04 11:33:57 +0800 | [diff] [blame] | 229 | raise errors.ExternalError('unable to read /etc/lsb-release; not a DUT') |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 230 | return dict(re.findall(r'^(\w+)=(.*)$', output, re.M)) |
| 231 | |
| 232 | |
Kuang-che Wu | f134ebf | 2020-08-18 21:06:47 +0800 | [diff] [blame] | 233 | def query_dut_os_release(host): |
| 234 | """Query /etc/os-release of given DUT |
| 235 | |
| 236 | Args: |
| 237 | host: the DUT address |
| 238 | |
| 239 | Returns: |
| 240 | dict for keys and values of /etc/os-release. |
| 241 | |
| 242 | Raises: |
| 243 | errors.SshConnectionError: cannot connect to host |
| 244 | errors.ExternalError: lsb-release file doesn't exist |
| 245 | """ |
| 246 | try: |
| 247 | output = util.ssh_cmd(host, 'cat', '/etc/os-release') |
| 248 | except subprocess.CalledProcessError: |
| 249 | raise errors.ExternalError('unable to read /etc/os-release; not a DUT') |
| 250 | return dict(re.findall(r'^(\w+)=(.*)$', output, re.M)) |
| 251 | |
| 252 | |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 253 | def is_dut(host): |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 254 | """Determines whether a host is a chromeos device. |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 255 | |
| 256 | Args: |
| 257 | host: the DUT address |
| 258 | |
| 259 | Returns: |
| 260 | True if the host is a chromeos device. |
| 261 | """ |
Kuang-che Wu | 4427814 | 2019-03-04 11:33:57 +0800 | [diff] [blame] | 262 | try: |
Kuang-che Wu | f134ebf | 2020-08-18 21:06:47 +0800 | [diff] [blame] | 263 | return query_dut_os_release(host).get('ID') in [ |
| 264 | 'chromiumos', |
| 265 | 'chromeos', |
Kuang-che Wu | 4427814 | 2019-03-04 11:33:57 +0800 | [diff] [blame] | 266 | ] |
| 267 | except (errors.ExternalError, errors.SshConnectionError): |
| 268 | return False |
| 269 | |
| 270 | |
| 271 | def is_good_dut(host): |
| 272 | if not is_dut(host): |
| 273 | return False |
| 274 | |
| 275 | # Sometimes python is broken after 'cros flash'. |
| 276 | try: |
| 277 | util.ssh_cmd(host, 'python', '-c', '1') |
| 278 | return True |
| 279 | except (subprocess.CalledProcessError, errors.SshConnectionError): |
| 280 | return False |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 281 | |
| 282 | |
| 283 | def query_dut_board(host): |
| 284 | """Query board name of a given DUT""" |
| 285 | return query_dut_lsb_release(host).get('CHROMEOS_RELEASE_BOARD') |
| 286 | |
| 287 | |
| 288 | def query_dut_short_version(host): |
Kuang-che Wu | acb6efd | 2018-04-25 18:52:58 +0800 | [diff] [blame] | 289 | """Query short version of a given DUT. |
| 290 | |
| 291 | This function may return version of local build, which |
| 292 | is_cros_short_version() is false. |
| 293 | """ |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 294 | return query_dut_lsb_release(host).get('CHROMEOS_RELEASE_VERSION') |
| 295 | |
| 296 | |
Zheng-Jie Chang | 2608454 | 2020-04-29 06:34:33 +0800 | [diff] [blame] | 297 | def query_dut_prebuilt_version(host): |
| 298 | """Return a snapshot version or short version of a given DUT. |
| 299 | |
| 300 | Args: |
| 301 | host: dut host |
| 302 | |
| 303 | Returns: |
| 304 | Snapshot version or short version. |
| 305 | """ |
| 306 | lsb_release = query_dut_lsb_release(host) |
| 307 | release_version = lsb_release.get('CHROMEOS_RELEASE_VERSION') |
| 308 | builder_path = lsb_release.get('CHROMEOS_RELEASE_BUILDER_PATH') |
| 309 | match = re.match(r'\S+-(?:snapshot|postsubmit)/(R\d+-\d+\.\d+\.\d+-\d+)-\d+', |
| 310 | builder_path) |
| 311 | if match: |
| 312 | return match.group(1) |
| 313 | return release_version |
| 314 | |
| 315 | |
Zheng-Jie Chang | 181be6f | 2020-03-17 16:16:08 +0800 | [diff] [blame] | 316 | def query_dut_is_by_official_builder(host): |
| 317 | """Query if given DUT is build by buildbucket builder""" |
| 318 | return query_dut_lsb_release(host).get('CHROMEOS_RELEASE_BUILD_TYPE', |
| 319 | '').startswith('Continuous Builder') |
| 320 | |
| 321 | |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 322 | def query_dut_boot_id(host, connect_timeout=None): |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 323 | """Query boot id. |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 324 | |
| 325 | Args: |
| 326 | host: DUT address |
| 327 | connect_timeout: connection timeout |
| 328 | |
| 329 | Returns: |
| 330 | boot uuid |
| 331 | """ |
Kuang-che Wu | 4427814 | 2019-03-04 11:33:57 +0800 | [diff] [blame] | 332 | return util.ssh_cmd( |
| 333 | host, |
| 334 | 'cat', |
| 335 | '/proc/sys/kernel/random/boot_id', |
| 336 | connect_timeout=connect_timeout).strip() |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 337 | |
| 338 | |
Kuang-che Wu | 2ac9a92 | 2020-09-03 16:50:12 +0800 | [diff] [blame] | 339 | def reboot(host, force_reboot_callback=None): |
| 340 | """Reboot a DUT and verify. |
| 341 | |
| 342 | Args: |
| 343 | host: DUT address |
| 344 | force_reboot_callback: powerful reboot hook (via servo). This will be |
| 345 | invoked if normal reboot failed. |
| 346 | """ |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 347 | logger.debug('reboot %s', host) |
| 348 | boot_id = query_dut_boot_id(host) |
| 349 | |
Kuang-che Wu | 4427814 | 2019-03-04 11:33:57 +0800 | [diff] [blame] | 350 | try: |
| 351 | util.ssh_cmd(host, 'reboot') |
Kuang-che Wu | 5f662e8 | 2019-03-05 11:49:56 +0800 | [diff] [blame] | 352 | except errors.SshConnectionError: |
| 353 | # Depends on timing, ssh may return failure due to broken pipe, which is |
| 354 | # working as intended. Ignore such kind of errors. |
Kuang-che Wu | 4427814 | 2019-03-04 11:33:57 +0800 | [diff] [blame] | 355 | pass |
Kuang-che Wu | 2ac9a92 | 2020-09-03 16:50:12 +0800 | [diff] [blame] | 356 | |
| 357 | try: |
| 358 | wait_reboot_done(host, boot_id) |
| 359 | except (errors.SshConnectionError, errors.ExternalError): |
| 360 | if force_reboot_callback and force_reboot_callback(host): |
| 361 | wait_reboot_done(host, boot_id) |
| 362 | return |
| 363 | raise |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 364 | |
Kuang-che Wu | 708310b | 2018-03-28 17:24:34 +0800 | [diff] [blame] | 365 | |
| 366 | def wait_reboot_done(host, boot_id): |
Kuang-che Wu | 4fe945b | 2018-03-31 16:46:38 +0800 | [diff] [blame] | 367 | # For dev-mode test image, the reboot time is roughly at least 16 seconds |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 368 | # (dev screen short delay) or more (long delay). |
| 369 | time.sleep(15) |
| 370 | for _ in range(100): |
| 371 | try: |
| 372 | # During boot, DUT does not response and thus ssh may hang a while. So |
| 373 | # set a connect timeout. 3 seconds are enough and 2 are not. It's okay to |
| 374 | # set tight limit because it's inside retry loop. |
| 375 | assert boot_id != query_dut_boot_id(host, connect_timeout=3) |
| 376 | return |
Kuang-che Wu | 5f662e8 | 2019-03-05 11:49:56 +0800 | [diff] [blame] | 377 | except errors.SshConnectionError: |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 378 | logger.debug('reboot not ready? sleep wait 1 sec') |
| 379 | time.sleep(1) |
| 380 | |
Kuang-che Wu | e121fae | 2018-11-09 16:18:39 +0800 | [diff] [blame] | 381 | raise errors.ExternalError('reboot failed?') |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 382 | |
| 383 | |
Kuang-che Wu | 80bf6a5 | 2019-05-31 12:48:06 +0800 | [diff] [blame] | 384 | def gs_release_boardpath(board): |
| 385 | """Normalizes board name for gs://chromeos-releases/ |
| 386 | |
| 387 | This follows behavior of PushImage() in chromite/scripts/pushimage.py |
| 388 | Note, only gs://chromeos-releases/ needs normalization, |
| 389 | gs://chromeos-image-archive does not. |
| 390 | |
| 391 | Args: |
| 392 | board: ChromeOS board name |
| 393 | |
| 394 | Returns: |
| 395 | normalized board name |
| 396 | """ |
| 397 | return board.replace('_', '-') |
| 398 | |
| 399 | |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 400 | def gsutil(*args, **kwargs): |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 401 | """gsutil command line wrapper. |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 402 | |
| 403 | Args: |
| 404 | args: command line arguments passed to gsutil |
| 405 | kwargs: |
| 406 | ignore_errors: if true, return '' for failures, for example 'gsutil ls' |
| 407 | but the path not found. |
| 408 | |
| 409 | Returns: |
| 410 | stdout of gsutil |
| 411 | |
| 412 | Raises: |
Zheng-Jie Chang | b869704 | 2019-10-29 16:03:26 +0800 | [diff] [blame] | 413 | errors.ExternalError: gsutil failed to run |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 414 | subprocess.CalledProcessError: command failed |
| 415 | """ |
| 416 | stderr_lines = [] |
| 417 | try: |
| 418 | return util.check_output( |
| 419 | gsutil_bin, *args, stderr_callback=stderr_lines.append) |
| 420 | except subprocess.CalledProcessError as e: |
| 421 | stderr = ''.join(stderr_lines) |
| 422 | if re.search(r'ServiceException:.* does not have .*access', stderr): |
Kuang-che Wu | e121fae | 2018-11-09 16:18:39 +0800 | [diff] [blame] | 423 | raise errors.ExternalError( |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 424 | 'gsutil failed due to permission. ' + |
| 425 | 'Run "%s config" and follow its instruction. ' % gsutil_bin + |
| 426 | 'Fill any string if it asks for project-id') |
| 427 | if kwargs.get('ignore_errors'): |
| 428 | return '' |
| 429 | raise |
| 430 | except OSError as e: |
| 431 | if e.errno == errno.ENOENT: |
Kuang-che Wu | e121fae | 2018-11-09 16:18:39 +0800 | [diff] [blame] | 432 | raise errors.ExternalError( |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 433 | 'Unable to run %s. gsutil is not installed or not in PATH?' % |
| 434 | gsutil_bin) |
| 435 | raise |
| 436 | |
| 437 | |
| 438 | def gsutil_ls(*args, **kwargs): |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 439 | """gsutil ls. |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 440 | |
| 441 | Args: |
| 442 | args: arguments passed to 'gsutil ls' |
| 443 | kwargs: extra parameters, where |
Kuang-che Wu | 4fe945b | 2018-03-31 16:46:38 +0800 | [diff] [blame] | 444 | ignore_errors: if true, return empty list instead of raising |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 445 | exception, ex. path not found. |
| 446 | |
| 447 | Returns: |
| 448 | list of 'gsutil ls' result. One element for one line of gsutil output. |
| 449 | |
| 450 | Raises: |
| 451 | subprocess.CalledProcessError: gsutil failed, usually means path not found |
| 452 | """ |
| 453 | return gsutil('ls', *args, **kwargs).splitlines() |
| 454 | |
| 455 | |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 456 | def gsutil_stat_update_time(*args, **kwargs): |
| 457 | """Returns the last modified time of a file or multiple files. |
| 458 | |
| 459 | Args: |
| 460 | args: arguments passed to 'gsutil stat'. |
| 461 | kwargs: extra parameters for gsutil. |
| 462 | |
| 463 | Returns: |
| 464 | A integer indicates the last modified timestamp. |
| 465 | |
| 466 | Raises: |
| 467 | subprocess.CalledProcessError: gsutil failed, usually means path not found |
| 468 | errors.ExternalError: update time is not found |
| 469 | """ |
| 470 | result = -1 |
| 471 | # Currently we believe stat always returns a UTC time, and strptime also |
| 472 | # parses a UTC time by default. |
| 473 | time_format = '%a, %d %b %Y %H:%M:%S GMT' |
| 474 | |
| 475 | for line in gsutil('stat', *args, **kwargs).splitlines(): |
| 476 | if ':' not in line: |
| 477 | continue |
Kuang-che Wu | c89f2a2 | 2019-11-26 15:30:50 +0800 | [diff] [blame] | 478 | key, value = line.split(':', 1) |
| 479 | key, value = key.strip(), value.strip() |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 480 | if key != 'Update time': |
| 481 | continue |
| 482 | dt = datetime.datetime.strptime(value, time_format) |
Kuang-che Wu | 72b5a57 | 2019-10-29 20:37:57 +0800 | [diff] [blame] | 483 | unixtime = int(calendar.timegm(dt.utctimetuple())) |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 484 | result = max(result, unixtime) |
| 485 | |
| 486 | if result == -1: |
| 487 | raise errors.ExternalError("didn't find update time") |
| 488 | return result |
| 489 | |
| 490 | |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 491 | def query_milestone_by_version(board, short_version): |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 492 | """Query milestone by ChromeOS version number. |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 493 | |
| 494 | Args: |
| 495 | board: ChromeOS board name |
| 496 | short_version: ChromeOS version number in short format, ex. 9300.0.0 |
| 497 | |
| 498 | Returns: |
| 499 | ChromeOS milestone number (string). For example, '58' for '9300.0.0'. |
| 500 | None if failed. |
| 501 | """ |
| 502 | path = gs_archive_path.format(board=board) + '/R*-' + short_version |
| 503 | for line in gsutil_ls('-d', path, ignore_errors=True): |
| 504 | m = re.search(r'/R(\d+)-', line) |
| 505 | if not m: |
| 506 | continue |
| 507 | return m.group(1) |
| 508 | |
| 509 | for channel in ['canary', 'dev', 'beta', 'stable']: |
| 510 | path = gs_release_path.format( |
Kuang-che Wu | 80bf6a5 | 2019-05-31 12:48:06 +0800 | [diff] [blame] | 511 | channel=channel, |
| 512 | boardpath=gs_release_boardpath(board), |
| 513 | short_version=short_version) |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 514 | for line in gsutil_ls(path, ignore_errors=True): |
| 515 | m = re.search(r'\bR(\d+)-' + short_version, line) |
| 516 | if not m: |
| 517 | continue |
| 518 | return m.group(1) |
| 519 | |
Zheng-Jie Chang | 4e25a8d | 2020-01-08 16:00:13 +0800 | [diff] [blame] | 520 | logger.debug('unable to query milestone of %s for %s', short_version, board) |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 521 | return None |
| 522 | |
| 523 | |
Kuang-che Wu | 80bf6a5 | 2019-05-31 12:48:06 +0800 | [diff] [blame] | 524 | def list_board_names(chromeos_root): |
| 525 | """List board names. |
| 526 | |
| 527 | Args: |
| 528 | chromeos_root: chromeos tree root |
| 529 | |
| 530 | Returns: |
| 531 | list of board names |
| 532 | """ |
| 533 | # Following logic is simplified from chromite/lib/portage_util.py |
| 534 | cros_list_overlays = os.path.join(chromeos_root, |
| 535 | 'chromite/bin/cros_list_overlays') |
| 536 | overlays = util.check_output(cros_list_overlays).splitlines() |
| 537 | result = set() |
| 538 | for overlay in overlays: |
| 539 | conf_file = os.path.join(overlay, 'metadata', 'layout.conf') |
| 540 | name = None |
| 541 | if os.path.exists(conf_file): |
| 542 | for line in open(conf_file): |
| 543 | m = re.match(r'^repo-name\s*=\s*(\S+)\s*$', line) |
| 544 | if m: |
| 545 | name = m.group(1) |
| 546 | break |
| 547 | |
| 548 | if not name: |
| 549 | name_file = os.path.join(overlay, 'profiles', 'repo_name') |
| 550 | if os.path.exists(name_file): |
Kuang-che Wu | a572349 | 2019-11-25 20:59:34 +0800 | [diff] [blame] | 551 | with open(name_file) as f: |
| 552 | name = f.read().strip() |
Kuang-che Wu | 80bf6a5 | 2019-05-31 12:48:06 +0800 | [diff] [blame] | 553 | |
| 554 | if name: |
| 555 | name = re.sub(r'-private$', '', name) |
| 556 | result.add(name) |
| 557 | |
| 558 | return list(result) |
| 559 | |
| 560 | |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 561 | def recognize_version(board, version): |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 562 | """Recognize ChromeOS version. |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 563 | |
| 564 | Args: |
| 565 | board: ChromeOS board name |
| 566 | version: ChromeOS version number in short or full format |
| 567 | |
| 568 | Returns: |
| 569 | (milestone, version in short format) |
| 570 | """ |
| 571 | if is_cros_short_version(version): |
| 572 | milestone = query_milestone_by_version(board, version) |
| 573 | short_version = version |
| 574 | else: |
| 575 | milestone, short_version = version_split(version) |
| 576 | return milestone, short_version |
| 577 | |
| 578 | |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 579 | def extract_major_version(version): |
| 580 | """Converts a version to its major version. |
| 581 | |
| 582 | Args: |
Kuang-che Wu | 9501f34 | 2019-11-15 17:15:21 +0800 | [diff] [blame] | 583 | version: ChromeOS version number or snapshot version |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 584 | |
| 585 | Returns: |
| 586 | major version number in string format |
| 587 | """ |
| 588 | version = version_to_short(version) |
| 589 | m = re.match(r'^(\d+)\.\d+\.\d+$', version) |
| 590 | return m.group(1) |
| 591 | |
| 592 | |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 593 | def version_to_short(version): |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 594 | """Convert ChromeOS version number to short format. |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 595 | |
| 596 | Args: |
| 597 | version: ChromeOS version number in short or full format |
| 598 | |
| 599 | Returns: |
| 600 | version number in short format |
| 601 | """ |
| 602 | if is_cros_short_version(version): |
| 603 | return version |
| 604 | _, short_version = version_split(version) |
| 605 | return short_version |
| 606 | |
| 607 | |
| 608 | def version_to_full(board, version): |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 609 | """Convert ChromeOS version number to full format. |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 610 | |
| 611 | Args: |
| 612 | board: ChromeOS board name |
| 613 | version: ChromeOS version number in short or full format |
| 614 | |
| 615 | Returns: |
| 616 | version number in full format |
| 617 | """ |
Zheng-Jie Chang | c47af3a | 2019-11-11 17:28:58 +0800 | [diff] [blame] | 618 | if is_cros_snapshot_version(version): |
| 619 | milestone, short_version, _ = snapshot_version_split(version) |
| 620 | return make_cros_full_version(milestone, short_version) |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 621 | if is_cros_full_version(version): |
| 622 | return version |
| 623 | milestone = query_milestone_by_version(board, version) |
Kuang-che Wu | 0205f05 | 2019-05-23 12:48:37 +0800 | [diff] [blame] | 624 | assert milestone, 'incorrect board=%s or version=%s ?' % (board, version) |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 625 | return make_cros_full_version(milestone, version) |
| 626 | |
| 627 | |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 628 | def list_snapshots_from_image_archive(board, major_version): |
Kuang-che Wu | 9501f34 | 2019-11-15 17:15:21 +0800 | [diff] [blame] | 629 | """List ChromeOS snapshot image available from gs://chromeos-image-archive. |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 630 | |
| 631 | Args: |
| 632 | board: ChromeOS board |
| 633 | major_version: ChromeOS major version |
| 634 | |
| 635 | Returns: |
| 636 | list of (version, gs_path): |
| 637 | version: Chrome OS snapshot version |
| 638 | gs_path: gs path of test image |
| 639 | """ |
| 640 | |
Zheng-Jie Chang | 43a0841 | 2019-12-05 17:05:45 +0800 | [diff] [blame] | 641 | def extract_snapshot_id(result): |
| 642 | m = re.match(r'^R\d+-\d+\.\d+\.\d+-(\d+)', result[0]) |
| 643 | assert m |
| 644 | return int(m.group(1)) |
| 645 | |
Zheng-Jie Chang | a331b87 | 2020-01-06 17:09:32 +0800 | [diff] [blame] | 646 | short_version = '%s.0.0' % major_version |
| 647 | milestone = query_milestone_by_version(board, short_version) |
| 648 | if not milestone: |
| 649 | milestone = '*' |
| 650 | |
Zheng-Jie Chang | 5402083 | 2020-04-21 15:52:48 +0800 | [diff] [blame] | 651 | path = ('gs://chromeos-image-archive/{board}-snapshot/R{milestone}-' |
Zheng-Jie Chang | a331b87 | 2020-01-06 17:09:32 +0800 | [diff] [blame] | 652 | '{short_version}-*/image.zip') |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 653 | result = [] |
| 654 | output = gsutil_ls( |
Zheng-Jie Chang | a331b87 | 2020-01-06 17:09:32 +0800 | [diff] [blame] | 655 | path.format( |
| 656 | board=board, milestone=milestone, short_version=short_version), |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 657 | ignore_errors=True) |
| 658 | |
Zheng-Jie Chang | a331b87 | 2020-01-06 17:09:32 +0800 | [diff] [blame] | 659 | for gs_path in sorted(output): |
| 660 | m = re.match(r'^gs:\S+(R\d+-\d+\.\d+\.\d+-\d+)', gs_path) |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 661 | if m: |
| 662 | snapshot_version = m.group(1) |
Zheng-Jie Chang | 43a0841 | 2019-12-05 17:05:45 +0800 | [diff] [blame] | 663 | # we should skip if there is duplicate snapshot |
| 664 | if result and result[-1][0] == snapshot_version: |
| 665 | continue |
Zheng-Jie Chang | 1ae64c3 | 2020-03-10 14:08:28 +0800 | [diff] [blame] | 666 | |
| 667 | # b/151054108: snapshot version in [29288, 29439] is broken |
| 668 | _, _, snapshot_id = snapshot_version_split(snapshot_version) |
| 669 | if 29288 <= int(snapshot_id) <= 29439: |
| 670 | continue |
| 671 | |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 672 | result.append((snapshot_version, gs_path)) |
Zheng-Jie Chang | 43a0841 | 2019-12-05 17:05:45 +0800 | [diff] [blame] | 673 | |
| 674 | # sort by its snapshot_id |
| 675 | result.sort(key=extract_snapshot_id) |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 676 | return result |
| 677 | |
| 678 | |
Kuang-che Wu | 575dc44 | 2019-03-05 10:30:55 +0800 | [diff] [blame] | 679 | def list_prebuilt_from_image_archive(board): |
| 680 | """Lists ChromeOS prebuilt image available from gs://chromeos-image-archive. |
| 681 | |
| 682 | gs://chromeos-image-archive contains only recent builds (in two years). |
| 683 | We prefer this function to list_prebuilt_from_chromeos_releases() because |
| 684 | - this is what "cros flash" supports directly. |
| 685 | - the paths have milestone information, so we don't need to do slow query |
| 686 | by ourselves. |
| 687 | |
| 688 | Args: |
| 689 | board: ChromeOS board name |
| 690 | |
| 691 | Returns: |
| 692 | list of (version, gs_path): |
| 693 | version: Chrome OS version in full format |
| 694 | gs_path: gs path of test image |
| 695 | """ |
| 696 | result = [] |
| 697 | for line in gsutil_ls(gs_archive_path.format(board=board)): |
| 698 | m = re.match(r'^gs:\S+(R\d+-\d+\.\d+\.\d+)', line) |
| 699 | if m: |
| 700 | full_version = m.group(1) |
| 701 | test_image = 'chromiumos_test_image.tar.xz' |
| 702 | assert line.endswith('/') |
| 703 | gs_path = line + test_image |
| 704 | result.append((full_version, gs_path)) |
| 705 | return result |
| 706 | |
| 707 | |
| 708 | def list_prebuilt_from_chromeos_releases(board): |
| 709 | """Lists ChromeOS versions available from gs://chromeos-releases. |
| 710 | |
| 711 | gs://chromeos-releases contains more builds. However, 'cros flash' doesn't |
| 712 | support it. |
| 713 | |
| 714 | Args: |
| 715 | board: ChromeOS board name |
| 716 | |
| 717 | Returns: |
| 718 | list of (version, gs_path): |
| 719 | version: Chrome OS version in short format |
| 720 | gs_path: gs path of test image (with wildcard) |
| 721 | """ |
| 722 | result = [] |
| 723 | for line in gsutil_ls( |
Kuang-che Wu | 80bf6a5 | 2019-05-31 12:48:06 +0800 | [diff] [blame] | 724 | gs_release_path.format( |
| 725 | channel='*', boardpath=gs_release_boardpath(board), short_version=''), |
Kuang-che Wu | 575dc44 | 2019-03-05 10:30:55 +0800 | [diff] [blame] | 726 | ignore_errors=True): |
| 727 | m = re.match(r'gs:\S+/(\d+\.\d+\.\d+)/$', line) |
| 728 | if m: |
| 729 | short_version = m.group(1) |
| 730 | test_image = 'ChromeOS-test-R*-{short_version}-{board}.tar.xz'.format( |
| 731 | short_version=short_version, board=board) |
| 732 | gs_path = line + test_image |
| 733 | result.append((short_version, gs_path)) |
| 734 | return result |
| 735 | |
| 736 | |
Kuang-che Wu | e180840 | 2020-01-06 20:27:45 +0800 | [diff] [blame] | 737 | def has_test_image(board, version): |
| 738 | if is_cros_snapshot_version(version): |
| 739 | return bool(query_snapshot_buildbucket_id(board, version)) |
| 740 | |
| 741 | full_version = version_to_full(board, version) |
| 742 | short_version = version_to_short(version) |
| 743 | paths = [ |
| 744 | gs_archive_path.format(board=board) + |
| 745 | '/%s/chromiumos_test_image.tar.xz' % full_version, |
| 746 | gs_release_path.format( |
| 747 | channel='*', |
| 748 | boardpath=gs_release_boardpath(board), |
| 749 | short_version=short_version), |
| 750 | ] |
| 751 | |
| 752 | for path in paths: |
| 753 | if gsutil_ls(path, ignore_errors=True): |
| 754 | return True |
| 755 | return False |
| 756 | |
| 757 | |
Kuang-che Wu | 575dc44 | 2019-03-05 10:30:55 +0800 | [diff] [blame] | 758 | def list_chromeos_prebuilt_versions(board, |
| 759 | old, |
| 760 | new, |
| 761 | only_good_build=True, |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 762 | include_older_build=True, |
| 763 | use_snapshot=False): |
Kuang-che Wu | 575dc44 | 2019-03-05 10:30:55 +0800 | [diff] [blame] | 764 | """Lists ChromeOS version numbers with prebuilt between given range |
| 765 | |
| 766 | Args: |
| 767 | board: ChromeOS board name |
| 768 | old: start version (inclusive) |
| 769 | new: end version (inclusive) |
| 770 | only_good_build: only if test image is available |
| 771 | include_older_build: include prebuilt in gs://chromeos-releases |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 772 | use_snapshot: return snapshot versions if found |
Kuang-che Wu | 575dc44 | 2019-03-05 10:30:55 +0800 | [diff] [blame] | 773 | |
| 774 | Returns: |
| 775 | list of sorted version numbers (in full format) between [old, new] range |
| 776 | (inclusive). |
| 777 | """ |
Zheng-Jie Chang | aea4fba | 2020-02-17 17:12:09 +0800 | [diff] [blame] | 778 | old_short = version_to_short(old) |
| 779 | new_short = version_to_short(new) |
Kuang-che Wu | 575dc44 | 2019-03-05 10:30:55 +0800 | [diff] [blame] | 780 | |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 781 | rev_map = { |
| 782 | } # dict: short version -> list of (short/full or snapshot version, gs path) |
Kuang-che Wu | 575dc44 | 2019-03-05 10:30:55 +0800 | [diff] [blame] | 783 | for full_version, gs_path in list_prebuilt_from_image_archive(board): |
| 784 | short_version = version_to_short(full_version) |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 785 | rev_map[short_version] = [(full_version, gs_path)] |
Kuang-che Wu | 575dc44 | 2019-03-05 10:30:55 +0800 | [diff] [blame] | 786 | |
Zheng-Jie Chang | aea4fba | 2020-02-17 17:12:09 +0800 | [diff] [blame] | 787 | if include_older_build and old_short not in rev_map: |
Kuang-che Wu | 575dc44 | 2019-03-05 10:30:55 +0800 | [diff] [blame] | 788 | for short_version, gs_path in list_prebuilt_from_chromeos_releases(board): |
| 789 | if short_version not in rev_map: |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 790 | rev_map[short_version] = [(short_version, gs_path)] |
| 791 | |
| 792 | if use_snapshot: |
| 793 | for major_version in range( |
| 794 | int(extract_major_version(old)), |
| 795 | int(extract_major_version(new)) + 1): |
| 796 | short_version = '%s.0.0' % major_version |
Zheng-Jie Chang | c47af3a | 2019-11-11 17:28:58 +0800 | [diff] [blame] | 797 | next_short_version = '%s.0.0' % (major_version + 1) |
Zheng-Jie Chang | b869704 | 2019-10-29 16:03:26 +0800 | [diff] [blame] | 798 | # If current version is smaller than cutover, ignore it as it might not |
| 799 | # contain enough information for continuing android and chrome bisection. |
| 800 | if not util.is_version_lesseq(snapshot_cutover_version, short_version): |
| 801 | continue |
Zheng-Jie Chang | c47af3a | 2019-11-11 17:28:58 +0800 | [diff] [blame] | 802 | |
| 803 | # Given the fact that snapshots are images between two release versions. |
| 804 | # Adding snapshots of 12345.0.0 should be treated as adding commits |
| 805 | # between [12345.0.0, 12346.0.0). |
| 806 | # So in the following lines we check two facts: |
| 807 | # 1) If 12346.0.0(next_short_version) is a version between old and new |
Zheng-Jie Chang | aea4fba | 2020-02-17 17:12:09 +0800 | [diff] [blame] | 808 | if not util.is_direct_relative_version(next_short_version, old_short): |
Zheng-Jie Chang | c47af3a | 2019-11-11 17:28:58 +0800 | [diff] [blame] | 809 | continue |
Zheng-Jie Chang | aea4fba | 2020-02-17 17:12:09 +0800 | [diff] [blame] | 810 | if not util.is_direct_relative_version(next_short_version, new_short): |
Zheng-Jie Chang | c47af3a | 2019-11-11 17:28:58 +0800 | [diff] [blame] | 811 | continue |
| 812 | # 2) If 12345.0.0(short_version) is a version between old and new |
Zheng-Jie Chang | aea4fba | 2020-02-17 17:12:09 +0800 | [diff] [blame] | 813 | if not util.is_direct_relative_version(short_version, old_short): |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 814 | continue |
Zheng-Jie Chang | aea4fba | 2020-02-17 17:12:09 +0800 | [diff] [blame] | 815 | if not util.is_direct_relative_version(short_version, new_short): |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 816 | continue |
Zheng-Jie Chang | c47af3a | 2019-11-11 17:28:58 +0800 | [diff] [blame] | 817 | |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 818 | snapshots = list_snapshots_from_image_archive(board, str(major_version)) |
| 819 | if snapshots: |
| 820 | # if snapshots found, we can append them after the release version, |
| 821 | # so the prebuilt image list of this version will be |
| 822 | # release_image, snapshot1, snapshot2,... |
Zheng-Jie Chang | b869704 | 2019-10-29 16:03:26 +0800 | [diff] [blame] | 823 | if short_version not in rev_map: |
| 824 | rev_map[short_version] = [] |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 825 | rev_map[short_version] += snapshots |
Kuang-che Wu | 575dc44 | 2019-03-05 10:30:55 +0800 | [diff] [blame] | 826 | |
| 827 | result = [] |
| 828 | for rev in sorted(rev_map, key=util.version_key_func): |
Zheng-Jie Chang | aea4fba | 2020-02-17 17:12:09 +0800 | [diff] [blame] | 829 | if not util.is_direct_relative_version(new_short, rev): |
Kuang-che Wu | 575dc44 | 2019-03-05 10:30:55 +0800 | [diff] [blame] | 830 | continue |
Zheng-Jie Chang | aea4fba | 2020-02-17 17:12:09 +0800 | [diff] [blame] | 831 | if not util.is_version_lesseq(old_short, rev): |
Kuang-che Wu | 575dc44 | 2019-03-05 10:30:55 +0800 | [diff] [blame] | 832 | continue |
Zheng-Jie Chang | aea4fba | 2020-02-17 17:12:09 +0800 | [diff] [blame] | 833 | if not util.is_version_lesseq(rev, new_short): |
Kuang-che Wu | 575dc44 | 2019-03-05 10:30:55 +0800 | [diff] [blame] | 834 | continue |
| 835 | |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 836 | for version, gs_path in rev_map[rev]: |
Kuang-che Wu | 575dc44 | 2019-03-05 10:30:55 +0800 | [diff] [blame] | 837 | |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 838 | # version_to_full() and gsutil_ls() may take long time if versions are a |
| 839 | # lot. This is acceptable because we usually bisect only short range. |
Kuang-che Wu | 575dc44 | 2019-03-05 10:30:55 +0800 | [diff] [blame] | 840 | |
Zheng-Jie Chang | a331b87 | 2020-01-06 17:09:32 +0800 | [diff] [blame] | 841 | if only_good_build and not is_cros_snapshot_version(version): |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 842 | gs_result = gsutil_ls(gs_path, ignore_errors=True) |
| 843 | if not gs_result: |
| 844 | logger.warning('%s is not a good build, ignore', version) |
| 845 | continue |
| 846 | assert len(gs_result) == 1 |
| 847 | m = re.search(r'(R\d+-\d+\.\d+\.\d+)', gs_result[0]) |
| 848 | if not m: |
| 849 | logger.warning('format of image path is unexpected: %s', gs_result[0]) |
| 850 | continue |
Zheng-Jie Chang | a331b87 | 2020-01-06 17:09:32 +0800 | [diff] [blame] | 851 | version = m.group(1) |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 852 | elif is_cros_short_version(version): |
| 853 | version = version_to_full(board, version) |
Kuang-che Wu | 575dc44 | 2019-03-05 10:30:55 +0800 | [diff] [blame] | 854 | |
Zheng-Jie Chang | aea4fba | 2020-02-17 17:12:09 +0800 | [diff] [blame] | 855 | if is_cros_version_lesseq(old, version) and is_cros_version_lesseq( |
| 856 | version, new): |
| 857 | result.append(version) |
Kuang-che Wu | 575dc44 | 2019-03-05 10:30:55 +0800 | [diff] [blame] | 858 | |
| 859 | return result |
| 860 | |
| 861 | |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 862 | def prepare_snapshot_image(chromeos_root, board, snapshot_version): |
| 863 | """Prepare chromeos snapshot image. |
| 864 | |
| 865 | Args: |
| 866 | chromeos_root: chromeos tree root |
| 867 | board: ChromeOS board name |
| 868 | snapshot_version: ChromeOS snapshot version number |
| 869 | |
| 870 | Returns: |
| 871 | local file path of test image relative to chromeos_root |
| 872 | """ |
| 873 | assert is_cros_snapshot_version(snapshot_version) |
| 874 | milestone, short_version, snapshot_id = snapshot_version_split( |
| 875 | snapshot_version) |
| 876 | full_version = make_cros_full_version(milestone, short_version) |
| 877 | tmp_dir = os.path.join( |
| 878 | chromeos_root, 'tmp', |
| 879 | 'ChromeOS-test-%s-%s-%s' % (full_version, board, snapshot_id)) |
| 880 | if not os.path.exists(tmp_dir): |
| 881 | os.makedirs(tmp_dir) |
| 882 | |
Zheng-Jie Chang | 5402083 | 2020-04-21 15:52:48 +0800 | [diff] [blame] | 883 | gs_path = ('gs://chromeos-image-archive/{board}-snapshot/' + |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 884 | '{snapshot_version}-*/image.zip') |
| 885 | gs_path = gs_path.format(board=board, snapshot_version=snapshot_version) |
| 886 | |
Zheng-Jie Chang | c47af3a | 2019-11-11 17:28:58 +0800 | [diff] [blame] | 887 | full_path = os.path.join(tmp_dir, test_image_filename) |
| 888 | rel_path = os.path.relpath(full_path, chromeos_root) |
| 889 | if os.path.exists(full_path): |
| 890 | return rel_path |
| 891 | |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 892 | files = gsutil_ls(gs_path, ignore_errors=True) |
Zheng-Jie Chang | eb7308f | 2019-12-05 14:25:05 +0800 | [diff] [blame] | 893 | if len(files) >= 1: |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 894 | gs_path = files[0] |
| 895 | gsutil('cp', gs_path, tmp_dir) |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 896 | util.check_call( |
| 897 | 'unzip', '-j', 'image.zip', test_image_filename, cwd=tmp_dir) |
| 898 | os.remove(os.path.join(tmp_dir, 'image.zip')) |
Zheng-Jie Chang | c47af3a | 2019-11-11 17:28:58 +0800 | [diff] [blame] | 899 | return rel_path |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 900 | |
Zheng-Jie Chang | c47af3a | 2019-11-11 17:28:58 +0800 | [diff] [blame] | 901 | assert False |
Kuang-che Wu | a7ddf9b | 2019-11-25 18:59:57 +0800 | [diff] [blame] | 902 | return None |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 903 | |
| 904 | |
Kuang-che Wu | 28980b2 | 2019-07-31 19:51:45 +0800 | [diff] [blame] | 905 | def prepare_prebuilt_image(chromeos_root, board, version): |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 906 | """Prepare chromeos prebuilt image. |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 907 | |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 908 | It searches for xbuddy image which "cros flash" can use, or fetch image to |
| 909 | local disk. |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 910 | |
| 911 | Args: |
Kuang-che Wu | 28980b2 | 2019-07-31 19:51:45 +0800 | [diff] [blame] | 912 | chromeos_root: chromeos tree root |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 913 | board: ChromeOS board name |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 914 | version: ChromeOS version number in short or full format |
| 915 | |
| 916 | Returns: |
Kuang-che Wu | 28980b2 | 2019-07-31 19:51:45 +0800 | [diff] [blame] | 917 | xbuddy path or file path (relative to chromeos_root) |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 918 | """ |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 919 | assert is_cros_version(version) |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 920 | full_version = version_to_full(board, version) |
| 921 | short_version = version_to_short(full_version) |
| 922 | |
| 923 | image_path = None |
| 924 | gs_path = gs_archive_path.format(board=board) + '/' + full_version |
| 925 | if gsutil_ls('-d', gs_path, ignore_errors=True): |
| 926 | image_path = 'xbuddy://remote/{board}/{full_version}/test'.format( |
| 927 | board=board, full_version=full_version) |
| 928 | else: |
Kuang-che Wu | 28980b2 | 2019-07-31 19:51:45 +0800 | [diff] [blame] | 929 | tmp_dir = os.path.join(chromeos_root, 'tmp', |
| 930 | 'ChromeOS-test-%s-%s' % (full_version, board)) |
Zheng-Jie Chang | c47af3a | 2019-11-11 17:28:58 +0800 | [diff] [blame] | 931 | full_path = os.path.join(tmp_dir, test_image_filename) |
| 932 | rel_path = os.path.relpath(full_path, chromeos_root) |
| 933 | if os.path.exists(full_path): |
| 934 | return rel_path |
| 935 | |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 936 | if not os.path.exists(tmp_dir): |
| 937 | os.makedirs(tmp_dir) |
| 938 | # gs://chromeos-releases may have more old images than |
Kuang-che Wu | 4fe945b | 2018-03-31 16:46:38 +0800 | [diff] [blame] | 939 | # gs://chromeos-image-archive, but 'cros flash' doesn't support it. We have |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 940 | # to fetch the image by ourselves |
| 941 | for channel in ['canary', 'dev', 'beta', 'stable']: |
| 942 | fn = 'ChromeOS-test-{full_version}-{board}.tar.xz'.format( |
| 943 | full_version=full_version, board=board) |
| 944 | gs_path = gs_release_path.format( |
Kuang-che Wu | 80bf6a5 | 2019-05-31 12:48:06 +0800 | [diff] [blame] | 945 | channel=channel, |
| 946 | boardpath=gs_release_boardpath(board), |
| 947 | short_version=short_version) |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 948 | gs_path += '/' + fn |
| 949 | if gsutil_ls(gs_path, ignore_errors=True): |
| 950 | # TODO(kcwu): delete tmp |
| 951 | gsutil('cp', gs_path, tmp_dir) |
| 952 | util.check_call('tar', 'Jxvf', fn, cwd=tmp_dir) |
Zheng-Jie Chang | c47af3a | 2019-11-11 17:28:58 +0800 | [diff] [blame] | 953 | image_path = os.path.relpath(full_path, chromeos_root) |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 954 | break |
| 955 | |
| 956 | assert image_path |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 957 | return image_path |
| 958 | |
| 959 | |
| 960 | def cros_flash(chromeos_root, |
| 961 | host, |
| 962 | board, |
| 963 | image_path, |
| 964 | version=None, |
| 965 | clobber_stateful=False, |
Kuang-che Wu | 2ac9a92 | 2020-09-03 16:50:12 +0800 | [diff] [blame] | 966 | disable_rootfs_verification=True, |
| 967 | force_reboot_callback=None): |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 968 | """Flash a DUT with given ChromeOS image. |
| 969 | |
| 970 | This is implemented by 'cros flash' command line. |
| 971 | |
| 972 | Args: |
| 973 | chromeos_root: use 'cros flash' of which chromeos tree |
| 974 | host: DUT address |
| 975 | board: ChromeOS board name |
Kuang-che Wu | 28980b2 | 2019-07-31 19:51:45 +0800 | [diff] [blame] | 976 | image_path: chromeos image xbuddy path or file path. For relative |
| 977 | path, it should be relative to chromeos_root. |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 978 | version: ChromeOS version in short or full format |
| 979 | clobber_stateful: Clobber stateful partition when performing update |
| 980 | disable_rootfs_verification: Disable rootfs verification after update |
| 981 | is completed |
Kuang-che Wu | 2ac9a92 | 2020-09-03 16:50:12 +0800 | [diff] [blame] | 982 | force_reboot_callback: powerful reboot hook (via servo) |
Kuang-che Wu | 414d67f | 2019-05-28 11:28:57 +0800 | [diff] [blame] | 983 | |
| 984 | Raises: |
| 985 | errors.ExternalError: cros flash failed |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 986 | """ |
| 987 | logger.info('cros_flash %s %s %s %s', host, board, version, image_path) |
| 988 | |
| 989 | # Reboot is necessary because sometimes previous 'cros flash' failed and |
| 990 | # entered a bad state. |
Kuang-che Wu | 2ac9a92 | 2020-09-03 16:50:12 +0800 | [diff] [blame] | 991 | reboot(host, force_reboot_callback=force_reboot_callback) |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 992 | |
Kuang-che Wu | 020a118 | 2020-09-08 17:17:22 +0800 | [diff] [blame] | 993 | # Stop service ap-update-manager to prevent rebooting during auto update. |
Kuang-che Wu | 2e0680b | 2020-08-19 22:41:28 +0800 | [diff] [blame] | 994 | # The service is used in jetstream boards, but not other CrOS devices. |
| 995 | if query_dut_os_release(host).get('GOOGLE_CRASH_ID') == 'Jetstream': |
| 996 | try: |
Kuang-che Wu | 29e2eaf | 2020-08-21 16:51:26 +0800 | [diff] [blame] | 997 | # Sleep to wait ap-update-manager start, which may take up to 27 seconds. |
| 998 | # For simplicity, we wait 60 seconds here, which is the timeout value of |
| 999 | # jetstream_host. |
| 1000 | # https://chromium.googlesource.com/chromiumos/third_party/autotest |
| 1001 | # /+/master/server/hosts/jetstream_host.py#27 |
| 1002 | time.sleep(60) |
Kuang-che Wu | 2e0680b | 2020-08-19 22:41:28 +0800 | [diff] [blame] | 1003 | util.ssh_cmd(host, 'stop', 'ap-update-manager') |
| 1004 | except subprocess.CalledProcessError: |
| 1005 | pass # not started; do nothing |
| 1006 | |
Kuang-che Wu | 28980b2 | 2019-07-31 19:51:45 +0800 | [diff] [blame] | 1007 | # Handle relative path. |
| 1008 | if '://' not in image_path and not os.path.isabs(image_path): |
| 1009 | assert os.path.exists(os.path.join(chromeos_root, image_path)) |
| 1010 | image_path = os.path.join(chromeos_root_inside_chroot, image_path) |
| 1011 | |
Kuang-che Wu | f3d03ca | 2019-03-11 17:31:40 +0800 | [diff] [blame] | 1012 | args = [ |
| 1013 | '--debug', '--no-ping', '--send-payload-in-parallel', host, image_path |
| 1014 | ] |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 1015 | if clobber_stateful: |
| 1016 | args.append('--clobber-stateful') |
| 1017 | if disable_rootfs_verification: |
| 1018 | args.append('--disable-rootfs-verification') |
| 1019 | |
Kuang-che Wu | 414d67f | 2019-05-28 11:28:57 +0800 | [diff] [blame] | 1020 | try: |
| 1021 | cros_sdk(chromeos_root, 'cros', 'flash', *args) |
| 1022 | except subprocess.CalledProcessError: |
| 1023 | raise errors.ExternalError('cros flash failed') |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 1024 | |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1025 | if version: |
| 1026 | # In the past, cros flash may fail with returncode=0 |
| 1027 | # So let's have an extra check. |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 1028 | if is_cros_snapshot_version(version): |
| 1029 | builder_path = query_dut_lsb_release(host).get( |
| 1030 | 'CHROMEOS_RELEASE_BUILDER_PATH', '') |
Zheng-Jie Chang | 5402083 | 2020-04-21 15:52:48 +0800 | [diff] [blame] | 1031 | expect_prefix = '%s-snapshot/%s-' % (board, version) |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 1032 | if not builder_path.startswith(expect_prefix): |
| 1033 | raise errors.ExternalError( |
| 1034 | 'although cros flash succeeded, the OS builder path is ' |
| 1035 | 'unexpected: actual=%s expect=%s' % (builder_path, expect_prefix)) |
| 1036 | else: |
| 1037 | expect_version = version_to_short(version) |
| 1038 | dut_version = query_dut_short_version(host) |
| 1039 | if dut_version != expect_version: |
| 1040 | raise errors.ExternalError( |
| 1041 | 'although cros flash succeeded, the OS version is unexpected: ' |
| 1042 | 'actual=%s expect=%s' % (dut_version, expect_version)) |
Kuang-che Wu | 414d67f | 2019-05-28 11:28:57 +0800 | [diff] [blame] | 1043 | |
Kuang-che Wu | 4a81ea7 | 2019-10-05 15:35:17 +0800 | [diff] [blame] | 1044 | # "cros flash" may terminate successfully but the DUT starts self-repairing |
Kuang-che Wu | 414d67f | 2019-05-28 11:28:57 +0800 | [diff] [blame] | 1045 | # (b/130786578), so it's necessary to do sanity check. |
| 1046 | if not is_good_dut(host): |
| 1047 | raise errors.ExternalError( |
| 1048 | 'although cros flash succeeded, the DUT is in bad state') |
| 1049 | |
| 1050 | |
| 1051 | def cros_flash_with_retry(chromeos_root, |
| 1052 | host, |
| 1053 | board, |
| 1054 | image_path, |
| 1055 | version=None, |
| 1056 | clobber_stateful=False, |
| 1057 | disable_rootfs_verification=True, |
Kuang-che Wu | 2ac9a92 | 2020-09-03 16:50:12 +0800 | [diff] [blame] | 1058 | repair_callback=None, |
| 1059 | force_reboot_callback=None): |
Kuang-che Wu | 414d67f | 2019-05-28 11:28:57 +0800 | [diff] [blame] | 1060 | # 'cros flash' is not 100% reliable, retry if necessary. |
| 1061 | for attempt in range(2): |
| 1062 | if attempt > 0: |
| 1063 | logger.info('will retry 60 seconds later') |
| 1064 | time.sleep(60) |
| 1065 | |
| 1066 | try: |
| 1067 | cros_flash( |
| 1068 | chromeos_root, |
| 1069 | host, |
| 1070 | board, |
| 1071 | image_path, |
| 1072 | version=version, |
| 1073 | clobber_stateful=clobber_stateful, |
Kuang-che Wu | 2ac9a92 | 2020-09-03 16:50:12 +0800 | [diff] [blame] | 1074 | disable_rootfs_verification=disable_rootfs_verification, |
| 1075 | force_reboot_callback=force_reboot_callback) |
Kuang-che Wu | 414d67f | 2019-05-28 11:28:57 +0800 | [diff] [blame] | 1076 | return True |
| 1077 | except errors.ExternalError: |
| 1078 | logger.exception('cros flash failed') |
| 1079 | if repair_callback and not repair_callback(host): |
| 1080 | logger.warning('not repaired, assume it is harmless') |
| 1081 | continue |
| 1082 | return False |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 1083 | |
| 1084 | |
| 1085 | def version_info(board, version): |
| 1086 | """Query subcomponents version info of given version of ChromeOS |
| 1087 | |
| 1088 | Args: |
| 1089 | board: ChromeOS board name |
| 1090 | version: ChromeOS version number in short or full format |
| 1091 | |
| 1092 | Returns: |
| 1093 | dict of component and version info, including (if available): |
| 1094 | cros_short_version: ChromeOS version |
| 1095 | cros_full_version: ChromeOS version |
| 1096 | milestone: milestone of ChromeOS |
| 1097 | cr_version: Chrome version |
Kuang-che Wu | 708310b | 2018-03-28 17:24:34 +0800 | [diff] [blame] | 1098 | android_build_id: Android build id |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 1099 | android_branch: Android branch, in format like 'git_nyc-mr1-arc' |
| 1100 | """ |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 1101 | if is_cros_snapshot_version(version): |
Zheng-Jie Chang | 2b6d147 | 2019-11-13 12:40:17 +0800 | [diff] [blame] | 1102 | api = buildbucket_util.BuildbucketApi() |
Zheng-Jie Chang | b869704 | 2019-10-29 16:03:26 +0800 | [diff] [blame] | 1103 | milestone, short_version, _ = snapshot_version_split(version) |
| 1104 | buildbucket_id = query_snapshot_buildbucket_id(board, version) |
Zheng-Jie Chang | 597c38b | 2020-03-12 22:24:10 +0800 | [diff] [blame] | 1105 | data = api.get_build(int(buildbucket_id)).output.properties |
Zheng-Jie Chang | 4fabff6 | 2019-12-08 21:54:35 +0800 | [diff] [blame] | 1106 | target_versions = json_format.MessageToDict(data['target_versions']) |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 1107 | return { |
Zheng-Jie Chang | b869704 | 2019-10-29 16:03:26 +0800 | [diff] [blame] | 1108 | VERSION_KEY_MILESTONE: milestone, |
| 1109 | VERSION_KEY_CROS_FULL_VERSION: version, |
| 1110 | VERSION_KEY_CROS_SHORT_VERSION: short_version, |
Zheng-Jie Chang | 2d1dd9b | 2019-12-07 23:30:20 +0800 | [diff] [blame] | 1111 | VERSION_KEY_CR_VERSION: target_versions.get('chromeVersion'), |
| 1112 | VERSION_KEY_ANDROID_BUILD_ID: target_versions.get('androidVersion'), |
| 1113 | VERSION_KEY_ANDROID_BRANCH: target_versions.get('androidBranchVersion'), |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 1114 | } |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 1115 | info = {} |
| 1116 | full_version = version_to_full(board, version) |
| 1117 | |
| 1118 | # Some boards may have only partial-metadata.json but no metadata.json. |
| 1119 | # e.g. caroline R60-9462.0.0 |
| 1120 | # Let's try both. |
| 1121 | metadata = None |
| 1122 | for metadata_filename in ['metadata.json', 'partial-metadata.json']: |
Kuang-che Wu | 0768b97 | 2019-10-05 15:18:59 +0800 | [diff] [blame] | 1123 | path = gs_archive_path.format( |
| 1124 | board=board) + '/%s/%s' % (full_version, metadata_filename) |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 1125 | metadata = gsutil('cat', path, ignore_errors=True) |
| 1126 | if metadata: |
| 1127 | o = json.loads(metadata) |
| 1128 | v = o['version'] |
| 1129 | board_metadata = o['board-metadata'][board] |
| 1130 | info.update({ |
| 1131 | VERSION_KEY_CROS_SHORT_VERSION: v['platform'], |
| 1132 | VERSION_KEY_CROS_FULL_VERSION: v['full'], |
| 1133 | VERSION_KEY_MILESTONE: v['milestone'], |
| 1134 | VERSION_KEY_CR_VERSION: v['chrome'], |
| 1135 | }) |
| 1136 | |
| 1137 | if 'android' in v: |
Kuang-che Wu | 708310b | 2018-03-28 17:24:34 +0800 | [diff] [blame] | 1138 | info[VERSION_KEY_ANDROID_BUILD_ID] = v['android'] |
Kuang-che Wu | 2ea804f | 2017-11-28 17:11:41 +0800 | [diff] [blame] | 1139 | if 'android-branch' in v: # this appears since R58-9317.0.0 |
| 1140 | info[VERSION_KEY_ANDROID_BRANCH] = v['android-branch'] |
| 1141 | elif 'android-container-branch' in board_metadata: |
| 1142 | info[VERSION_KEY_ANDROID_BRANCH] = v['android-container-branch'] |
| 1143 | break |
| 1144 | else: |
| 1145 | logger.error('Failed to read metadata from gs://chromeos-image-archive') |
| 1146 | logger.error( |
| 1147 | 'Note, so far no quick way to look up version info for too old builds') |
| 1148 | |
| 1149 | return info |
Kuang-che Wu | 848b1af | 2018-02-01 20:59:36 +0800 | [diff] [blame] | 1150 | |
| 1151 | |
| 1152 | def query_chrome_version(board, version): |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1153 | """Queries chrome version of chromeos build. |
Kuang-che Wu | 848b1af | 2018-02-01 20:59:36 +0800 | [diff] [blame] | 1154 | |
| 1155 | Args: |
| 1156 | board: ChromeOS board name |
| 1157 | version: ChromeOS version number in short or full format |
| 1158 | |
| 1159 | Returns: |
| 1160 | Chrome version number |
| 1161 | """ |
| 1162 | info = version_info(board, version) |
| 1163 | return info['cr_version'] |
Kuang-che Wu | 708310b | 2018-03-28 17:24:34 +0800 | [diff] [blame] | 1164 | |
| 1165 | |
| 1166 | def query_android_build_id(board, rev): |
| 1167 | info = version_info(board, rev) |
| 1168 | rev = info['android_build_id'] |
| 1169 | return rev |
| 1170 | |
| 1171 | |
| 1172 | def query_android_branch(board, rev): |
| 1173 | info = version_info(board, rev) |
| 1174 | rev = info['android_branch'] |
| 1175 | return rev |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1176 | |
| 1177 | |
Kuang-che Wu | 3eb6b50 | 2018-06-06 16:15:18 +0800 | [diff] [blame] | 1178 | def guess_chrome_version(board, rev): |
| 1179 | """Guess chrome version number. |
| 1180 | |
| 1181 | Args: |
| 1182 | board: chromeos board name |
| 1183 | rev: chrome or chromeos version |
| 1184 | |
| 1185 | Returns: |
| 1186 | chrome version number |
| 1187 | """ |
| 1188 | if is_cros_version(rev): |
| 1189 | assert board, 'need to specify BOARD for cros version' |
| 1190 | rev = query_chrome_version(board, rev) |
| 1191 | assert cr_util.is_chrome_version(rev) |
| 1192 | |
| 1193 | return rev |
| 1194 | |
| 1195 | |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1196 | def is_inside_chroot(): |
| 1197 | """Returns True if we are inside chroot.""" |
| 1198 | return os.path.exists('/etc/cros_chroot_version') |
| 1199 | |
| 1200 | |
Kuang-che Wu | ad5ac7b | 2020-02-20 19:02:52 +0800 | [diff] [blame] | 1201 | def convert_path_outside_chroot(chromeos_root, path): |
| 1202 | """Converts path in chroot to outside. |
| 1203 | |
| 1204 | Args: |
| 1205 | chromeos_root: chromeos tree root |
| 1206 | path: path inside chroot; support starting with '~/' |
| 1207 | |
| 1208 | Returns: |
| 1209 | The corresponding path outside chroot assuming the chroot is mounted |
| 1210 | """ |
| 1211 | if path.startswith('~/'): |
| 1212 | path = path.replace('~', '/home/' + os.environ['USER']) |
| 1213 | assert '~' not in path, 'tilde (~) character is not fully supported' |
| 1214 | |
| 1215 | assert os.path.isabs(path) |
| 1216 | assert path[0] == os.sep |
| 1217 | return os.path.join(chromeos_root, 'chroot', path[1:]) |
| 1218 | |
| 1219 | |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1220 | def cros_sdk(chromeos_root, *args, **kwargs): |
| 1221 | """Run commands inside chromeos chroot. |
| 1222 | |
| 1223 | Args: |
| 1224 | chromeos_root: chromeos tree root |
| 1225 | *args: command to run |
| 1226 | **kwargs: |
Kuang-che Wu | d4603d7 | 2018-11-29 17:51:21 +0800 | [diff] [blame] | 1227 | chrome_root: pass to cros_sdk; mount this path into the SDK chroot |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1228 | env: (dict) environment variables for the command |
Kuang-che Wu | bcafc55 | 2019-08-15 15:27:02 +0800 | [diff] [blame] | 1229 | log_stdout: Whether write the stdout output of the child process to log. |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1230 | stdin: standard input file handle for the command |
Kuang-che Wu | 9890ce8 | 2018-07-07 15:14:10 +0800 | [diff] [blame] | 1231 | stderr_callback: Callback function for stderr. Called once per line. |
Kuang-che Wu | a9a20bb | 2019-09-05 22:24:04 +0800 | [diff] [blame] | 1232 | goma_dir: Goma installed directory to mount into the chroot |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1233 | """ |
| 1234 | envs = [] |
| 1235 | for k, v in kwargs.get('env', {}).items(): |
| 1236 | assert re.match(r'^[A-Za-z_][A-Za-z0-9_]*$', k) |
| 1237 | envs.append('%s=%s' % (k, v)) |
| 1238 | |
| 1239 | # Use --no-ns-pid to prevent cros_sdk change our pgid, otherwise subsequent |
| 1240 | # commands would be considered as background process. |
Kuang-che Wu | 399d466 | 2019-06-06 15:23:37 +0800 | [diff] [blame] | 1241 | prefix = ['chromite/bin/cros_sdk', '--no-ns-pid'] |
Kuang-che Wu | d4603d7 | 2018-11-29 17:51:21 +0800 | [diff] [blame] | 1242 | |
| 1243 | if kwargs.get('chrome_root'): |
Kuang-che Wu | 399d466 | 2019-06-06 15:23:37 +0800 | [diff] [blame] | 1244 | prefix += ['--chrome_root', kwargs['chrome_root']] |
Kuang-che Wu | a9a20bb | 2019-09-05 22:24:04 +0800 | [diff] [blame] | 1245 | if kwargs.get('goma_dir'): |
| 1246 | prefix += ['--goma_dir', kwargs['goma_dir']] |
Kuang-che Wu | d4603d7 | 2018-11-29 17:51:21 +0800 | [diff] [blame] | 1247 | |
Kuang-che Wu | 399d466 | 2019-06-06 15:23:37 +0800 | [diff] [blame] | 1248 | prefix += envs + ['--'] |
Kuang-che Wu | d4603d7 | 2018-11-29 17:51:21 +0800 | [diff] [blame] | 1249 | |
Kuang-che Wu | 399d466 | 2019-06-06 15:23:37 +0800 | [diff] [blame] | 1250 | # In addition to the output of command we are interested, cros_sdk may |
| 1251 | # generate its own messages. For example, chroot creation messages if we run |
| 1252 | # cros_sdk the first time. |
| 1253 | # This is the hack to run dummy command once, so we can get clean output for |
| 1254 | # the command we are interested. |
| 1255 | cmd = prefix + ['true'] |
Kuang-che Wu | 6267701 | 2020-07-13 14:25:18 +0800 | [diff] [blame] | 1256 | try: |
| 1257 | util.check_call(*cmd, cwd=chromeos_root) |
| 1258 | except subprocess.CalledProcessError: |
| 1259 | logger.exception('cros_sdk init/update failed') |
| 1260 | raise |
Kuang-che Wu | 399d466 | 2019-06-06 15:23:37 +0800 | [diff] [blame] | 1261 | |
| 1262 | cmd = prefix + list(args) |
Kuang-che Wu | 9890ce8 | 2018-07-07 15:14:10 +0800 | [diff] [blame] | 1263 | return util.check_output( |
| 1264 | *cmd, |
| 1265 | cwd=chromeos_root, |
Kuang-che Wu | bcafc55 | 2019-08-15 15:27:02 +0800 | [diff] [blame] | 1266 | log_stdout=kwargs.get('log_stdout', True), |
Kuang-che Wu | 9890ce8 | 2018-07-07 15:14:10 +0800 | [diff] [blame] | 1267 | stdin=kwargs.get('stdin'), |
| 1268 | stderr_callback=kwargs.get('stderr_callback')) |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1269 | |
| 1270 | |
Kuang-che Wu | a9a20bb | 2019-09-05 22:24:04 +0800 | [diff] [blame] | 1271 | def create_chroot(chromeos_root): |
Kuang-che Wu | ad5ac7b | 2020-02-20 19:02:52 +0800 | [diff] [blame] | 1272 | """Creates ChromeOS chroot if necessary. |
Kuang-che Wu | a9a20bb | 2019-09-05 22:24:04 +0800 | [diff] [blame] | 1273 | |
| 1274 | Args: |
| 1275 | chromeos_root: chromeos tree root |
| 1276 | """ |
| 1277 | if os.path.exists(os.path.join(chromeos_root, 'chroot')): |
| 1278 | return |
| 1279 | if os.path.exists(os.path.join(chromeos_root, 'chroot.img')): |
| 1280 | return |
| 1281 | |
| 1282 | util.check_output('chromite/bin/cros_sdk', '--create', cwd=chromeos_root) |
| 1283 | |
| 1284 | |
Kuang-che Wu | ad5ac7b | 2020-02-20 19:02:52 +0800 | [diff] [blame] | 1285 | def mount_chroot(chromeos_root): |
| 1286 | """Creates ChromeOS chroot if necessary. |
| 1287 | |
| 1288 | Args: |
| 1289 | chromeos_root: chromeos tree root |
| 1290 | """ |
| 1291 | # An arbitrary file must exist in chroot. |
| 1292 | path = convert_path_outside_chroot(chromeos_root, '/bin/ls') |
| 1293 | |
| 1294 | # Not created or mounted yet. |
| 1295 | if not os.path.exists(path): |
| 1296 | create_chroot(chromeos_root) |
| 1297 | # After this command, the chroot is mounted. |
| 1298 | cros_sdk(chromeos_root, 'true') |
| 1299 | assert os.path.exists(path) |
| 1300 | |
| 1301 | |
| 1302 | def copy_into_chroot(chromeos_root, src, dst, overwrite=True): |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1303 | """Copies file into chromeos chroot. |
| 1304 | |
Kuang-che Wu | ad5ac7b | 2020-02-20 19:02:52 +0800 | [diff] [blame] | 1305 | The side effect is chroot created and mounted. |
| 1306 | |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1307 | Args: |
| 1308 | chromeos_root: chromeos tree root |
| 1309 | src: path outside chroot |
| 1310 | dst: path inside chroot |
Kuang-che Wu | 020a118 | 2020-09-08 17:17:22 +0800 | [diff] [blame] | 1311 | overwrite: overwrite if dst already exists |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1312 | """ |
Kuang-che Wu | ad5ac7b | 2020-02-20 19:02:52 +0800 | [diff] [blame] | 1313 | mount_chroot(chromeos_root) |
| 1314 | src = os.path.expanduser(src) |
| 1315 | dst_outside = convert_path_outside_chroot(chromeos_root, dst) |
| 1316 | if not overwrite and os.path.exists(dst_outside): |
| 1317 | return |
| 1318 | |
| 1319 | # Haven't support directory or special files yet. |
| 1320 | assert os.path.isfile(src) |
| 1321 | assert os.path.isfile(dst_outside) or not os.path.exists(dst_outside) |
| 1322 | |
| 1323 | dirname = os.path.dirname(dst_outside) |
| 1324 | if not os.path.exists(dirname): |
| 1325 | os.makedirs(dirname) |
| 1326 | shutil.copy(src, dst_outside) |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1327 | |
| 1328 | |
Kuang-che Wu | ad5ac7b | 2020-02-20 19:02:52 +0800 | [diff] [blame] | 1329 | def prepare_chroot(chromeos_root): |
| 1330 | mount_chroot(chromeos_root) |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1331 | |
Kuang-che Wu | ad5ac7b | 2020-02-20 19:02:52 +0800 | [diff] [blame] | 1332 | # Work around b/149077936: |
| 1333 | # The creds file is copied into the chroot since 12866.0.0. |
| 1334 | # But earlier versions need this file as well because of cipd ACL change. |
| 1335 | creds_path = '~/.config/chrome_infra/auth/creds.json' |
| 1336 | if os.path.exists(os.path.expanduser(creds_path)): |
| 1337 | copy_into_chroot(chromeos_root, creds_path, creds_path, overwrite=False) |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1338 | |
| 1339 | |
Kuang-che Wu | 9890ce8 | 2018-07-07 15:14:10 +0800 | [diff] [blame] | 1340 | def check_if_need_recreate_chroot(stdout, stderr): |
| 1341 | """Analyze build log and determine if chroot should be recreated. |
| 1342 | |
| 1343 | Args: |
| 1344 | stdout: stdout output of build |
| 1345 | stderr: stderr output of build |
| 1346 | |
| 1347 | Returns: |
| 1348 | the reason if chroot needs recreated; None otherwise |
| 1349 | """ |
Kuang-che Wu | 74768d3 | 2018-09-07 12:03:24 +0800 | [diff] [blame] | 1350 | if re.search( |
| 1351 | r"The current version of portage supports EAPI '\d+'. " |
Kuang-che Wu | ae6824b | 2019-08-27 22:20:01 +0800 | [diff] [blame] | 1352 | 'You must upgrade', stderr): |
Kuang-che Wu | 9890ce8 | 2018-07-07 15:14:10 +0800 | [diff] [blame] | 1353 | return 'EAPI version mismatch' |
| 1354 | |
Kuang-che Wu | 5ac8132 | 2018-11-26 14:04:06 +0800 | [diff] [blame] | 1355 | if 'Chroot is too new. Consider running:' in stderr: |
| 1356 | return 'chroot version is too new' |
| 1357 | |
| 1358 | # old message before Oct 2018 |
Kuang-che Wu | 9890ce8 | 2018-07-07 15:14:10 +0800 | [diff] [blame] | 1359 | if 'Chroot version is too new. Consider running cros_sdk --replace' in stderr: |
| 1360 | return 'chroot version is too new' |
| 1361 | |
Kuang-che Wu | 6fe987f | 2018-08-28 15:24:20 +0800 | [diff] [blame] | 1362 | # https://groups.google.com/a/chromium.org/forum/#!msg/chromium-os-dev/uzwT5APspB4/NFakFyCIDwAJ |
| 1363 | if "undefined reference to 'std::__1::basic_string" in stdout: |
| 1364 | return 'might be due to compiler change' |
| 1365 | |
Kuang-che Wu | 94e3b45 | 2019-11-21 12:49:18 +0800 | [diff] [blame] | 1366 | # Detect failures due to file collisions. |
| 1367 | # For example, kernel uprev from 3.x to 4.x, they are two separate packages |
| 1368 | # and conflict with each other. Other possible cases are package renaming or |
| 1369 | # refactoring. Let's recreate chroot to work around them. |
| 1370 | if 'Detected file collision' in stdout: |
| 1371 | # Using wildcard between words because the text wraps to the next line |
| 1372 | # depending on length of package name and each line is prefixed with |
| 1373 | # package name. |
| 1374 | # Using ".{,100}" instead of ".*" to prevent regex matching time explodes |
| 1375 | # exponentially. 100 is chosen arbitrarily. It should be longer than any |
| 1376 | # package name (65 now). |
| 1377 | m = re.search( |
| 1378 | r'Package (\S+).{,100}NOT.{,100}merged.{,100}' |
| 1379 | r'due.{,100}to.{,100}file.{,100}collisions', stdout, re.S) |
| 1380 | if m: |
| 1381 | return 'failed to install package due to file collision: ' + m.group(1) |
Kuang-che Wu | 356c352 | 2019-11-19 16:11:05 +0800 | [diff] [blame] | 1382 | |
Kuang-che Wu | 9890ce8 | 2018-07-07 15:14:10 +0800 | [diff] [blame] | 1383 | return None |
| 1384 | |
| 1385 | |
Kuang-che Wu | a9a20bb | 2019-09-05 22:24:04 +0800 | [diff] [blame] | 1386 | def build_packages(chromeos_root, |
| 1387 | board, |
| 1388 | chrome_root=None, |
| 1389 | goma_dir=None, |
| 1390 | afdo_use=False): |
Kuang-che Wu | 28980b2 | 2019-07-31 19:51:45 +0800 | [diff] [blame] | 1391 | """Build ChromeOS packages. |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1392 | |
| 1393 | Args: |
| 1394 | chromeos_root: chromeos tree root |
| 1395 | board: ChromeOS board name |
Kuang-che Wu | a9a20bb | 2019-09-05 22:24:04 +0800 | [diff] [blame] | 1396 | chrome_root: Chrome tree root. If specified, build chrome using the |
| 1397 | provided tree |
| 1398 | goma_dir: Goma installed directory to mount into the chroot. If specified, |
| 1399 | build chrome with goma. |
| 1400 | afdo_use: build chrome with AFDO optimization |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1401 | """ |
Zheng-Jie Chang | ffd4946 | 2019-12-16 12:15:18 +0800 | [diff] [blame] | 1402 | |
| 1403 | def has_build_package_argument(argument): |
| 1404 | stderr_lines = [] |
| 1405 | try: |
| 1406 | util.check_call( |
| 1407 | 'src/scripts/build_packages', |
| 1408 | '--help', |
| 1409 | cwd=chromeos_root, |
| 1410 | stderr_callback=stderr_lines.append) |
| 1411 | except subprocess.CalledProcessError: |
| 1412 | help_output = ''.join(stderr_lines) |
| 1413 | return '--[no]%s' % argument in help_output |
| 1414 | |
Kuang-che Wu | a9a20bb | 2019-09-05 22:24:04 +0800 | [diff] [blame] | 1415 | common_env = { |
| 1416 | 'USE': '-cros-debug chrome_internal', |
| 1417 | 'FEATURES': 'separatedebug', |
| 1418 | } |
Kuang-che Wu | 9890ce8 | 2018-07-07 15:14:10 +0800 | [diff] [blame] | 1419 | stderr_lines = [] |
| 1420 | try: |
Kuang-che Wu | fb55310 | 2018-10-02 18:14:29 +0800 | [diff] [blame] | 1421 | with locking.lock_file(locking.LOCK_FILE_FOR_BUILD): |
Kuang-che Wu | a9a20bb | 2019-09-05 22:24:04 +0800 | [diff] [blame] | 1422 | env = common_env.copy() |
| 1423 | env['FEATURES'] += ' -separatedebug splitdebug' |
Kuang-che Wu | fb55310 | 2018-10-02 18:14:29 +0800 | [diff] [blame] | 1424 | cros_sdk( |
| 1425 | chromeos_root, |
Kuang-che Wu | 28980b2 | 2019-07-31 19:51:45 +0800 | [diff] [blame] | 1426 | './update_chroot', |
| 1427 | '--toolchain_boards', |
Kuang-che Wu | fb55310 | 2018-10-02 18:14:29 +0800 | [diff] [blame] | 1428 | board, |
Kuang-che Wu | a9a20bb | 2019-09-05 22:24:04 +0800 | [diff] [blame] | 1429 | env=env, |
Kuang-che Wu | 28980b2 | 2019-07-31 19:51:45 +0800 | [diff] [blame] | 1430 | stderr_callback=stderr_lines.append) |
Kuang-che Wu | a9a20bb | 2019-09-05 22:24:04 +0800 | [diff] [blame] | 1431 | |
| 1432 | env = common_env.copy() |
| 1433 | cmd = [ |
Kuang-che Wu | 28980b2 | 2019-07-31 19:51:45 +0800 | [diff] [blame] | 1434 | './build_packages', |
| 1435 | '--board', |
| 1436 | board, |
| 1437 | '--withdev', |
| 1438 | '--noworkon', |
| 1439 | '--skip_chroot_upgrade', |
| 1440 | '--accept_licenses=@CHROMEOS', |
Kuang-che Wu | a9a20bb | 2019-09-05 22:24:04 +0800 | [diff] [blame] | 1441 | ] |
Zheng-Jie Chang | ffd4946 | 2019-12-16 12:15:18 +0800 | [diff] [blame] | 1442 | |
| 1443 | # `use_any_chrome` flag is default on and will force to use a chrome |
| 1444 | # prebuilt even if the version doesn't match. |
| 1445 | |
| 1446 | # As this argument is landed in 12681, we should check if the argument |
| 1447 | # exists before adding this. |
| 1448 | if has_build_package_argument('use_any_chrome'): |
| 1449 | cmd.append('--nouse_any_chrome') |
| 1450 | |
Kuang-che Wu | a9a20bb | 2019-09-05 22:24:04 +0800 | [diff] [blame] | 1451 | if goma_dir: |
| 1452 | # Tell build_packages to start and stop goma |
| 1453 | cmd.append('--run_goma') |
| 1454 | env['USE_GOMA'] = 'true' |
| 1455 | if afdo_use: |
| 1456 | env['USE'] += ' afdo_use' |
| 1457 | cros_sdk( |
| 1458 | chromeos_root, |
| 1459 | *cmd, |
| 1460 | env=env, |
| 1461 | chrome_root=chrome_root, |
| 1462 | stderr_callback=stderr_lines.append, |
| 1463 | goma_dir=goma_dir) |
Kuang-che Wu | 9890ce8 | 2018-07-07 15:14:10 +0800 | [diff] [blame] | 1464 | except subprocess.CalledProcessError as e: |
| 1465 | # Detect failures due to incompatibility between chroot and source tree. If |
| 1466 | # so, notify the caller to recreate chroot and retry. |
| 1467 | reason = check_if_need_recreate_chroot(e.output, ''.join(stderr_lines)) |
| 1468 | if reason: |
| 1469 | raise NeedRecreateChrootException(reason) |
| 1470 | |
| 1471 | # For other failures, don't know how to handle. Just bail out. |
| 1472 | raise |
| 1473 | |
Kuang-che Wu | 28980b2 | 2019-07-31 19:51:45 +0800 | [diff] [blame] | 1474 | |
| 1475 | def build_image(chromeos_root, board): |
| 1476 | """Build ChromeOS image. |
| 1477 | |
| 1478 | Args: |
| 1479 | chromeos_root: chromeos tree root |
| 1480 | board: ChromeOS board name |
| 1481 | |
| 1482 | Returns: |
| 1483 | image folder; relative to chromeos_root |
| 1484 | """ |
| 1485 | stderr_lines = [] |
| 1486 | try: |
| 1487 | with locking.lock_file(locking.LOCK_FILE_FOR_BUILD): |
| 1488 | cros_sdk( |
| 1489 | chromeos_root, |
| 1490 | './build_image', |
| 1491 | '--board', |
| 1492 | board, |
| 1493 | '--noenable_rootfs_verification', |
| 1494 | 'test', |
| 1495 | env={ |
| 1496 | 'USE': '-cros-debug chrome_internal', |
| 1497 | 'FEATURES': 'separatedebug', |
| 1498 | }, |
| 1499 | stderr_callback=stderr_lines.append) |
| 1500 | except subprocess.CalledProcessError as e: |
| 1501 | # Detect failures due to incompatibility between chroot and source tree. If |
| 1502 | # so, notify the caller to recreate chroot and retry. |
| 1503 | reason = check_if_need_recreate_chroot(e.output, ''.join(stderr_lines)) |
| 1504 | if reason: |
| 1505 | raise NeedRecreateChrootException(reason) |
| 1506 | |
| 1507 | # For other failures, don't know how to handle. Just bail out. |
| 1508 | raise |
| 1509 | |
| 1510 | image_symlink = os.path.join(chromeos_root, cached_images_dir, board, |
| 1511 | 'latest') |
| 1512 | assert os.path.exists(image_symlink) |
| 1513 | image_name = os.readlink(image_symlink) |
| 1514 | image_folder = os.path.join(cached_images_dir, board, image_name) |
| 1515 | assert os.path.exists( |
| 1516 | os.path.join(chromeos_root, image_folder, test_image_filename)) |
| 1517 | return image_folder |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1518 | |
| 1519 | |
Kuang-che Wu | 23192ad | 2020-03-11 18:12:46 +0800 | [diff] [blame] | 1520 | class AutotestControlInfo: |
Kuang-che Wu | b9705bd | 2018-06-28 17:59:18 +0800 | [diff] [blame] | 1521 | """Parsed content of autotest control file. |
| 1522 | |
| 1523 | Attributes: |
| 1524 | name: test name |
| 1525 | path: control file path |
| 1526 | variables: dict of top-level control variables. Sample keys: NAME, AUTHOR, |
| 1527 | DOC, ATTRIBUTES, DEPENDENCIES, etc. |
| 1528 | """ |
| 1529 | |
| 1530 | def __init__(self, path, variables): |
| 1531 | self.name = variables['NAME'] |
| 1532 | self.path = path |
| 1533 | self.variables = variables |
| 1534 | |
| 1535 | |
| 1536 | def parse_autotest_control_file(path): |
| 1537 | """Parses autotest control file. |
| 1538 | |
| 1539 | This only parses simple top-level string assignments. |
| 1540 | |
| 1541 | Returns: |
| 1542 | AutotestControlInfo object |
| 1543 | """ |
| 1544 | variables = {} |
Kuang-che Wu | a572349 | 2019-11-25 20:59:34 +0800 | [diff] [blame] | 1545 | with open(path) as f: |
| 1546 | code = ast.parse(f.read()) |
Kuang-che Wu | b9705bd | 2018-06-28 17:59:18 +0800 | [diff] [blame] | 1547 | for stmt in code.body: |
| 1548 | # Skip if not simple "NAME = *" assignment. |
| 1549 | if not (isinstance(stmt, ast.Assign) and len(stmt.targets) == 1 and |
| 1550 | isinstance(stmt.targets[0], ast.Name)): |
| 1551 | continue |
| 1552 | |
| 1553 | # Only support string value. |
| 1554 | if isinstance(stmt.value, ast.Str): |
| 1555 | variables[stmt.targets[0].id] = stmt.value.s |
| 1556 | |
| 1557 | return AutotestControlInfo(path, variables) |
| 1558 | |
| 1559 | |
| 1560 | def enumerate_autotest_control_files(autotest_dir): |
| 1561 | """Enumerate autotest control files. |
| 1562 | |
| 1563 | Args: |
| 1564 | autotest_dir: autotest folder |
| 1565 | |
| 1566 | Returns: |
| 1567 | list of paths to control files |
| 1568 | """ |
| 1569 | # Where to find control files. Relative to autotest_dir. |
| 1570 | subpaths = [ |
| 1571 | 'server/site_tests', |
| 1572 | 'client/site_tests', |
| 1573 | 'server/tests', |
| 1574 | 'client/tests', |
| 1575 | ] |
| 1576 | |
| 1577 | blacklist = ['site-packages', 'venv', 'results', 'logs', 'containers'] |
| 1578 | result = [] |
| 1579 | for subpath in subpaths: |
| 1580 | path = os.path.join(autotest_dir, subpath) |
| 1581 | for root, dirs, files in os.walk(path): |
| 1582 | |
| 1583 | for black in blacklist: |
| 1584 | if black in dirs: |
| 1585 | dirs.remove(black) |
| 1586 | |
| 1587 | for filename in files: |
| 1588 | if filename == 'control' or filename.startswith('control.'): |
| 1589 | result.append(os.path.join(root, filename)) |
| 1590 | |
| 1591 | return result |
| 1592 | |
| 1593 | |
| 1594 | def get_autotest_test_info(autotest_dir, test_name): |
| 1595 | """Get metadata of given test. |
| 1596 | |
| 1597 | Args: |
| 1598 | autotest_dir: autotest folder |
| 1599 | test_name: test name |
| 1600 | |
| 1601 | Returns: |
| 1602 | AutotestControlInfo object. None if test not found. |
| 1603 | """ |
| 1604 | for control_file in enumerate_autotest_control_files(autotest_dir): |
Zheng-Jie Chang | 1504a55 | 2020-02-20 16:38:57 +0800 | [diff] [blame] | 1605 | try: |
| 1606 | info = parse_autotest_control_file(control_file) |
| 1607 | except SyntaxError: |
| 1608 | logger.warning('%s is not parsable, ignore', control_file) |
| 1609 | continue |
| 1610 | |
Kuang-che Wu | b9705bd | 2018-06-28 17:59:18 +0800 | [diff] [blame] | 1611 | if info.name == test_name: |
| 1612 | return info |
| 1613 | return None |
| 1614 | |
| 1615 | |
Zheng-Jie Chang | 868c175 | 2020-01-21 14:42:41 +0800 | [diff] [blame] | 1616 | def detect_branch_level(branch): |
| 1617 | """Given a branch name of manifest-internal, detect it's branch level. |
| 1618 | |
| 1619 | level1: if ChromeOS version is x.0.0 |
| 1620 | level2: if ChromeOS version is x.x.0 |
| 1621 | level3: if ChromeOS version is x.x.x |
| 1622 | Where x is an non-zero integer. |
| 1623 | |
| 1624 | Args: |
| 1625 | branch: branch name or ref name in manifest-internal |
| 1626 | |
| 1627 | Returns: |
| 1628 | An integer indicates the branch level, or zero if not detectable. |
| 1629 | """ |
| 1630 | level1 = r'^(refs\/\S+(\/\S+)?/)?master$' |
| 1631 | level2 = r'^\S+-(\d+)(\.0)?\.B$' |
| 1632 | level3 = r'^\S+-(\d+)\.(\d+)(\.0)?\.B$' |
| 1633 | |
| 1634 | if re.match(level1, branch): |
| 1635 | return 1 |
| 1636 | if re.match(level2, branch): |
| 1637 | return 2 |
| 1638 | if re.match(level3, branch): |
| 1639 | return 3 |
| 1640 | return 0 |
| 1641 | |
| 1642 | |
Zheng-Jie Chang | 5f9ae4e | 2020-02-07 14:26:06 +0800 | [diff] [blame] | 1643 | def get_crosland_link(old, new): |
| 1644 | """Generates crosland link between two versions. |
| 1645 | |
| 1646 | Args: |
| 1647 | old: ChromeOS version |
| 1648 | new: ChromeOS version |
| 1649 | |
| 1650 | Returns: |
| 1651 | A crosland url. |
| 1652 | """ |
| 1653 | |
| 1654 | def version_to_url_parameter(ver): |
| 1655 | if is_cros_snapshot_version(ver): |
| 1656 | return snapshot_version_split(ver)[2] |
| 1657 | return version_to_short(ver) |
| 1658 | |
| 1659 | old_parameter = version_to_url_parameter(old) |
| 1660 | new_parameter = version_to_url_parameter(new) |
| 1661 | return CROSLAND_URL_TEMPLATE % (old_parameter, new_parameter) |
| 1662 | |
| 1663 | |
Kuang-che Wu | e4bae0b | 2018-07-19 12:10:14 +0800 | [diff] [blame] | 1664 | class ChromeOSSpecManager(codechange.SpecManager): |
| 1665 | """Repo manifest related operations. |
| 1666 | |
| 1667 | This class enumerates chromeos manifest files, parses them, |
| 1668 | and sync to disk state according to them. |
| 1669 | """ |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1670 | |
| 1671 | def __init__(self, config): |
| 1672 | self.config = config |
Kuang-che Wu | e4bae0b | 2018-07-19 12:10:14 +0800 | [diff] [blame] | 1673 | self.manifest_dir = os.path.join(self.config['chromeos_root'], '.repo', |
| 1674 | 'manifests') |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 1675 | self.manifest_internal_dir = os.path.join(self.config['chromeos_mirror'], |
| 1676 | 'manifest-internal.git') |
Kuang-che Wu | e4bae0b | 2018-07-19 12:10:14 +0800 | [diff] [blame] | 1677 | self.historical_manifest_git_dir = os.path.join( |
Kuang-che Wu | d8fc957 | 2018-10-03 21:00:41 +0800 | [diff] [blame] | 1678 | self.config['chromeos_mirror'], 'chromeos/manifest-versions.git') |
Kuang-che Wu | e4bae0b | 2018-07-19 12:10:14 +0800 | [diff] [blame] | 1679 | if not os.path.exists(self.historical_manifest_git_dir): |
Kuang-che Wu | e121fae | 2018-11-09 16:18:39 +0800 | [diff] [blame] | 1680 | raise errors.InternalError('Manifest snapshots should be cloned into %s' % |
| 1681 | self.historical_manifest_git_dir) |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1682 | |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 1683 | def lookup_snapshot_manifest_revisions(self, old, new): |
| 1684 | """Get manifest commits between snapshot versions. |
Kuang-che Wu | e4bae0b | 2018-07-19 12:10:14 +0800 | [diff] [blame] | 1685 | |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 1686 | Returns: |
| 1687 | list of (timestamp, commit_id, snapshot_id): |
| 1688 | timestamp: integer unix timestamp |
| 1689 | commit_id: a string indicates commit hash |
| 1690 | snapshot_id: a string indicates snapshot id |
| 1691 | """ |
| 1692 | assert is_cros_snapshot_version(old) |
| 1693 | assert is_cros_snapshot_version(new) |
| 1694 | |
| 1695 | gs_path = ( |
Zheng-Jie Chang | 5402083 | 2020-04-21 15:52:48 +0800 | [diff] [blame] | 1696 | 'gs://chromeos-image-archive/{board}-snapshot/{version}-*/image.zip') |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 1697 | # Try to guess the commit time of a snapshot manifest, it is usually a few |
| 1698 | # minutes different between snapshot manifest commit and image.zip |
| 1699 | # generate. |
| 1700 | try: |
| 1701 | old_timestamp = gsutil_stat_update_time( |
| 1702 | gs_path.format(board=self.config['board'], version=old)) - 86400 |
| 1703 | except subprocess.CalledProcessError: |
| 1704 | old_timestamp = None |
| 1705 | try: |
| 1706 | new_timestamp = gsutil_stat_update_time( |
| 1707 | gs_path.format(board=self.config['board'], version=new)) + 86400 |
| 1708 | # 1558657989 is snapshot_id 5982's commit time, this ensures every time |
| 1709 | # we can find snapshot 5982 |
| 1710 | # snapshot_id <= 5982 has different commit message format, so we need |
| 1711 | # to identify its id in different ways, see below comment for more info. |
| 1712 | new_timestamp = max(new_timestamp, 1558657989 + 1) |
| 1713 | except subprocess.CalledProcessError: |
| 1714 | new_timestamp = None |
| 1715 | result = [] |
| 1716 | _, _, old_snapshot_id = snapshot_version_split(old) |
| 1717 | _, _, new_snapshot_id = snapshot_version_split(new) |
| 1718 | repo = self.manifest_internal_dir |
| 1719 | path = 'snapshot.xml' |
| 1720 | branch = 'snapshot' |
| 1721 | commits = git_util.get_history( |
| 1722 | repo, |
| 1723 | path, |
| 1724 | branch, |
| 1725 | after=old_timestamp, |
| 1726 | before=new_timestamp, |
| 1727 | with_subject=True) |
| 1728 | |
| 1729 | # Unfortunately, we can not identify snapshot_id <= 5982 from its commit |
| 1730 | # subject, as their subjects are all `Annealing manifest snapshot.`. |
| 1731 | # So instead we count the snapshot_id manually. |
| 1732 | count = 5982 |
| 1733 | # There are two snapshot_id = 2633 in commit history, ignore the former |
| 1734 | # one. |
| 1735 | ignore_list = ['95c8526a7f0798d02f692010669dcbd5a152439a'] |
| 1736 | # We examine the commits in reverse order as there are some testing |
| 1737 | # commits before snapshot_id=2, this method works fine after |
| 1738 | # snapshot 2, except snapshot 2633 |
| 1739 | for commit in reversed(commits): |
| 1740 | msg = commit[2] |
| 1741 | if commit[1] in ignore_list: |
| 1742 | continue |
| 1743 | |
| 1744 | match = re.match(r'^annealing manifest snapshot (\d+)', msg) |
| 1745 | if match: |
| 1746 | snapshot_id = match.group(1) |
| 1747 | elif 'Annealing manifest snapshot' in msg: |
| 1748 | snapshot_id = str(count) |
| 1749 | count -= 1 |
| 1750 | else: |
| 1751 | continue |
Zheng-Jie Chang | 34595ea | 2020-03-10 13:04:22 +0800 | [diff] [blame] | 1752 | # b/151054108: snapshot version in [29288, 29439] is broken |
| 1753 | if 29288 <= int(snapshot_id) <= 29439: |
| 1754 | continue |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 1755 | if int(old_snapshot_id) <= int(snapshot_id) <= int(new_snapshot_id): |
| 1756 | result.append((commit[0], commit[1], snapshot_id)) |
| 1757 | # We find commits in reversed order, now reverse it again to chronological |
| 1758 | # order. |
| 1759 | return list(reversed(result)) |
| 1760 | |
| 1761 | def lookup_build_timestamp(self, rev): |
| 1762 | assert is_cros_full_version(rev) or is_cros_snapshot_version(rev) |
| 1763 | if is_cros_full_version(rev): |
| 1764 | return self.lookup_release_build_timestamp(rev) |
Kuang-che Wu | a7ddf9b | 2019-11-25 18:59:57 +0800 | [diff] [blame] | 1765 | return self.lookup_snapshot_build_timestamp(rev) |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 1766 | |
| 1767 | def lookup_snapshot_build_timestamp(self, rev): |
| 1768 | assert is_cros_snapshot_version(rev) |
| 1769 | return int(self.lookup_snapshot_manifest_revisions(rev, rev)[0][0]) |
| 1770 | |
| 1771 | def lookup_release_build_timestamp(self, rev): |
| 1772 | assert is_cros_full_version(rev) |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1773 | milestone, short_version = version_split(rev) |
Kuang-che Wu | e4bae0b | 2018-07-19 12:10:14 +0800 | [diff] [blame] | 1774 | path = os.path.join('buildspecs', milestone, short_version + '.xml') |
| 1775 | try: |
| 1776 | timestamp = git_util.get_commit_time(self.historical_manifest_git_dir, |
| 1777 | 'refs/heads/master', path) |
| 1778 | except ValueError: |
Kuang-che Wu | d1b7415 | 2020-05-20 08:46:46 +0800 | [diff] [blame] | 1779 | raise errors.InternalError('%s does not have %s' % |
| 1780 | (self.historical_manifest_git_dir, path)) |
Kuang-che Wu | e4bae0b | 2018-07-19 12:10:14 +0800 | [diff] [blame] | 1781 | return timestamp |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1782 | |
Zheng-Jie Chang | 868c175 | 2020-01-21 14:42:41 +0800 | [diff] [blame] | 1783 | def detect_float_spec_branch_level(self, spec): |
| 1784 | results = [ |
| 1785 | detect_branch_level(branch) for branch in git_util.get_branches( |
| 1786 | self.manifest_dir, commit=spec.name) |
| 1787 | ] |
| 1788 | results = [x for x in results if x > 0] |
| 1789 | return min(results) if results else 0 |
| 1790 | |
| 1791 | def branch_between_float_specs(self, old_spec, new_spec): |
| 1792 | if old_spec.spec_type != codechange.SPEC_FLOAT: |
| 1793 | return False |
| 1794 | if new_spec.spec_type != codechange.SPEC_FLOAT: |
| 1795 | return False |
| 1796 | |
| 1797 | level_old = self.detect_float_spec_branch_level(old_spec) |
| 1798 | level_new = self.detect_float_spec_branch_level(new_spec) |
| 1799 | |
| 1800 | if not level_old or not level_new: |
| 1801 | logger.warning('branch level detect failed, assume master') |
| 1802 | return False |
| 1803 | return level_old != level_new |
| 1804 | |
Kuang-che Wu | d558a04 | 2020-06-06 02:11:00 +0800 | [diff] [blame] | 1805 | def _determine_float_branch(self, old, new, fixed_specs): |
| 1806 | # There is no revision tag in snapshot's xml. We know snapshot |
| 1807 | # builds are on master branch. |
| 1808 | master_refname = 'refs/remotes/origin/master' |
Zheng-Jie Chang | 3e19196 | 2020-02-06 14:25:31 +0800 | [diff] [blame] | 1809 | if fixed_specs[0].revision: |
| 1810 | old_branches = git_util.get_branches( |
Kuang-che Wu | d558a04 | 2020-06-06 02:11:00 +0800 | [diff] [blame] | 1811 | self.manifest_dir, commit=fixed_specs[0].revision, remote=True) |
| 1812 | else: |
| 1813 | old_branches = [master_refname] |
| 1814 | |
Zheng-Jie Chang | 3e19196 | 2020-02-06 14:25:31 +0800 | [diff] [blame] | 1815 | if fixed_specs[-1].revision: |
| 1816 | new_branches = git_util.get_branches( |
Kuang-che Wu | d558a04 | 2020-06-06 02:11:00 +0800 | [diff] [blame] | 1817 | self.manifest_dir, commit=fixed_specs[-1].revision, remote=True) |
Zheng-Jie Chang | 3e19196 | 2020-02-06 14:25:31 +0800 | [diff] [blame] | 1818 | else: |
Kuang-che Wu | d558a04 | 2020-06-06 02:11:00 +0800 | [diff] [blame] | 1819 | new_branches = [master_refname] |
Zheng-Jie Chang | 3e19196 | 2020-02-06 14:25:31 +0800 | [diff] [blame] | 1820 | |
Kuang-che Wu | d558a04 | 2020-06-06 02:11:00 +0800 | [diff] [blame] | 1821 | common_branches = list(set(old_branches) & set(new_branches)) |
| 1822 | assert common_branches, '%s and %s are not on common branches?' % (old, new) |
| 1823 | |
| 1824 | if len(common_branches) == 1: |
| 1825 | return common_branches[0] |
| 1826 | |
| 1827 | # There are more than one common branches, use heuristic to tie breaking. |
| 1828 | # The heuristic is simple: choice the branch with "smallest" number. |
| 1829 | # "Smaller" means the more major branch (not branched) or branched later. |
| 1830 | # |
| 1831 | # Following is the commit graph of manifest-internal repo. It shows many |
| 1832 | # interesting cases. |
| 1833 | # |
| 1834 | # 84/13021.0.0 84/13022.0.0 84/13024.0.0 |
| 1835 | # --A--+---X--------------X------B-------X-----------> master |
| 1836 | # \ |
| 1837 | # \ 83/13020.1.0 83/13020.56.0 83/13020.68.0 |
| 1838 | # C---X----D--+-------X-------+--------X-----> release-R83-13020.B |
| 1839 | # \ \ |
| 1840 | # \ E------------> stabilize-13020.67.B |
| 1841 | # \ 83/13020.55.1 |
| 1842 | # F-----X--------------------> stabilize-13020.55.B |
| 1843 | # |
| 1844 | # How to read this graph: |
| 1845 | # - Time goes from left to right. Branch names are on the right side of |
| 1846 | # arrows. |
| 1847 | # - Letters A-F are manifest commits. |
| 1848 | # - Marker X means release image build at that time, the version numbers |
| 1849 | # are labeled above the X marker. |
| 1850 | # For example, |
| 1851 | # 1) 13021.0.0 release is based on manifest A, which is on all branches |
| 1852 | # shown on the graph. |
| 1853 | # We know 13021.0.0 is on master (and R84 branch later, not shown in |
| 1854 | # this graph), not on 13020* branches. |
| 1855 | # 2) 13020.56.0 release is based on manifest D, which is on 3 branches |
| 1856 | # (R83-13020.B, 13020.67.B, and 13020.55.B). |
| 1857 | # We know 13020.56.0 is on R83-13020.B and 13020.67.B, but not |
| 1858 | # 13020.55.B. |
| 1859 | # |
| 1860 | # There is an important property here. Every time a new branch is created, |
| 1861 | # there will always be a commit (like C, E, and F) to fix "revision" field |
| 1862 | # in the manifest file. In other words, xxxxx.1.0 is impossible based on |
| 1863 | # manifest on master branch. xxxxx.yy.1 is impossible based on manifest on |
| 1864 | # xxxxx.B branch. |
| 1865 | # |
| 1866 | # With such property, among the branches containing the given manifest |
| 1867 | # file, the branch with "smallest" number guarantees where the release is. |
| 1868 | |
| 1869 | def branch_key(s): |
| 1870 | if s == master_refname: |
| 1871 | return 0, 0, 0 |
| 1872 | m = re.search(r'-(\d+)\.B$', s) |
| 1873 | if m: |
| 1874 | return int(m.group(1)), 0, 0 |
| 1875 | m = re.search(r'-(\d+)\.(\d+)\.B$', s) |
| 1876 | if m: |
| 1877 | return int(m.group(1)), int(m.group(2)), 0 |
| 1878 | m = re.search(r'-(\d+)\.(\d+)\.(\d+)\.B$', s) |
| 1879 | if m: |
| 1880 | return int(m.group(1)), int(m.group(2)), int(m.group(3)) |
| 1881 | |
| 1882 | logger.warning('unexpected branch name: %s', s) |
| 1883 | return (sys.maxsize, sys.maxsize, sys.maxsize, s) |
| 1884 | |
| 1885 | common_branches.sort(key=branch_key) |
| 1886 | return common_branches[0] |
| 1887 | |
| 1888 | def collect_float_spec(self, old, new, fixed_specs=None): |
| 1889 | assert fixed_specs |
| 1890 | branch = self._determine_float_branch(old, new, fixed_specs) |
| 1891 | logger.debug('float branch=%s', branch) |
Zheng-Jie Chang | d968f55 | 2020-01-16 13:31:57 +0800 | [diff] [blame] | 1892 | |
Kuang-che Wu | e4bae0b | 2018-07-19 12:10:14 +0800 | [diff] [blame] | 1893 | old_timestamp = self.lookup_build_timestamp(old) |
| 1894 | new_timestamp = self.lookup_build_timestamp(new) |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 1895 | # snapshot time is different from commit time |
| 1896 | # usually it's a few minutes different |
| 1897 | # 30 minutes should be safe in most cases |
| 1898 | if is_cros_snapshot_version(old): |
| 1899 | old_timestamp = old_timestamp - 1800 |
| 1900 | if is_cros_snapshot_version(new): |
| 1901 | new_timestamp = new_timestamp + 1800 |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1902 | |
Zheng-Jie Chang | 1ace301 | 2020-02-15 04:51:05 +0800 | [diff] [blame] | 1903 | # TODO(zjchang): add logic to combine symlink target's (full.xml) history |
Kuang-che Wu | e4bae0b | 2018-07-19 12:10:14 +0800 | [diff] [blame] | 1904 | result = [] |
Zheng-Jie Chang | 1ace301 | 2020-02-15 04:51:05 +0800 | [diff] [blame] | 1905 | path = 'default.xml' |
Kuang-che Wu | e4bae0b | 2018-07-19 12:10:14 +0800 | [diff] [blame] | 1906 | parser = repo_util.ManifestParser(self.manifest_dir) |
| 1907 | for timestamp, git_rev in parser.enumerate_manifest_commits( |
Zheng-Jie Chang | d968f55 | 2020-01-16 13:31:57 +0800 | [diff] [blame] | 1908 | old_timestamp, new_timestamp, path, branch=branch): |
Kuang-che Wu | e4bae0b | 2018-07-19 12:10:14 +0800 | [diff] [blame] | 1909 | result.append( |
| 1910 | codechange.Spec(codechange.SPEC_FLOAT, git_rev, timestamp, path)) |
| 1911 | return result |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1912 | |
Kuang-che Wu | e4bae0b | 2018-07-19 12:10:14 +0800 | [diff] [blame] | 1913 | def collect_fixed_spec(self, old, new): |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 1914 | assert is_cros_full_version(old) or is_cros_snapshot_version(old) |
| 1915 | assert is_cros_full_version(new) or is_cros_snapshot_version(new) |
| 1916 | |
| 1917 | # case 1: if both are snapshot, return a list of snapshot |
| 1918 | if is_cros_snapshot_version(old) and is_cros_snapshot_version(new): |
| 1919 | return self.collect_snapshot_specs(old, new) |
| 1920 | |
| 1921 | # case 2: if both are release version |
| 1922 | # return a list of release version |
| 1923 | if is_cros_full_version(old) and is_cros_full_version(new): |
| 1924 | return self.collect_release_specs(old, new) |
| 1925 | |
| 1926 | # case 3: return a list of release version and append a snapshot |
| 1927 | # before or at the end |
Zheng-Jie Chang | c47af3a | 2019-11-11 17:28:58 +0800 | [diff] [blame] | 1928 | result = self.collect_release_specs( |
| 1929 | version_to_full(self.config['board'], old), |
| 1930 | version_to_full(self.config['board'], new)) |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 1931 | if is_cros_snapshot_version(old): |
Zheng-Jie Chang | c47af3a | 2019-11-11 17:28:58 +0800 | [diff] [blame] | 1932 | result = self.collect_snapshot_specs(old, old) + result[1:] |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 1933 | elif is_cros_snapshot_version(new): |
Zheng-Jie Chang | 5cd62dd | 2019-12-09 13:21:12 +0800 | [diff] [blame] | 1934 | result += self.collect_snapshot_specs(new, new) |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 1935 | return result |
| 1936 | |
| 1937 | def collect_snapshot_specs(self, old, new): |
| 1938 | assert is_cros_snapshot_version(old) |
| 1939 | assert is_cros_snapshot_version(new) |
| 1940 | |
| 1941 | def guess_snapshot_version(board, snapshot_id, old, new): |
| 1942 | if old.endswith('-' + snapshot_id): |
| 1943 | return old |
| 1944 | if new.endswith('-' + snapshot_id): |
| 1945 | return new |
Zheng-Jie Chang | 5402083 | 2020-04-21 15:52:48 +0800 | [diff] [blame] | 1946 | gs_path = ('gs://chromeos-image-archive/{board}-snapshot/' |
Kuang-che Wu | f791afa | 2019-10-28 19:53:26 +0800 | [diff] [blame] | 1947 | 'R*-{snapshot_id}-*'.format( |
| 1948 | board=board, snapshot_id=snapshot_id)) |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 1949 | for line in gsutil_ls(gs_path, ignore_errors=True): |
| 1950 | m = re.match(r'^gs:\S+(R\d+-\d+\.\d+\.\d+-\d+)\S+', line) |
| 1951 | if m: |
| 1952 | return m.group(1) |
Zheng-Jie Chang | 026cd5d | 2019-12-04 12:13:01 +0800 | [diff] [blame] | 1953 | return None |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 1954 | |
| 1955 | result = [] |
| 1956 | path = 'snapshot.xml' |
| 1957 | revisions = self.lookup_snapshot_manifest_revisions(old, new) |
Kuang-che Wu | f791afa | 2019-10-28 19:53:26 +0800 | [diff] [blame] | 1958 | for timestamp, _git_rev, snapshot_id in revisions: |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 1959 | snapshot_version = guess_snapshot_version(self.config['board'], |
| 1960 | snapshot_id, old, new) |
Zheng-Jie Chang | 026cd5d | 2019-12-04 12:13:01 +0800 | [diff] [blame] | 1961 | if snapshot_version: |
| 1962 | result.append( |
| 1963 | codechange.Spec(codechange.SPEC_FIXED, snapshot_version, timestamp, |
| 1964 | path)) |
| 1965 | else: |
| 1966 | logger.warning('snapshot id %s is not found, ignore', snapshot_id) |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 1967 | return result |
| 1968 | |
| 1969 | def collect_release_specs(self, old, new): |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1970 | assert is_cros_full_version(old) |
| 1971 | assert is_cros_full_version(new) |
| 1972 | old_milestone, old_short_version = version_split(old) |
| 1973 | new_milestone, new_short_version = version_split(new) |
| 1974 | |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1975 | result = [] |
Kuang-che Wu | e4bae0b | 2018-07-19 12:10:14 +0800 | [diff] [blame] | 1976 | for milestone in git_util.list_dir_from_revision( |
| 1977 | self.historical_manifest_git_dir, 'refs/heads/master', 'buildspecs'): |
| 1978 | if not milestone.isdigit(): |
| 1979 | continue |
| 1980 | if not int(old_milestone) <= int(milestone) <= int(new_milestone): |
| 1981 | continue |
| 1982 | |
Kuang-che Wu | 74768d3 | 2018-09-07 12:03:24 +0800 | [diff] [blame] | 1983 | files = git_util.list_dir_from_revision( |
| 1984 | self.historical_manifest_git_dir, 'refs/heads/master', |
| 1985 | os.path.join('buildspecs', milestone)) |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1986 | |
| 1987 | for fn in files: |
Kuang-che Wu | e4bae0b | 2018-07-19 12:10:14 +0800 | [diff] [blame] | 1988 | path = os.path.join('buildspecs', milestone, fn) |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1989 | short_version, ext = os.path.splitext(fn) |
| 1990 | if ext != '.xml': |
| 1991 | continue |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 1992 | if (util.is_version_lesseq(old_short_version, short_version) and |
| 1993 | util.is_version_lesseq(short_version, new_short_version) and |
| 1994 | util.is_direct_relative_version(short_version, new_short_version)): |
Kuang-che Wu | e4bae0b | 2018-07-19 12:10:14 +0800 | [diff] [blame] | 1995 | rev = make_cros_full_version(milestone, short_version) |
| 1996 | timestamp = git_util.get_commit_time(self.historical_manifest_git_dir, |
| 1997 | 'refs/heads/master', path) |
| 1998 | result.append( |
| 1999 | codechange.Spec(codechange.SPEC_FIXED, rev, timestamp, path)) |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 2000 | |
Kuang-che Wu | e4bae0b | 2018-07-19 12:10:14 +0800 | [diff] [blame] | 2001 | def version_key_func(spec): |
| 2002 | _milestone, short_version = version_split(spec.name) |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 2003 | return util.version_key_func(short_version) |
| 2004 | |
| 2005 | result.sort(key=version_key_func) |
Kuang-che Wu | e4bae0b | 2018-07-19 12:10:14 +0800 | [diff] [blame] | 2006 | assert result[0].name == old |
| 2007 | assert result[-1].name == new |
Kuang-che Wu | bfc4a64 | 2018-04-19 11:54:08 +0800 | [diff] [blame] | 2008 | return result |
| 2009 | |
Kuang-che Wu | e4bae0b | 2018-07-19 12:10:14 +0800 | [diff] [blame] | 2010 | def get_manifest(self, rev): |
Zheng-Jie Chang | 127c330 | 2019-09-10 17:17:04 +0800 | [diff] [blame] | 2011 | assert is_cros_full_version(rev) or is_cros_snapshot_version(rev) |
| 2012 | if is_cros_full_version(rev): |
| 2013 | milestone, short_version = version_split(rev) |
| 2014 | path = os.path.join('buildspecs', milestone, '%s.xml' % short_version) |
| 2015 | manifest = git_util.get_file_from_revision( |
| 2016 | self.historical_manifest_git_dir, 'refs/heads/master', path) |
| 2017 | else: |
| 2018 | revisions = self.lookup_snapshot_manifest_revisions(rev, rev) |
| 2019 | commit_id = revisions[0][1] |
| 2020 | manifest = git_util.get_file_from_revision(self.manifest_internal_dir, |
| 2021 | commit_id, 'snapshot.xml') |
Zheng-Jie Chang | 0fc704b | 2019-12-09 18:43:38 +0800 | [diff] [blame] | 2022 | return manifest |
| 2023 | |
| 2024 | def get_manifest_file(self, rev): |
| 2025 | assert is_cros_full_version(rev) or is_cros_snapshot_version(rev) |
Kuang-che Wu | e4bae0b | 2018-07-19 12:10:14 +0800 | [diff] [blame] | 2026 | manifest_name = 'manifest_%s.xml' % rev |
| 2027 | manifest_path = os.path.join(self.manifest_dir, manifest_name) |
| 2028 | with open(manifest_path, 'w') as f: |
Zheng-Jie Chang | 0fc704b | 2019-12-09 18:43:38 +0800 | [diff] [blame] | 2029 | f.write(self.get_manifest(rev)) |
Zheng-Jie Chang | ec368b5 | 2020-03-02 16:25:25 +0800 | [diff] [blame] | 2030 | |
| 2031 | # workaround for b/150572399 |
| 2032 | # for chromeOS version < 12931.0.0, manifests are included from incorrect |
| 2033 | # folder .repo instead of.repo/manifests |
| 2034 | if is_cros_version_lesseq(rev, '12931.0.0'): |
| 2035 | repo_path = os.path.join(self.config['chromeos_root'], '.repo') |
| 2036 | manifest_patch_path = os.path.join(repo_path, manifest_name) |
| 2037 | with open(manifest_patch_path, 'w') as f: |
| 2038 | f.write(self.get_manifest(rev)) |
| 2039 | |
Kuang-che Wu | e4bae0b | 2018-07-19 12:10:14 +0800 | [diff] [blame] | 2040 | return manifest_name |
| 2041 | |
| 2042 | def parse_spec(self, spec): |
| 2043 | parser = repo_util.ManifestParser(self.manifest_dir) |
| 2044 | if spec.spec_type == codechange.SPEC_FIXED: |
Zheng-Jie Chang | 0fc704b | 2019-12-09 18:43:38 +0800 | [diff] [blame] | 2045 | manifest_name = self.get_manifest_file(spec.name) |
Kuang-che Wu | e4bae0b | 2018-07-19 12:10:14 +0800 | [diff] [blame] | 2046 | manifest_path = os.path.join(self.manifest_dir, manifest_name) |
Kuang-che Wu | a572349 | 2019-11-25 20:59:34 +0800 | [diff] [blame] | 2047 | with open(manifest_path) as f: |
| 2048 | content = f.read() |
Kuang-che Wu | e4bae0b | 2018-07-19 12:10:14 +0800 | [diff] [blame] | 2049 | root = parser.parse_single_xml(content, allow_include=False) |
| 2050 | else: |
| 2051 | root = parser.parse_xml_recursive(spec.name, spec.path) |
| 2052 | |
| 2053 | spec.entries = parser.process_parsed_result(root) |
| 2054 | if spec.spec_type == codechange.SPEC_FIXED: |
Kuang-che Wu | fe1e88a | 2019-09-10 21:52:25 +0800 | [diff] [blame] | 2055 | if not spec.is_static(): |
Kuang-che Wu | d1b7415 | 2020-05-20 08:46:46 +0800 | [diff] [blame] | 2056 | raise ValueError('fixed spec %r has unexpected floating entries' % |
| 2057 | spec.name) |
Zheng-Jie Chang | d968f55 | 2020-01-16 13:31:57 +0800 | [diff] [blame] | 2058 | spec.revision = root.get('revision') |
Kuang-che Wu | e4bae0b | 2018-07-19 12:10:14 +0800 | [diff] [blame] | 2059 | |
| 2060 | def sync_disk_state(self, rev): |
Zheng-Jie Chang | 0fc704b | 2019-12-09 18:43:38 +0800 | [diff] [blame] | 2061 | manifest_name = self.get_manifest_file(rev) |
Kuang-che Wu | e4bae0b | 2018-07-19 12:10:14 +0800 | [diff] [blame] | 2062 | |
| 2063 | # For ChromeOS, mark_as_stable step requires 'repo init -m', which sticks |
| 2064 | # manifest. 'repo sync -m' is not enough |
| 2065 | repo_util.init( |
| 2066 | self.config['chromeos_root'], |
| 2067 | 'https://chrome-internal.googlesource.com/chromeos/manifest-internal', |
| 2068 | manifest_name=manifest_name, |
| 2069 | repo_url='https://chromium.googlesource.com/external/repo.git', |
Kuang-che Wu | d8fc957 | 2018-10-03 21:00:41 +0800 | [diff] [blame] | 2070 | reference=self.config['chromeos_mirror'], |
Zheng-Jie Chang | 85529ad | 2020-03-06 18:40:18 +0800 | [diff] [blame] | 2071 | # b/150753074: moblab is in non-default group and causes mark_as_stable |
| 2072 | # fail |
Kuang-che Wu | 084eef2 | 2020-03-11 18:29:48 +0800 | [diff] [blame] | 2073 | groups='default,moblab,platform-linux', |
Kuang-che Wu | e4bae0b | 2018-07-19 12:10:14 +0800 | [diff] [blame] | 2074 | ) |
| 2075 | |
| 2076 | # Note, don't sync with current_branch=True for chromeos. One of its |
| 2077 | # build steps (inside mark_as_stable) executes "git describe" which |
| 2078 | # needs git tag information. |
| 2079 | repo_util.sync(self.config['chromeos_root']) |