Luca Farsi | 5717d6f | 2023-12-28 15:09:28 -0800 | [diff] [blame] | 1 | # Copyright 2024, The Android Open Source Project |
| 2 | # |
| 3 | # Licensed under the Apache License, Version 2.0 (the "License"); |
| 4 | # you may not use this file except in compliance with the License. |
| 5 | # You may obtain a copy of the License at |
| 6 | # |
| 7 | # http://www.apache.org/licenses/LICENSE-2.0 |
| 8 | # |
| 9 | # Unless required by applicable law or agreed to in writing, software |
| 10 | # distributed under the License is distributed on an "AS IS" BASIS, |
| 11 | # WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. |
| 12 | # See the License for the specific language governing permissions and |
| 13 | # limitations under the License. |
| 14 | |
Luca Farsi | db13644 | 2024-03-26 10:55:21 -0700 | [diff] [blame] | 15 | """Build script for the CI `test_suites` target.""" |
Luca Farsi | 5717d6f | 2023-12-28 15:09:28 -0800 | [diff] [blame] | 16 | |
| 17 | import argparse |
Luca Farsi | 040fabe | 2024-05-22 17:21:47 -0700 | [diff] [blame] | 18 | from dataclasses import dataclass |
| 19 | import json |
Luca Farsi | db13644 | 2024-03-26 10:55:21 -0700 | [diff] [blame] | 20 | import logging |
Luca Farsi | 5717d6f | 2023-12-28 15:09:28 -0800 | [diff] [blame] | 21 | import os |
| 22 | import pathlib |
Luca Farsi | 5dbad40 | 2024-11-07 12:43:13 -0800 | [diff] [blame] | 23 | import re |
Luca Farsi | 5717d6f | 2023-12-28 15:09:28 -0800 | [diff] [blame] | 24 | import subprocess |
| 25 | import sys |
Luca Farsi | 8ea6742 | 2024-09-17 15:48:11 -0700 | [diff] [blame] | 26 | from typing import Callable |
Luca Farsi | b130e79 | 2024-08-22 12:04:41 -0700 | [diff] [blame] | 27 | from build_context import BuildContext |
Luca Farsi | 040fabe | 2024-05-22 17:21:47 -0700 | [diff] [blame] | 28 | import optimized_targets |
Luca Farsi | 7d85971 | 2024-11-06 16:09:16 -0800 | [diff] [blame] | 29 | import metrics_agent |
Luca Farsi | 5dbad40 | 2024-11-07 12:43:13 -0800 | [diff] [blame] | 30 | import test_discovery_agent |
Luca Farsi | 040fabe | 2024-05-22 17:21:47 -0700 | [diff] [blame] | 31 | |
| 32 | |
Luca Farsi | 62035d9 | 2024-11-25 18:21:45 +0000 | [diff] [blame] | 33 | REQUIRED_ENV_VARS = frozenset(['TARGET_PRODUCT', 'TARGET_RELEASE', 'TOP', 'DIST_DIR']) |
Luca Farsi | 040fabe | 2024-05-22 17:21:47 -0700 | [diff] [blame] | 34 | SOONG_UI_EXE_REL_PATH = 'build/soong/soong_ui.bash' |
Luca Farsi | 2eaa5d0 | 2024-07-23 16:34:27 -0700 | [diff] [blame] | 35 | LOG_PATH = 'logs/build_test_suites.log' |
Julien Desprez | cd6d27c | 2024-12-10 12:21:30 -0800 | [diff] [blame^] | 36 | # Currently, this prevents the removal of those tags when they exist. In the future we likely |
| 37 | # want the script to supply 'dist directly |
| 38 | REQUIRED_BUILD_TARGETS = frozenset(['dist', 'droid']) |
Luca Farsi | 5717d6f | 2023-12-28 15:09:28 -0800 | [diff] [blame] | 39 | |
| 40 | |
Luca Farsi | db13644 | 2024-03-26 10:55:21 -0700 | [diff] [blame] | 41 | class Error(Exception): |
| 42 | |
| 43 | def __init__(self, message): |
| 44 | super().__init__(message) |
Luca Farsi | 5717d6f | 2023-12-28 15:09:28 -0800 | [diff] [blame] | 45 | |
| 46 | |
Luca Farsi | db13644 | 2024-03-26 10:55:21 -0700 | [diff] [blame] | 47 | class BuildFailureError(Error): |
| 48 | |
| 49 | def __init__(self, return_code): |
| 50 | super().__init__(f'Build command failed with return code: f{return_code}') |
| 51 | self.return_code = return_code |
| 52 | |
| 53 | |
Luca Farsi | 040fabe | 2024-05-22 17:21:47 -0700 | [diff] [blame] | 54 | class BuildPlanner: |
| 55 | """Class in charge of determining how to optimize build targets. |
| 56 | |
| 57 | Given the build context and targets to build it will determine a final list of |
| 58 | targets to build along with getting a set of packaging functions to package up |
| 59 | any output zip files needed by the build. |
| 60 | """ |
| 61 | |
| 62 | def __init__( |
| 63 | self, |
Luca Farsi | b130e79 | 2024-08-22 12:04:41 -0700 | [diff] [blame] | 64 | build_context: BuildContext, |
Luca Farsi | 040fabe | 2024-05-22 17:21:47 -0700 | [diff] [blame] | 65 | args: argparse.Namespace, |
| 66 | target_optimizations: dict[str, optimized_targets.OptimizedBuildTarget], |
| 67 | ): |
| 68 | self.build_context = build_context |
| 69 | self.args = args |
| 70 | self.target_optimizations = target_optimizations |
| 71 | |
| 72 | def create_build_plan(self): |
| 73 | |
Luca Farsi | b130e79 | 2024-08-22 12:04:41 -0700 | [diff] [blame] | 74 | if 'optimized_build' not in self.build_context.enabled_build_features: |
Luca Farsi | 040fabe | 2024-05-22 17:21:47 -0700 | [diff] [blame] | 75 | return BuildPlan(set(self.args.extra_targets), set()) |
| 76 | |
| 77 | build_targets = set() |
Luca Farsi | 8ea6742 | 2024-09-17 15:48:11 -0700 | [diff] [blame] | 78 | packaging_commands_getters = [] |
Luca Farsi | 26c0d8a | 2024-11-22 00:09:47 +0000 | [diff] [blame] | 79 | # In order to roll optimizations out differently between test suites and |
| 80 | # device builds, we have separate flags. |
| 81 | if ( |
| 82 | 'test_suites_zip_test_discovery' |
| 83 | in self.build_context.enabled_build_features |
| 84 | and not self.args.device_build |
| 85 | ) or ( |
| 86 | 'device_zip_test_discovery' |
| 87 | in self.build_context.enabled_build_features |
| 88 | and self.args.device_build |
| 89 | ): |
| 90 | preliminary_build_targets = self._collect_preliminary_build_targets() |
| 91 | else: |
| 92 | preliminary_build_targets = self._legacy_collect_preliminary_build_targets() |
| 93 | |
| 94 | # Keep reporting metrics when test discovery is disabled. |
| 95 | # To be removed once test discovery is fully rolled out. |
| 96 | optimization_rationale = '' |
| 97 | test_discovery_zip_regexes = set() |
| 98 | try: |
| 99 | test_discovery_zip_regexes = self._get_test_discovery_zip_regexes() |
| 100 | logging.info(f'Discovered test discovery regexes: {test_discovery_zip_regexes}') |
| 101 | except test_discovery_agent.TestDiscoveryError as e: |
| 102 | optimization_rationale = e.message |
| 103 | logging.warning(f'Unable to perform test discovery: {optimization_rationale}') |
| 104 | |
| 105 | for target in self.args.extra_targets: |
| 106 | if optimization_rationale: |
| 107 | get_metrics_agent().report_unoptimized_target(target, optimization_rationale) |
| 108 | continue |
Luca Farsi | 1b9b6ed | 2024-12-04 03:47:03 -0800 | [diff] [blame] | 109 | try: |
Luca Farsi | 26c0d8a | 2024-11-22 00:09:47 +0000 | [diff] [blame] | 110 | regex = r'\b(%s.*)\b' % re.escape(target) |
Luca Farsi | 1b9b6ed | 2024-12-04 03:47:03 -0800 | [diff] [blame] | 111 | if any(re.search(regex, opt) for opt in test_discovery_zip_regexes): |
| 112 | get_metrics_agent().report_unoptimized_target(target, 'Test artifact used.') |
Luca Farsi | 26c0d8a | 2024-11-22 00:09:47 +0000 | [diff] [blame] | 113 | continue |
| 114 | get_metrics_agent().report_optimized_target(target) |
Luca Farsi | 1b9b6ed | 2024-12-04 03:47:03 -0800 | [diff] [blame] | 115 | except Exception as e: |
| 116 | logging.error(f'unable to parse test discovery output: {repr(e)}') |
Luca Farsi | 5dbad40 | 2024-11-07 12:43:13 -0800 | [diff] [blame] | 117 | |
Luca Farsi | 26c0d8a | 2024-11-22 00:09:47 +0000 | [diff] [blame] | 118 | for target in preliminary_build_targets: |
Luca Farsi | 040fabe | 2024-05-22 17:21:47 -0700 | [diff] [blame] | 119 | target_optimizer_getter = self.target_optimizations.get(target, None) |
| 120 | if not target_optimizer_getter: |
| 121 | build_targets.add(target) |
| 122 | continue |
| 123 | |
| 124 | target_optimizer = target_optimizer_getter( |
| 125 | target, self.build_context, self.args |
| 126 | ) |
| 127 | build_targets.update(target_optimizer.get_build_targets()) |
Luca Farsi | 8ea6742 | 2024-09-17 15:48:11 -0700 | [diff] [blame] | 128 | packaging_commands_getters.append( |
| 129 | target_optimizer.get_package_outputs_commands |
| 130 | ) |
Luca Farsi | 040fabe | 2024-05-22 17:21:47 -0700 | [diff] [blame] | 131 | |
Luca Farsi | 8ea6742 | 2024-09-17 15:48:11 -0700 | [diff] [blame] | 132 | return BuildPlan(build_targets, packaging_commands_getters) |
Luca Farsi | db13644 | 2024-03-26 10:55:21 -0700 | [diff] [blame] | 133 | |
Luca Farsi | 26c0d8a | 2024-11-22 00:09:47 +0000 | [diff] [blame] | 134 | def _collect_preliminary_build_targets(self): |
| 135 | build_targets = set() |
| 136 | try: |
| 137 | test_discovery_zip_regexes = self._get_test_discovery_zip_regexes() |
| 138 | logging.info(f'Discovered test discovery regexes: {test_discovery_zip_regexes}') |
| 139 | except test_discovery_agent.TestDiscoveryError as e: |
| 140 | optimization_rationale = e.message |
| 141 | logging.warning(f'Unable to perform test discovery: {optimization_rationale}') |
| 142 | |
| 143 | for target in self.args.extra_targets: |
| 144 | get_metrics_agent().report_unoptimized_target(target, optimization_rationale) |
| 145 | return self._legacy_collect_preliminary_build_targets() |
| 146 | |
| 147 | for target in self.args.extra_targets: |
| 148 | if target in REQUIRED_BUILD_TARGETS: |
| 149 | build_targets.add(target) |
| 150 | continue |
| 151 | |
| 152 | regex = r'\b(%s.*)\b' % re.escape(target) |
| 153 | for opt in test_discovery_zip_regexes: |
| 154 | try: |
| 155 | if re.search(regex, opt): |
| 156 | get_metrics_agent().report_unoptimized_target(target, 'Test artifact used.') |
| 157 | build_targets.add(target) |
| 158 | continue |
| 159 | get_metrics_agent().report_optimized_target(target) |
| 160 | except Exception as e: |
| 161 | # In case of exception report as unoptimized |
| 162 | build_targets.add(target) |
| 163 | get_metrics_agent().report_unoptimized_target(target, f'Error in parsing test discovery output for {target}: {repr(e)}') |
| 164 | logging.error(f'unable to parse test discovery output: {repr(e)}') |
| 165 | |
| 166 | return build_targets |
| 167 | |
| 168 | def _legacy_collect_preliminary_build_targets(self): |
| 169 | build_targets = set() |
| 170 | for target in self.args.extra_targets: |
| 171 | if self._unused_target_exclusion_enabled( |
| 172 | target |
| 173 | ) and not self.build_context.build_target_used(target): |
| 174 | continue |
| 175 | |
| 176 | build_targets.add(target) |
| 177 | return build_targets |
| 178 | |
Luca Farsi | b24c1c3 | 2024-08-01 14:47:10 -0700 | [diff] [blame] | 179 | def _unused_target_exclusion_enabled(self, target: str) -> bool: |
Luca Farsi | b130e79 | 2024-08-22 12:04:41 -0700 | [diff] [blame] | 180 | return ( |
| 181 | f'{target}_unused_exclusion' |
| 182 | in self.build_context.enabled_build_features |
Luca Farsi | b24c1c3 | 2024-08-01 14:47:10 -0700 | [diff] [blame] | 183 | ) |
| 184 | |
Luca Farsi | 5dbad40 | 2024-11-07 12:43:13 -0800 | [diff] [blame] | 185 | def _get_test_discovery_zip_regexes(self) -> set[str]: |
| 186 | build_target_regexes = set() |
| 187 | for test_info in self.build_context.test_infos: |
| 188 | tf_command = self._build_tf_command(test_info) |
| 189 | discovery_agent = test_discovery_agent.TestDiscoveryAgent(tradefed_args=tf_command) |
| 190 | for regex in discovery_agent.discover_test_zip_regexes(): |
| 191 | build_target_regexes.add(regex) |
| 192 | return build_target_regexes |
| 193 | |
| 194 | |
| 195 | def _build_tf_command(self, test_info) -> list[str]: |
| 196 | command = [test_info.command] |
| 197 | for extra_option in test_info.extra_options: |
| 198 | if not extra_option.get('key'): |
| 199 | continue |
| 200 | arg_key = '--' + extra_option.get('key') |
| 201 | if arg_key == '--build-id': |
| 202 | command.append(arg_key) |
| 203 | command.append(os.environ.get('BUILD_NUMBER')) |
| 204 | continue |
| 205 | if extra_option.get('values'): |
| 206 | for value in extra_option.get('values'): |
| 207 | command.append(arg_key) |
| 208 | command.append(value) |
| 209 | else: |
| 210 | command.append(arg_key) |
| 211 | |
| 212 | return command |
Luca Farsi | db13644 | 2024-03-26 10:55:21 -0700 | [diff] [blame] | 213 | |
Luca Farsi | 040fabe | 2024-05-22 17:21:47 -0700 | [diff] [blame] | 214 | @dataclass(frozen=True) |
| 215 | class BuildPlan: |
| 216 | build_targets: set[str] |
Luca Farsi | 8ea6742 | 2024-09-17 15:48:11 -0700 | [diff] [blame] | 217 | packaging_commands_getters: list[Callable[[], list[list[str]]]] |
Luca Farsi | db13644 | 2024-03-26 10:55:21 -0700 | [diff] [blame] | 218 | |
| 219 | |
| 220 | def build_test_suites(argv: list[str]) -> int: |
Luca Farsi | 040fabe | 2024-05-22 17:21:47 -0700 | [diff] [blame] | 221 | """Builds all test suites passed in, optimizing based on the build_context content. |
Luca Farsi | db13644 | 2024-03-26 10:55:21 -0700 | [diff] [blame] | 222 | |
| 223 | Args: |
| 224 | argv: The command line arguments passed in. |
| 225 | |
| 226 | Returns: |
| 227 | The exit code of the build. |
| 228 | """ |
Luca Farsi | 7d85971 | 2024-11-06 16:09:16 -0800 | [diff] [blame] | 229 | get_metrics_agent().analysis_start() |
| 230 | try: |
| 231 | args = parse_args(argv) |
| 232 | check_required_env() |
| 233 | build_context = BuildContext(load_build_context()) |
| 234 | build_planner = BuildPlanner( |
| 235 | build_context, args, optimized_targets.OPTIMIZED_BUILD_TARGETS |
| 236 | ) |
| 237 | build_plan = build_planner.create_build_plan() |
| 238 | except: |
| 239 | raise |
| 240 | finally: |
| 241 | get_metrics_agent().analysis_end() |
Luca Farsi | 5717d6f | 2023-12-28 15:09:28 -0800 | [diff] [blame] | 242 | |
Luca Farsi | db13644 | 2024-03-26 10:55:21 -0700 | [diff] [blame] | 243 | try: |
Luca Farsi | 040fabe | 2024-05-22 17:21:47 -0700 | [diff] [blame] | 244 | execute_build_plan(build_plan) |
Luca Farsi | db13644 | 2024-03-26 10:55:21 -0700 | [diff] [blame] | 245 | except BuildFailureError as e: |
| 246 | logging.error('Build command failed! Check build_log for details.') |
| 247 | return e.return_code |
Luca Farsi | 7d85971 | 2024-11-06 16:09:16 -0800 | [diff] [blame] | 248 | finally: |
| 249 | get_metrics_agent().end_reporting() |
Luca Farsi | db13644 | 2024-03-26 10:55:21 -0700 | [diff] [blame] | 250 | |
| 251 | return 0 |
| 252 | |
| 253 | |
Luca Farsi | 040fabe | 2024-05-22 17:21:47 -0700 | [diff] [blame] | 254 | def parse_args(argv: list[str]) -> argparse.Namespace: |
| 255 | argparser = argparse.ArgumentParser() |
| 256 | |
| 257 | argparser.add_argument( |
| 258 | 'extra_targets', nargs='*', help='Extra test suites to build.' |
| 259 | ) |
Luca Farsi | 62035d9 | 2024-11-25 18:21:45 +0000 | [diff] [blame] | 260 | argparser.add_argument( |
| 261 | '--device-build', |
| 262 | action='store_true', |
| 263 | help='Flag to indicate running a device build.', |
| 264 | ) |
Luca Farsi | 040fabe | 2024-05-22 17:21:47 -0700 | [diff] [blame] | 265 | |
| 266 | return argparser.parse_args(argv) |
| 267 | |
| 268 | |
Luca Farsi | db13644 | 2024-03-26 10:55:21 -0700 | [diff] [blame] | 269 | def check_required_env(): |
| 270 | """Check for required env vars. |
| 271 | |
| 272 | Raises: |
| 273 | RuntimeError: If any required env vars are not found. |
| 274 | """ |
| 275 | missing_env_vars = sorted(v for v in REQUIRED_ENV_VARS if v not in os.environ) |
| 276 | |
| 277 | if not missing_env_vars: |
| 278 | return |
| 279 | |
| 280 | t = ','.join(missing_env_vars) |
| 281 | raise Error(f'Missing required environment variables: {t}') |
Luca Farsi | 5717d6f | 2023-12-28 15:09:28 -0800 | [diff] [blame] | 282 | |
| 283 | |
Luca Farsi | 040fabe | 2024-05-22 17:21:47 -0700 | [diff] [blame] | 284 | def load_build_context(): |
| 285 | build_context_path = pathlib.Path(os.environ.get('BUILD_CONTEXT', '')) |
| 286 | if build_context_path.is_file(): |
| 287 | try: |
| 288 | with open(build_context_path, 'r') as f: |
| 289 | return json.load(f) |
| 290 | except json.decoder.JSONDecodeError as e: |
| 291 | raise Error(f'Failed to load JSON file: {build_context_path}') |
Luca Farsi | db13644 | 2024-03-26 10:55:21 -0700 | [diff] [blame] | 292 | |
Luca Farsi | 040fabe | 2024-05-22 17:21:47 -0700 | [diff] [blame] | 293 | logging.info('No BUILD_CONTEXT found, skipping optimizations.') |
| 294 | return empty_build_context() |
Luca Farsi | 11767d5 | 2024-03-07 13:33:57 -0800 | [diff] [blame] | 295 | |
| 296 | |
Luca Farsi | 040fabe | 2024-05-22 17:21:47 -0700 | [diff] [blame] | 297 | def empty_build_context(): |
Luca Farsi | b24c1c3 | 2024-08-01 14:47:10 -0700 | [diff] [blame] | 298 | return {'enabledBuildFeatures': []} |
Luca Farsi | db13644 | 2024-03-26 10:55:21 -0700 | [diff] [blame] | 299 | |
Luca Farsi | db13644 | 2024-03-26 10:55:21 -0700 | [diff] [blame] | 300 | |
Luca Farsi | 040fabe | 2024-05-22 17:21:47 -0700 | [diff] [blame] | 301 | def execute_build_plan(build_plan: BuildPlan): |
| 302 | build_command = [] |
| 303 | build_command.append(get_top().joinpath(SOONG_UI_EXE_REL_PATH)) |
| 304 | build_command.append('--make-mode') |
| 305 | build_command.extend(build_plan.build_targets) |
Luca Farsi | 5717d6f | 2023-12-28 15:09:28 -0800 | [diff] [blame] | 306 | |
Luca Farsi | db13644 | 2024-03-26 10:55:21 -0700 | [diff] [blame] | 307 | try: |
| 308 | run_command(build_command) |
| 309 | except subprocess.CalledProcessError as e: |
| 310 | raise BuildFailureError(e.returncode) from e |
Luca Farsi | 5717d6f | 2023-12-28 15:09:28 -0800 | [diff] [blame] | 311 | |
Luca Farsi | 7d85971 | 2024-11-06 16:09:16 -0800 | [diff] [blame] | 312 | get_metrics_agent().packaging_start() |
| 313 | try: |
| 314 | for packaging_commands_getter in build_plan.packaging_commands_getters: |
Luca Farsi | 8ea6742 | 2024-09-17 15:48:11 -0700 | [diff] [blame] | 315 | for packaging_command in packaging_commands_getter(): |
| 316 | run_command(packaging_command) |
Luca Farsi | 7d85971 | 2024-11-06 16:09:16 -0800 | [diff] [blame] | 317 | except subprocess.CalledProcessError as e: |
| 318 | raise BuildFailureError(e.returncode) from e |
| 319 | finally: |
| 320 | get_metrics_agent().packaging_end() |
Luca Farsi | 5717d6f | 2023-12-28 15:09:28 -0800 | [diff] [blame] | 321 | |
Luca Farsi | db13644 | 2024-03-26 10:55:21 -0700 | [diff] [blame] | 322 | |
Luca Farsi | 040fabe | 2024-05-22 17:21:47 -0700 | [diff] [blame] | 323 | def get_top() -> pathlib.Path: |
| 324 | return pathlib.Path(os.environ['TOP']) |
Luca Farsi | 5717d6f | 2023-12-28 15:09:28 -0800 | [diff] [blame] | 325 | |
| 326 | |
Luca Farsi | db13644 | 2024-03-26 10:55:21 -0700 | [diff] [blame] | 327 | def run_command(args: list[str], stdout=None): |
| 328 | subprocess.run(args=args, check=True, stdout=stdout) |
Luca Farsi | 5717d6f | 2023-12-28 15:09:28 -0800 | [diff] [blame] | 329 | |
| 330 | |
Luca Farsi | 7d85971 | 2024-11-06 16:09:16 -0800 | [diff] [blame] | 331 | def get_metrics_agent(): |
| 332 | return metrics_agent.MetricsAgent.instance() |
| 333 | |
| 334 | |
Luca Farsi | 2dc1701 | 2024-03-19 16:47:54 -0700 | [diff] [blame] | 335 | def main(argv): |
Luca Farsi | 2eaa5d0 | 2024-07-23 16:34:27 -0700 | [diff] [blame] | 336 | dist_dir = os.environ.get('DIST_DIR') |
| 337 | if dist_dir: |
| 338 | log_file = pathlib.Path(dist_dir) / LOG_PATH |
| 339 | logging.basicConfig( |
| 340 | level=logging.DEBUG, |
| 341 | format='%(asctime)s %(levelname)s %(message)s', |
| 342 | filename=log_file, |
| 343 | ) |
Luca Farsi | db13644 | 2024-03-26 10:55:21 -0700 | [diff] [blame] | 344 | sys.exit(build_test_suites(argv)) |
Luca Farsi | 10adf35 | 2024-11-06 14:23:36 -0800 | [diff] [blame] | 345 | |
| 346 | |
| 347 | if __name__ == '__main__': |
| 348 | main(sys.argv[1:]) |