Mike Frysinger | f1ba7ad | 2022-09-12 05:42:57 -0400 | [diff] [blame] | 1 | # Copyright 2019 The ChromiumOS Authors |
Alex Klein | c5403d6 | 2019-04-03 09:34:59 -0600 | [diff] [blame] | 2 | # Use of this source code is governed by a BSD-style license that can be |
| 3 | # found in the LICENSE file. |
| 4 | |
| 5 | """Test controller. |
| 6 | |
| 7 | Handles all testing related functionality, it is not itself a test. |
| 8 | """ |
| 9 | |
Jack Neus | c9707c3 | 2021-07-23 21:48:54 +0000 | [diff] [blame] | 10 | import functools |
Derek Beckett | 963901e | 2022-04-19 11:38:36 -0700 | [diff] [blame] | 11 | import logging |
Alex Klein | a2e42c4 | 2019-04-17 16:13:19 -0600 | [diff] [blame] | 12 | import os |
Sean McAllister | 17eed8d | 2021-09-21 10:41:16 -0600 | [diff] [blame] | 13 | import string |
Sean McAllister | 3834fef | 2021-10-08 15:45:18 -0600 | [diff] [blame] | 14 | import subprocess |
Josiah Hounyo | de113e5 | 2022-11-30 06:30:33 +0000 | [diff] [blame] | 15 | import traceback |
Alex Klein | a2e42c4 | 2019-04-17 16:13:19 -0600 | [diff] [blame] | 16 | |
Mike Frysinger | 1cc8f1f | 2022-04-28 22:40:40 -0400 | [diff] [blame] | 17 | from chromite.third_party.google.protobuf import json_format |
| 18 | |
Alex Klein | 8cb365a | 2019-05-15 16:24:53 -0600 | [diff] [blame] | 19 | from chromite.api import controller |
Alex Klein | 076841b | 2019-08-29 15:19:39 -0600 | [diff] [blame] | 20 | from chromite.api import faux |
Alex Klein | 2b23672 | 2019-06-19 15:44:26 -0600 | [diff] [blame] | 21 | from chromite.api import validate |
Alex Klein | a2e42c4 | 2019-04-17 16:13:19 -0600 | [diff] [blame] | 22 | from chromite.api.controller import controller_util |
Evan Hernandez | 4e388a5 | 2019-05-01 12:16:33 -0600 | [diff] [blame] | 23 | from chromite.api.gen.chromite.api import test_pb2 |
David Welling | c1433c2 | 2021-06-25 16:29:48 +0000 | [diff] [blame] | 24 | from chromite.api.gen.chromiumos import common_pb2 |
Sean McAllister | 3834fef | 2021-10-08 15:45:18 -0600 | [diff] [blame] | 25 | from chromite.api.gen.chromiumos.build.api import container_metadata_pb2 |
Mike Frysinger | 1cc8f1f | 2022-04-28 22:40:40 -0400 | [diff] [blame] | 26 | from chromite.api.metrics import deserialize_metrics_log |
Mike Frysinger | 06a51c8 | 2021-04-06 11:39:17 -0400 | [diff] [blame] | 27 | from chromite.lib import build_target_lib |
George Engelbrecht | 764b1cd | 2021-06-18 17:01:07 -0600 | [diff] [blame] | 28 | from chromite.lib import chroot_lib |
Alex Klein | a2e42c4 | 2019-04-17 16:13:19 -0600 | [diff] [blame] | 29 | from chromite.lib import constants |
Alex Klein | c5403d6 | 2019-04-03 09:34:59 -0600 | [diff] [blame] | 30 | from chromite.lib import cros_build_lib |
Ram Chandrasekar | e08e3ba | 2022-04-04 21:42:27 +0000 | [diff] [blame] | 31 | from chromite.lib import goma_lib |
Alex Klein | aef4194 | 2022-04-19 14:13:17 -0600 | [diff] [blame] | 32 | from chromite.lib import metrics_lib |
Alex Klein | a2e42c4 | 2019-04-17 16:13:19 -0600 | [diff] [blame] | 33 | from chromite.lib import osutils |
Alex Klein | c5403d6 | 2019-04-03 09:34:59 -0600 | [diff] [blame] | 34 | from chromite.lib import sysroot_lib |
Alex Klein | 18a60af | 2020-06-11 12:08:47 -0600 | [diff] [blame] | 35 | from chromite.lib.parser import package_info |
Jack Neus | c9707c3 | 2021-07-23 21:48:54 +0000 | [diff] [blame] | 36 | from chromite.service import packages as packages_service |
Alex Klein | c5403d6 | 2019-04-03 09:34:59 -0600 | [diff] [blame] | 37 | from chromite.service import test |
| 38 | |
| 39 | |
Michael Mortensen | 85d3840 | 2019-12-12 09:50:29 -0700 | [diff] [blame] | 40 | @faux.empty_success |
| 41 | @faux.empty_completed_unsuccessfully_error |
Alex Klein | 231d2da | 2019-07-22 16:44:45 -0600 | [diff] [blame] | 42 | def DebugInfoTest(input_proto, _output_proto, config): |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 43 | """Run the debug info tests.""" |
| 44 | sysroot_path = input_proto.sysroot.path |
| 45 | target_name = input_proto.sysroot.build_target.name |
Alex Klein | c5403d6 | 2019-04-03 09:34:59 -0600 | [diff] [blame] | 46 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 47 | if not sysroot_path: |
| 48 | if target_name: |
| 49 | sysroot_path = build_target_lib.get_default_sysroot_path( |
| 50 | target_name |
| 51 | ) |
| 52 | else: |
| 53 | cros_build_lib.Die( |
| 54 | "The sysroot path or the sysroot's build target name " |
| 55 | "must be provided." |
| 56 | ) |
| 57 | |
| 58 | # We could get away with out this, but it's a cheap check. |
| 59 | sysroot = sysroot_lib.Sysroot(sysroot_path) |
| 60 | if not sysroot.Exists(): |
| 61 | cros_build_lib.Die("The provided sysroot does not exist.") |
| 62 | |
| 63 | if config.validate_only: |
| 64 | return controller.RETURN_CODE_VALID_INPUT |
| 65 | |
| 66 | if test.DebugInfoTest(sysroot_path): |
| 67 | return controller.RETURN_CODE_SUCCESS |
Alex Klein | c5403d6 | 2019-04-03 09:34:59 -0600 | [diff] [blame] | 68 | else: |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 69 | return controller.RETURN_CODE_COMPLETED_UNSUCCESSFULLY |
Alex Klein | a2e42c4 | 2019-04-17 16:13:19 -0600 | [diff] [blame] | 70 | |
| 71 | |
Michael Mortensen | 82cd62d | 2019-12-01 14:58:54 -0700 | [diff] [blame] | 72 | def _BuildTargetUnitTestFailedResponse(_input_proto, output_proto, _config): |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 73 | """Add failed packages to a failed response.""" |
| 74 | packages = ["foo/bar", "cat/pkg"] |
| 75 | for pkg in packages: |
| 76 | pkg_info = package_info.parse(pkg) |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 77 | failed_pkg_data_msg = output_proto.failed_package_data.add() |
| 78 | controller_util.serialize_package_info( |
| 79 | pkg_info, failed_pkg_data_msg.name |
| 80 | ) |
| 81 | failed_pkg_data_msg.log_path.path = "/path/to/%s/log" % pkg |
Michael Mortensen | 82cd62d | 2019-12-01 14:58:54 -0700 | [diff] [blame] | 82 | |
| 83 | |
Alex Klein | 0aecf47 | 2022-05-23 10:48:45 -0600 | [diff] [blame] | 84 | @faux.empty_success |
Michael Mortensen | 82cd62d | 2019-12-01 14:58:54 -0700 | [diff] [blame] | 85 | @faux.error(_BuildTargetUnitTestFailedResponse) |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 86 | @validate.require_each("packages", ["category", "package_name"]) |
Alex Klein | 231d2da | 2019-07-22 16:44:45 -0600 | [diff] [blame] | 87 | @validate.validation_complete |
Alex Klein | aef4194 | 2022-04-19 14:13:17 -0600 | [diff] [blame] | 88 | @metrics_lib.collect_metrics |
Alex Klein | 231d2da | 2019-07-22 16:44:45 -0600 | [diff] [blame] | 89 | def BuildTargetUnitTest(input_proto, output_proto, _config): |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 90 | """Run a build target's ebuild unit tests.""" |
| 91 | # Method flags. |
| 92 | # An empty sysroot means build packages was not run. This is used for |
| 93 | # certain boards that need to use prebuilts (e.g. grunt's unittest-only). |
| 94 | was_built = not input_proto.flags.empty_sysroot |
Alex Klein | fa6ebdc | 2019-05-10 10:57:31 -0600 | [diff] [blame] | 95 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 96 | # Packages to be tested. |
Alex Klein | d3b8404 | 2023-05-19 14:43:59 -0600 | [diff] [blame^] | 97 | packages = [ |
| 98 | controller_util.deserialize_package_info(x).atom |
| 99 | for x in input_proto.packages |
| 100 | ] |
Navil Perez | c0b29a8 | 2020-07-07 14:17:48 +0000 | [diff] [blame] | 101 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 102 | # Skipped tests. |
Alex Klein | d3b8404 | 2023-05-19 14:43:59 -0600 | [diff] [blame^] | 103 | blocklist = [ |
| 104 | controller_util.deserialize_package_info(x).atom |
| 105 | for x in input_proto.package_blocklist |
| 106 | ] |
Alex Klein | f267446 | 2019-05-16 16:47:24 -0600 | [diff] [blame] | 107 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 108 | # Allow call to filter out non-cros_workon packages from the input packages. |
| 109 | filter_only_cros_workon = input_proto.flags.filter_only_cros_workon |
Navil Perez | 43fb45d | 2021-05-14 20:34:24 +0000 | [diff] [blame] | 110 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 111 | # Allow call to succeed if no tests were found. |
| 112 | testable_packages_optional = input_proto.flags.testable_packages_optional |
Navil Perez | 19dc479 | 2020-09-10 19:06:17 +0000 | [diff] [blame] | 113 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 114 | build_target = controller_util.ParseBuildTarget(input_proto.build_target) |
Alex Klein | a2e42c4 | 2019-04-17 16:13:19 -0600 | [diff] [blame] | 115 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 116 | code_coverage = input_proto.flags.code_coverage |
Srinivas Hegde | 0058b8b | 2022-09-12 22:31:10 +0000 | [diff] [blame] | 117 | rust_code_coverage = input_proto.flags.rust_code_coverage |
David Burger | a9c1187 | 2020-07-29 13:32:02 -0600 | [diff] [blame] | 118 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 119 | sysroot = sysroot_lib.Sysroot(build_target.root) |
Lizzy Presland | 4feb237 | 2022-01-20 05:16:30 +0000 | [diff] [blame] | 120 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 121 | result = test.BuildTargetUnitTest( |
| 122 | build_target, |
| 123 | packages=packages, |
| 124 | blocklist=blocklist, |
| 125 | was_built=was_built, |
| 126 | code_coverage=code_coverage, |
Srinivas Hegde | 0058b8b | 2022-09-12 22:31:10 +0000 | [diff] [blame] | 127 | rust_code_coverage=rust_code_coverage, |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 128 | testable_packages_optional=testable_packages_optional, |
| 129 | filter_only_cros_workon=filter_only_cros_workon, |
| 130 | ) |
Alex Klein | a2e42c4 | 2019-04-17 16:13:19 -0600 | [diff] [blame] | 131 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 132 | if not result.success: |
| 133 | # Record all failed packages and retrieve log locations. |
| 134 | controller_util.retrieve_package_log_paths( |
| 135 | result.failed_pkgs, output_proto, sysroot |
| 136 | ) |
| 137 | if result.failed_pkgs: |
| 138 | return controller.RETURN_CODE_UNSUCCESSFUL_RESPONSE_AVAILABLE |
| 139 | else: |
| 140 | return controller.RETURN_CODE_COMPLETED_UNSUCCESSFULLY |
Alex Klein | a2e42c4 | 2019-04-17 16:13:19 -0600 | [diff] [blame] | 141 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 142 | deserialize_metrics_log(output_proto.events, prefix=build_target.name) |
Alex Klein | e3fc3ca | 2019-04-30 16:20:55 -0600 | [diff] [blame] | 143 | |
| 144 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 145 | SRC_DIR = os.path.join(constants.SOURCE_ROOT, "src") |
| 146 | PLATFORM_DEV_DIR = os.path.join(SRC_DIR, "platform/dev") |
| 147 | TEST_SERVICE_DIR = os.path.join(PLATFORM_DEV_DIR, "src/chromiumos/test") |
C Shapiro | 91af1ce | 2021-06-17 12:42:09 -0500 | [diff] [blame] | 148 | |
| 149 | |
C Shapiro | e15660b | 2021-06-18 12:49:37 -0500 | [diff] [blame] | 150 | def _BuildTestServiceContainersResponse(input_proto, output_proto, _config): |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 151 | """Fake success response""" |
| 152 | # pylint: disable=unused-argument |
| 153 | output_proto.results.append( |
| 154 | test_pb2.TestServiceContainerBuildResult( |
| 155 | success=test_pb2.TestServiceContainerBuildResult.Success() |
| 156 | ) |
| 157 | ) |
C Shapiro | e15660b | 2021-06-18 12:49:37 -0500 | [diff] [blame] | 158 | |
| 159 | |
| 160 | def _BuildTestServiceContainersFailedResponse( |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 161 | _input_proto, output_proto, _config |
| 162 | ): |
| 163 | """Fake failure response""" |
C Shapiro | e15660b | 2021-06-18 12:49:37 -0500 | [diff] [blame] | 164 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 165 | # pylint: disable=unused-argument |
| 166 | output_proto.results.append( |
| 167 | test_pb2.TestServiceContainerBuildResult( |
| 168 | failure=test_pb2.TestServiceContainerBuildResult.Failure( |
| 169 | error_message="fake error" |
| 170 | ) |
| 171 | ) |
| 172 | ) |
C Shapiro | e15660b | 2021-06-18 12:49:37 -0500 | [diff] [blame] | 173 | |
| 174 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 175 | @validate.constraint("valid docker tag") |
Sean McAllister | 17eed8d | 2021-09-21 10:41:16 -0600 | [diff] [blame] | 176 | def _ValidDockerTag(tag): |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 177 | """Check that a string meets requirements for Docker tag naming.""" |
| 178 | # Tags can't start with period or dash |
| 179 | if tag[0] in ".-": |
| 180 | return "tag can't begin with '.' or '-'" |
Sean McAllister | 17eed8d | 2021-09-21 10:41:16 -0600 | [diff] [blame] | 181 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 182 | # Tags can only consist of [a-zA-Z0-9-_.] |
| 183 | allowed_chars = set(string.ascii_letters + string.digits + "-_.") |
| 184 | invalid_chars = set(tag) - allowed_chars |
| 185 | if invalid_chars: |
| 186 | return f'saw one or more invalid characters: [{"".join(invalid_chars)}]' |
Sean McAllister | 17eed8d | 2021-09-21 10:41:16 -0600 | [diff] [blame] | 187 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 188 | # Finally, max tag length is 128 characters |
| 189 | if len(tag) > 128: |
| 190 | return "maximum tag length is 128 characters" |
Sean McAllister | 17eed8d | 2021-09-21 10:41:16 -0600 | [diff] [blame] | 191 | |
| 192 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 193 | @validate.constraint("valid docker label key") |
Sean McAllister | 17eed8d | 2021-09-21 10:41:16 -0600 | [diff] [blame] | 194 | def _ValidDockerLabelKey(key): |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 195 | """Check that a string meets requirements for Docker tag naming.""" |
Sean McAllister | 17eed8d | 2021-09-21 10:41:16 -0600 | [diff] [blame] | 196 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 197 | # Label keys should start and end with a lowercase letter |
| 198 | lowercase = set(string.ascii_lowercase) |
| 199 | if not (key[0] in lowercase and key[-1] in lowercase): |
| 200 | return "label key doesn't start and end with lowercase letter" |
Sean McAllister | 17eed8d | 2021-09-21 10:41:16 -0600 | [diff] [blame] | 201 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 202 | # Label keys can have lower-case alphanumeric characters, period and dash |
| 203 | allowed_chars = set(string.ascii_lowercase + string.digits + "-.") |
| 204 | invalid_chars = set(key) - allowed_chars |
| 205 | if invalid_chars: |
| 206 | return f'saw one or more invalid characters: [{"".join(invalid_chars)}]' |
Sean McAllister | 17eed8d | 2021-09-21 10:41:16 -0600 | [diff] [blame] | 207 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 208 | # Repeated . and - aren't allowed |
| 209 | for char in ".-": |
| 210 | if char * 2 in key: |
| 211 | return f"'{char}' can't be repeated in label key" |
Sean McAllister | 17eed8d | 2021-09-21 10:41:16 -0600 | [diff] [blame] | 212 | |
| 213 | |
C Shapiro | e15660b | 2021-06-18 12:49:37 -0500 | [diff] [blame] | 214 | @faux.success(_BuildTestServiceContainersResponse) |
| 215 | @faux.error(_BuildTestServiceContainersFailedResponse) |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 216 | @validate.require("build_target.name") |
| 217 | @validate.require("chroot.path") |
| 218 | @validate.check_constraint("tags", _ValidDockerTag) |
| 219 | @validate.check_constraint("labels", _ValidDockerLabelKey) |
C Shapiro | 91af1ce | 2021-06-17 12:42:09 -0500 | [diff] [blame] | 220 | @validate.validation_complete |
Sean McAllister | 15fa833 | 2021-09-27 12:24:12 -0600 | [diff] [blame] | 221 | def BuildTestServiceContainers( |
| 222 | input_proto: test_pb2.BuildTestServiceContainersRequest, |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 223 | output_proto: test_pb2.BuildTestServiceContainersResponse, |
| 224 | _config, |
| 225 | ): |
Alex Klein | ab87ceb | 2023-01-24 12:00:51 -0700 | [diff] [blame] | 226 | """Build docker containers for all test services and push them to gcr.io.""" |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 227 | build_target = controller_util.ParseBuildTarget(input_proto.build_target) |
| 228 | chroot = controller_util.ParseChroot(input_proto.chroot) |
| 229 | sysroot = sysroot_lib.Sysroot(build_target.root) |
C Shapiro | 91af1ce | 2021-06-17 12:42:09 -0500 | [diff] [blame] | 230 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 231 | tags = ",".join(input_proto.tags) |
| 232 | labels = (f"{key}={value}" for key, value in input_proto.labels.items()) |
Sean McAllister | 15fa833 | 2021-09-27 12:24:12 -0600 | [diff] [blame] | 233 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 234 | build_script = os.path.join( |
| 235 | TEST_SERVICE_DIR, "python/src/docker_libs/cli/build-dockerimages.py" |
| 236 | ) |
| 237 | human_name = "Service Builder" |
Sean McAllister | 15fa833 | 2021-09-27 12:24:12 -0600 | [diff] [blame] | 238 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 239 | with osutils.TempDir(prefix="test_container") as tempdir: |
| 240 | result_file = "metadata.jsonpb" |
| 241 | output_path = os.path.join(tempdir, result_file) |
| 242 | # Note that we use an output file instead of stdout to avoid any issues |
| 243 | # with maintaining stdout hygiene. Stdout and stderr are combined to |
| 244 | # form the error log in response to any errors. |
| 245 | cmd = [build_script, chroot.path, sysroot.path] |
Sean McAllister | e6a4ae2 | 2021-10-21 13:17:08 -0600 | [diff] [blame] | 246 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 247 | if input_proto.HasField("repository"): |
| 248 | cmd += ["--host", input_proto.repository.hostname] |
| 249 | cmd += ["--project", input_proto.repository.project] |
Sean McAllister | e6a4ae2 | 2021-10-21 13:17:08 -0600 | [diff] [blame] | 250 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 251 | cmd += ["--tags", tags] |
| 252 | cmd += ["--output", output_path] |
Derek Beckett | b28b537 | 2022-04-15 13:08:32 -0700 | [diff] [blame] | 253 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 254 | # Translate generator to comma separated string. |
| 255 | ct_labels = ",".join(labels) |
| 256 | cmd += ["--labels", ct_labels] |
| 257 | cmd += ["--build_all"] |
| 258 | cmd += ["--upload"] |
Derek Beckett | cbe64c8 | 2022-05-05 15:00:18 -0700 | [diff] [blame] | 259 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 260 | cmd_result = cros_build_lib.run( |
| 261 | cmd, check=False, stderr=subprocess.STDOUT, stdout=True |
| 262 | ) |
Derek Beckett | a8d9fd2 | 2022-04-27 10:44:52 -0700 | [diff] [blame] | 263 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 264 | if cmd_result.returncode != 0: |
| 265 | # When failing, just record a fail response with the builder name. |
| 266 | logging.debug( |
| 267 | "%s build failed.\nStdout:\n%s\nStderr:\n%s", |
| 268 | human_name, |
| 269 | cmd_result.stdout, |
| 270 | cmd_result.stderr, |
| 271 | ) |
| 272 | result = test_pb2.TestServiceContainerBuildResult() |
| 273 | result.name = human_name |
| 274 | image_info = container_metadata_pb2.ContainerImageInfo() |
| 275 | result.failure.CopyFrom( |
| 276 | test_pb2.TestServiceContainerBuildResult.Failure( |
| 277 | error_message=cmd_result.stdout |
| 278 | ) |
| 279 | ) |
| 280 | output_proto.results.append(result) |
C Shapiro | 91af1ce | 2021-06-17 12:42:09 -0500 | [diff] [blame] | 281 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 282 | else: |
| 283 | logging.debug( |
| 284 | "%s build succeeded.\nStdout:\n%s\nStderr:\n%s", |
| 285 | human_name, |
| 286 | cmd_result.stdout, |
| 287 | cmd_result.stderr, |
| 288 | ) |
| 289 | files = os.listdir(tempdir) |
| 290 | # Iterate through the tempdir to output metadata files. |
| 291 | for file in files: |
| 292 | if result_file in file: |
| 293 | output_path = os.path.join(tempdir, file) |
Derek Beckett | 344b5a8 | 2022-05-09 17:21:45 -0700 | [diff] [blame] | 294 | |
Alex Klein | ab87ceb | 2023-01-24 12:00:51 -0700 | [diff] [blame] | 295 | # build-dockerimages.py will append the service name to |
| 296 | # outputfile with an underscore. |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 297 | human_name = file.split("_")[-1] |
Derek Beckett | 344b5a8 | 2022-05-09 17:21:45 -0700 | [diff] [blame] | 298 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 299 | result = test_pb2.TestServiceContainerBuildResult() |
| 300 | result.name = human_name |
| 301 | image_info = container_metadata_pb2.ContainerImageInfo() |
| 302 | json_format.Parse(osutils.ReadFile(output_path), image_info) |
| 303 | result.success.CopyFrom( |
| 304 | test_pb2.TestServiceContainerBuildResult.Success( |
| 305 | image_info=image_info |
| 306 | ) |
| 307 | ) |
| 308 | output_proto.results.append(result) |
Derek Beckett | 344b5a8 | 2022-05-09 17:21:45 -0700 | [diff] [blame] | 309 | |
C Shapiro | 91af1ce | 2021-06-17 12:42:09 -0500 | [diff] [blame] | 310 | |
Michael Mortensen | 7a860eb | 2019-12-03 20:25:15 -0700 | [diff] [blame] | 311 | @faux.empty_success |
| 312 | @faux.empty_completed_unsuccessfully_error |
Alex Klein | 231d2da | 2019-07-22 16:44:45 -0600 | [diff] [blame] | 313 | @validate.validation_complete |
| 314 | def ChromiteUnitTest(_input_proto, _output_proto, _config): |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 315 | """Run the chromite unit tests.""" |
| 316 | if test.ChromiteUnitTest(): |
| 317 | return controller.RETURN_CODE_SUCCESS |
| 318 | else: |
| 319 | return controller.RETURN_CODE_COMPLETED_UNSUCCESSFULLY |
Evan Hernandez | 4e388a5 | 2019-05-01 12:16:33 -0600 | [diff] [blame] | 320 | |
| 321 | |
Greg Edelston | f3fc8b6 | 2020-03-17 14:20:24 -0600 | [diff] [blame] | 322 | @faux.empty_success |
| 323 | @faux.empty_completed_unsuccessfully_error |
| 324 | @validate.validation_complete |
| 325 | def ChromitePytest(_input_proto, _output_proto, _config): |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 326 | """Run the chromite unit tests.""" |
| 327 | # TODO(vapier): Delete this stub. |
| 328 | return controller.RETURN_CODE_SUCCESS |
Greg Edelston | f3fc8b6 | 2020-03-17 14:20:24 -0600 | [diff] [blame] | 329 | |
| 330 | |
Sloan Johnson | 9fdd531 | 2022-03-02 00:55:26 +0000 | [diff] [blame] | 331 | @faux.empty_success |
| 332 | @faux.empty_completed_unsuccessfully_error |
| 333 | @validate.validation_complete |
| 334 | def RulesCrosUnitTest(_input_proto, _output_proto, _config): |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 335 | """Run the rules_cros unit tests.""" |
| 336 | if test.RulesCrosUnitTest(): |
| 337 | return controller.RETURN_CODE_SUCCESS |
| 338 | else: |
| 339 | return controller.RETURN_CODE_COMPLETED_UNSUCCESSFULLY |
Sloan Johnson | 9fdd531 | 2022-03-02 00:55:26 +0000 | [diff] [blame] | 340 | |
| 341 | |
Michael Mortensen | c28d6f1 | 2019-10-03 13:34:51 -0600 | [diff] [blame] | 342 | @faux.all_empty |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 343 | @validate.require("sysroot.path", "sysroot.build_target.name", "chrome_root") |
Michael Mortensen | c28d6f1 | 2019-10-03 13:34:51 -0600 | [diff] [blame] | 344 | @validate.validation_complete |
| 345 | def SimpleChromeWorkflowTest(input_proto, _output_proto, _config): |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 346 | """Run SimpleChromeWorkflow tests.""" |
| 347 | if input_proto.goma_config.goma_dir: |
| 348 | chromeos_goma_dir = input_proto.goma_config.chromeos_goma_dir or None |
| 349 | goma = goma_lib.Goma( |
| 350 | input_proto.goma_config.goma_dir, |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 351 | stage_name="BuildApiTestSimpleChrome", |
| 352 | chromeos_goma_dir=chromeos_goma_dir, |
| 353 | ) |
| 354 | else: |
| 355 | goma = None |
| 356 | return test.SimpleChromeWorkflowTest( |
| 357 | input_proto.sysroot.path, |
| 358 | input_proto.sysroot.build_target.name, |
| 359 | input_proto.chrome_root, |
| 360 | goma, |
| 361 | ) |
Michael Mortensen | c28d6f1 | 2019-10-03 13:34:51 -0600 | [diff] [blame] | 362 | |
| 363 | |
Alex Klein | 076841b | 2019-08-29 15:19:39 -0600 | [diff] [blame] | 364 | @faux.all_empty |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 365 | @validate.require( |
| 366 | "build_target.name", "vm_path.path", "test_harness", "vm_tests" |
| 367 | ) |
Alex Klein | 231d2da | 2019-07-22 16:44:45 -0600 | [diff] [blame] | 368 | @validate.validation_complete |
| 369 | def VmTest(input_proto, _output_proto, _config): |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 370 | """Run VM tests.""" |
| 371 | build_target_name = input_proto.build_target.name |
| 372 | vm_path = input_proto.vm_path.path |
Alex Klein | c05f3d1 | 2019-05-29 14:16:21 -0600 | [diff] [blame] | 373 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 374 | test_harness = input_proto.test_harness |
Evan Hernandez | 4e388a5 | 2019-05-01 12:16:33 -0600 | [diff] [blame] | 375 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 376 | vm_tests = input_proto.vm_tests |
Evan Hernandez | 4e388a5 | 2019-05-01 12:16:33 -0600 | [diff] [blame] | 377 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 378 | cmd = [ |
| 379 | "cros_run_test", |
| 380 | "--debug", |
| 381 | "--no-display", |
| 382 | "--copy-on-write", |
| 383 | "--board", |
| 384 | build_target_name, |
| 385 | "--image-path", |
| 386 | vm_path, |
| 387 | "--%s" % test_pb2.VmTestRequest.TestHarness.Name(test_harness).lower(), |
| 388 | ] |
| 389 | cmd.extend(vm_test.pattern for vm_test in vm_tests) |
Evan Hernandez | 4e388a5 | 2019-05-01 12:16:33 -0600 | [diff] [blame] | 390 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 391 | if input_proto.ssh_options.port: |
| 392 | cmd.extend(["--ssh-port", str(input_proto.ssh_options.port)]) |
Evan Hernandez | 4e388a5 | 2019-05-01 12:16:33 -0600 | [diff] [blame] | 393 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 394 | if input_proto.ssh_options.private_key_path: |
| 395 | cmd.extend( |
| 396 | ["--private-key", input_proto.ssh_options.private_key_path.path] |
| 397 | ) |
Evan Hernandez | 4e388a5 | 2019-05-01 12:16:33 -0600 | [diff] [blame] | 398 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 399 | # TODO(evanhernandez): Find a nice way to pass test_that-args through |
| 400 | # the build API. Or obviate them. |
| 401 | if test_harness == test_pb2.VmTestRequest.AUTOTEST: |
| 402 | cmd.append("--test_that-args=--allow-chrome-crashes") |
Evan Hernandez | 4e388a5 | 2019-05-01 12:16:33 -0600 | [diff] [blame] | 403 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 404 | with osutils.TempDir(prefix="vm-test-results.") as results_dir: |
| 405 | cmd.extend(["--results-dir", results_dir]) |
| 406 | cros_build_lib.run(cmd, kill_timeout=10 * 60) |
Evan Hernandez | dc3f0bb | 2019-06-06 12:46:52 -0600 | [diff] [blame] | 407 | |
| 408 | |
Alex Klein | 076841b | 2019-08-29 15:19:39 -0600 | [diff] [blame] | 409 | @faux.all_empty |
Alex Klein | 4bc8f4f | 2019-08-16 14:53:30 -0600 | [diff] [blame] | 410 | @validate.validation_complete |
| 411 | def CrosSigningTest(_input_proto, _output_proto, _config): |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 412 | """Run the cros-signing unit tests.""" |
| 413 | test_runner = os.path.join( |
| 414 | constants.SOURCE_ROOT, "cros-signing", "signer", "run_tests.py" |
| 415 | ) |
| 416 | result = cros_build_lib.run([test_runner], check=False) |
Alex Klein | 4bc8f4f | 2019-08-16 14:53:30 -0600 | [diff] [blame] | 417 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 418 | return result.returncode |
David Welling | c1433c2 | 2021-06-25 16:29:48 +0000 | [diff] [blame] | 419 | |
| 420 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 421 | def GetArtifacts( |
| 422 | in_proto: common_pb2.ArtifactsByService.Test, |
| 423 | chroot: chroot_lib.Chroot, |
| 424 | sysroot_class: sysroot_lib.Sysroot, |
| 425 | build_target: build_target_lib.BuildTarget, |
| 426 | output_dir: str, |
| 427 | ) -> list: |
| 428 | """Builds and copies test artifacts to specified output_dir. |
David Welling | c1433c2 | 2021-06-25 16:29:48 +0000 | [diff] [blame] | 429 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 430 | Copies test artifacts to output_dir, returning a list of (output_dir: str) |
| 431 | paths to the desired files. |
David Welling | c1433c2 | 2021-06-25 16:29:48 +0000 | [diff] [blame] | 432 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 433 | Args: |
Alex Klein | 611dddd | 2022-10-11 17:02:01 -0600 | [diff] [blame] | 434 | in_proto: Proto request defining reqs. |
| 435 | chroot: The chroot class used for these artifacts. |
| 436 | sysroot_class: The sysroot class used for these artifacts. |
| 437 | build_target: The build target used for these artifacts. |
| 438 | output_dir: The path to write artifacts to. |
David Welling | c1433c2 | 2021-06-25 16:29:48 +0000 | [diff] [blame] | 439 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 440 | Returns: |
Alex Klein | 611dddd | 2022-10-11 17:02:01 -0600 | [diff] [blame] | 441 | A list of dictionary mappings of ArtifactType to list of paths. |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 442 | """ |
| 443 | generated = [] |
George Engelbrecht | 764b1cd | 2021-06-18 17:01:07 -0600 | [diff] [blame] | 444 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 445 | artifact_types = { |
| 446 | in_proto.ArtifactType.CODE_COVERAGE_LLVM_JSON: functools.partial( |
| 447 | test.BundleCodeCoverageLlvmJson, build_target.name |
| 448 | ), |
Srinivas Hegde | b6fe4ac | 2022-09-22 00:04:40 +0000 | [diff] [blame] | 449 | in_proto.ArtifactType.CODE_COVERAGE_RUST_LLVM_JSON: functools.partial( |
| 450 | test.BundleCodeCoverageRustLlvmJson, build_target.name |
| 451 | ), |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 452 | in_proto.ArtifactType.HWQUAL: functools.partial( |
| 453 | test.BundleHwqualTarball, |
| 454 | build_target.name, |
| 455 | packages_service.determine_full_version(), |
| 456 | ), |
Charles Liu | c71379c | 2022-10-28 04:02:34 +0000 | [diff] [blame] | 457 | in_proto.ArtifactType.CODE_COVERAGE_GOLANG: functools.partial( |
| 458 | test.BundleCodeCoverageGolang |
| 459 | ), |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 460 | } |
George Engelbrecht | 764b1cd | 2021-06-18 17:01:07 -0600 | [diff] [blame] | 461 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 462 | for output_artifact in in_proto.output_artifacts: |
| 463 | for artifact_type, func in artifact_types.items(): |
| 464 | if artifact_type in output_artifact.artifact_types: |
Josiah Hounyo | de113e5 | 2022-11-30 06:30:33 +0000 | [diff] [blame] | 465 | try: |
| 466 | if ( |
| 467 | artifact_type |
| 468 | == in_proto.ArtifactType.CODE_COVERAGE_GOLANG |
| 469 | ): |
| 470 | paths = func(chroot, output_dir) |
| 471 | else: |
| 472 | paths = func(chroot, sysroot_class, output_dir) |
| 473 | except Exception as e: |
| 474 | generated.append( |
| 475 | { |
| 476 | "type": artifact_type, |
| 477 | "failed": True, |
| 478 | "failure_reason": str(e), |
| 479 | } |
| 480 | ) |
| 481 | artifact_name = ( |
| 482 | common_pb2.ArtifactsByService.Test.ArtifactType.Name( |
| 483 | artifact_type |
| 484 | ) |
| 485 | ) |
| 486 | logging.warning( |
| 487 | "%s artifact generation failed with exception %s", |
| 488 | artifact_name, |
| 489 | e, |
| 490 | ) |
| 491 | logging.warning("traceback:\n%s", traceback.format_exc()) |
| 492 | continue |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 493 | if paths: |
| 494 | generated.append( |
| 495 | { |
| 496 | "paths": [paths] |
| 497 | if isinstance(paths, str) |
| 498 | else paths, |
| 499 | "type": artifact_type, |
| 500 | } |
| 501 | ) |
George Engelbrecht | 764b1cd | 2021-06-18 17:01:07 -0600 | [diff] [blame] | 502 | |
Alex Klein | 1699fab | 2022-09-08 08:46:06 -0600 | [diff] [blame] | 503 | return generated |