Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 1 | # Copyright 2019 The Chromium OS Authors. All rights reserved. |
Xinan Lin | c61196b | 2019-08-13 10:37:30 -0700 | [diff] [blame] | 2 | # Use of this source code is governed by a BSD-style license that can be |
| 3 | # found in the LICENSE file. |
Xinan Lin | c61196b | 2019-08-13 10:37:30 -0700 | [diff] [blame] | 4 | """Module for interacting with Buildbucket.""" |
Xinan Lin | c61196b | 2019-08-13 10:37:30 -0700 | [diff] [blame] | 5 | |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 6 | import collections |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 7 | import datetime |
linxinan | e5eb455 | 2019-08-26 05:44:45 +0000 | [diff] [blame] | 8 | import logging |
Azizur Rahman | b720e51 | 2022-05-13 18:30:40 +0000 | [diff] [blame] | 9 | import os |
Xinan Lin | 081b5d3 | 2020-03-23 17:37:55 -0700 | [diff] [blame] | 10 | import uuid |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 11 | |
Xinan Lin | df0698a | 2020-02-05 22:38:11 -0800 | [diff] [blame] | 12 | import analytics |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 13 | import build_lib |
Xinan Lin | c61196b | 2019-08-13 10:37:30 -0700 | [diff] [blame] | 14 | import constants |
| 15 | import file_getter |
Garry Wang | 111a26f | 2021-07-23 15:25:14 -0700 | [diff] [blame] | 16 | from multi_duts_lib import restruct_secondary_targets_from_string |
Xinan Lin | c61196b | 2019-08-13 10:37:30 -0700 | [diff] [blame] | 17 | |
Jacob Kopczynski | 408f3fd | 2020-08-07 13:31:59 -0700 | [diff] [blame] | 18 | from chromite.api.gen.test_platform import request_pb2 as ctp_request |
Xinan Lin | c61196b | 2019-08-13 10:37:30 -0700 | [diff] [blame] | 19 | from components import auth |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 20 | from components.prpc import client as prpc_client |
Sean McAllister | af32bfb | 2021-07-19 09:20:17 -0600 | [diff] [blame] | 21 | from chromite.third_party.infra_libs.buildbucket.proto import common_pb2 as bb_common_pb2 |
Azizur Rahman | b720e51 | 2022-05-13 18:30:40 +0000 | [diff] [blame] | 22 | from chromite.third_party.infra_libs.buildbucket.proto import builds_service_pb2, builder_common_pb2 |
Sean McAllister | af32bfb | 2021-07-19 09:20:17 -0600 | [diff] [blame] | 23 | from chromite.third_party.infra_libs.buildbucket.proto.builds_service_prpc_pb2 import BuildsServiceDescription |
Azizur Rahman | a9a189f | 2022-07-29 20:28:28 +0000 | [diff] [blame] | 24 | from chromite.api.gen.chromiumos.test.api import test_suite_pb2 |
Xinan Lin | c61196b | 2019-08-13 10:37:30 -0700 | [diff] [blame] | 25 | |
| 26 | from oauth2client import service_account |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 27 | |
Sean McAllister | 66bf7e9 | 2021-07-16 18:46:04 +0000 | [diff] [blame] | 28 | from google.protobuf import json_format, struct_pb2 |
Xinan Lin | c61196b | 2019-08-13 10:37:30 -0700 | [diff] [blame] | 29 | |
Xinan Lin | 6e09738 | 2019-08-27 18:43:35 -0700 | [diff] [blame] | 30 | GS_PREFIX = 'gs://chromeos-image-archive/' |
Azizur Rahman | b720e51 | 2022-05-13 18:30:40 +0000 | [diff] [blame] | 31 | CONTAINER_METADATA_LOC = 'metadata/containers.jsonpb' |
Xinan Lin | 6e09738 | 2019-08-27 18:43:35 -0700 | [diff] [blame] | 32 | |
Xinan Lin | 57e2d96 | 2020-03-30 17:24:53 -0700 | [diff] [blame] | 33 | # The default prpc timeout(10 sec) is too short for the request_id |
| 34 | # to propgate in buildbucket, and could not fully dedup the |
| 35 | # ScheduleBuild request. Increase it while not hitting the default |
| 36 | # GAE deadline(60 sec) |
| 37 | PRPC_TIMEOUT_SEC = 55 |
| 38 | |
| 39 | |
Xinan Lin | c61196b | 2019-08-13 10:37:30 -0700 | [diff] [blame] | 40 | def _get_client(address): |
| 41 | """Create a prpc client instance for given address.""" |
| 42 | return prpc_client.Client(address, BuildsServiceDescription) |
| 43 | |
| 44 | |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 45 | class BuildbucketRunError(Exception): |
| 46 | """Raised when interactions with buildbucket server fail.""" |
| 47 | |
| 48 | |
Xinan Lin | c61196b | 2019-08-13 10:37:30 -0700 | [diff] [blame] | 49 | class TestPlatformClient(object): |
| 50 | """prpc client for cros_test_platform, aka frontdoor.""" |
Xinan Lin | c61196b | 2019-08-13 10:37:30 -0700 | [diff] [blame] | 51 | def __init__(self, address, project, bucket, builder): |
| 52 | self.client = _get_client(address) |
Azizur Rahman | b720e51 | 2022-05-13 18:30:40 +0000 | [diff] [blame] | 53 | self.builder = builder_common_pb2.BuilderID(project=project, |
Sean McAllister | af32bfb | 2021-07-19 09:20:17 -0600 | [diff] [blame] | 54 | bucket=bucket, |
| 55 | builder=builder) |
Xinan Lin | c61196b | 2019-08-13 10:37:30 -0700 | [diff] [blame] | 56 | self.scope = 'https://www.googleapis.com/auth/userinfo.email' |
| 57 | self.running_env = constants.environment() |
| 58 | |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 59 | def multirequest_run(self, tasks, suite): |
Xinan Lin | 1e8e791 | 2020-07-31 09:52:16 -0700 | [diff] [blame] | 60 | """Call cros_test_platform Builder to schedule a batch of suite tests. |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 61 | |
| 62 | Args: |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 63 | tasks: The suite tasks to run. |
| 64 | suite: suite name for the batch request. |
| 65 | |
| 66 | Returns: |
| 67 | List of executed tasks. |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 68 | |
| 69 | Raises: |
| 70 | BuildbucketRunError: if failed to get build info from task parameters. |
| 71 | """ |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 72 | requests = [] |
| 73 | executed_tasks = [] |
| 74 | counter = collections.defaultdict(int) |
Xinan Lin | df0698a | 2020-02-05 22:38:11 -0800 | [diff] [blame] | 75 | task_executions = [] |
Jared Loucks | 438bf3a | 2022-04-06 13:43:15 -0600 | [diff] [blame] | 76 | req_tags = _bb_tags(suite) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 77 | for task in tasks: |
| 78 | try: |
| 79 | params = task.extract_params() |
Xinan Lin | c54a746 | 2020-04-17 15:39:01 -0700 | [diff] [blame] | 80 | if _should_skip(params): |
| 81 | continue |
Xinan Lin | 8bb5b4b | 2020-07-21 23:17:55 -0700 | [diff] [blame] | 82 | req = _form_test_platform_request(params) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 83 | req_json = json_format.MessageToJson(req) |
| 84 | counter_key = params['board'] |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 85 | counter_key += '' if params['model'] in [None, 'None' |
| 86 | ] else ('_' + params['model']) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 87 | counter[counter_key] += 1 |
| 88 | req_name = counter_key |
| 89 | if counter[counter_key] > 1: |
| 90 | req_name += '_' + str(counter[counter_key] - 1) |
| 91 | requests.append('"%s": %s' % (req_name, req_json)) |
Jared Loucks | 438bf3a | 2022-04-06 13:43:15 -0600 | [diff] [blame] | 92 | req_tags.append( |
| 93 | bb_common_pb2.StringPair( |
| 94 | key='label-image', |
| 95 | value=_infer_build_from_task_params(params))) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 96 | executed_tasks.append(task) |
Xinan Lin | 083ba8f | 2020-02-06 13:55:18 -0800 | [diff] [blame] | 97 | if params.get('task_id'): |
Xinan Lin | 9b17c5b | 2020-08-06 10:43:30 -0700 | [diff] [blame] | 98 | task_executions.append( |
| 99 | analytics.ExecutionTask(params['task_id'], req_name)) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 100 | except (ValueError, BuildbucketRunError): |
Xinan Lin | ba3b932 | 2020-04-24 15:08:12 -0700 | [diff] [blame] | 101 | logging.error('Failed to process task: %r', params) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 102 | if not requests: |
| 103 | return [] |
| 104 | try: |
| 105 | requests_json = '{ "requests": { %s } }' % ', '.join(requests) |
Jared Loucks | 438bf3a | 2022-04-06 13:43:15 -0600 | [diff] [blame] | 106 | req_build = self._build_request(requests_json, req_tags) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 107 | logging.debug('Raw request to buildbucket: %r', req_build) |
linxinan | e5eb455 | 2019-08-26 05:44:45 +0000 | [diff] [blame] | 108 | |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 109 | if (self.running_env == constants.RunningEnv.ENV_STANDALONE |
| 110 | or self.running_env == constants.RunningEnv.ENV_DEVELOPMENT_SERVER): |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 111 | # If running locally, use the staging service account. |
| 112 | sa_key = self._gen_service_account_key( |
| 113 | file_getter.STAGING_CLIENT_SECRETS_FILE) |
| 114 | cred = prpc_client.service_account_credentials( |
| 115 | service_account_key=sa_key) |
| 116 | else: |
| 117 | cred = prpc_client.service_account_credentials() |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 118 | resp = self.client.ScheduleBuild(req_build, |
| 119 | credentials=cred, |
| 120 | timeout=PRPC_TIMEOUT_SEC) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 121 | logging.debug('Response from buildbucket: %r', resp) |
Xinan Lin | df0698a | 2020-02-05 22:38:11 -0800 | [diff] [blame] | 122 | for t in task_executions: |
| 123 | t.update_result(resp) |
| 124 | try: |
| 125 | if not t.upload(): |
| 126 | logging.warning('Failed to insert row: %r', t) |
| 127 | # For any exceptions from BQ, only log it. |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 128 | except Exception as e: #pylint: disable=broad-except |
| 129 | logging.exception('Failed to insert row: %r, got error: %s', t, |
| 130 | str(e)) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 131 | return executed_tasks |
| 132 | except Exception as e: |
| 133 | logging.debug('Failed to process tasks: %r', tasks) |
| 134 | logging.exception(str(e)) |
| 135 | return [] |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 136 | |
Xinan Lin | c61196b | 2019-08-13 10:37:30 -0700 | [diff] [blame] | 137 | def dummy_run(self): |
| 138 | """Perform a dummy run of prpc call to cros_test_platform-dev.""" |
| 139 | |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 140 | requests_json = '{ "requests": { "dummy": {} } }' |
| 141 | req_build = self._build_request(requests_json, tags=None) |
Xinan Lin | c61196b | 2019-08-13 10:37:30 -0700 | [diff] [blame] | 142 | # Use the staging service account to authorize the request. |
| 143 | sa_key = self._gen_service_account_key( |
| 144 | file_getter.STAGING_CLIENT_SECRETS_FILE) |
| 145 | cred = prpc_client.service_account_credentials(service_account_key=sa_key) |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 146 | return self.client.ScheduleBuild(req_build, |
| 147 | credentials=cred, |
| 148 | timeout=PRPC_TIMEOUT_SEC) |
Xinan Lin | c61196b | 2019-08-13 10:37:30 -0700 | [diff] [blame] | 149 | |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 150 | def _build_request(self, reqs_json, tags): |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 151 | """Generate ScheduleBuildRequest for calling buildbucket. |
Xinan Lin | c61196b | 2019-08-13 10:37:30 -0700 | [diff] [blame] | 152 | |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 153 | Args: |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 154 | reqs_json: A json string of requests. |
Prathmesh Prabhu | 2382a18 | 2019-09-07 21:18:10 -0700 | [diff] [blame] | 155 | tags: A list of tags for the buildbucket build. |
Xinan Lin | c61196b | 2019-08-13 10:37:30 -0700 | [diff] [blame] | 156 | |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 157 | Returns: |
| 158 | A ScheduleBuildRequest instance. |
| 159 | """ |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 160 | requests_struct = struct_pb2.Struct() |
| 161 | recipe_struct = json_format.Parse(reqs_json, requests_struct) |
Sean McAllister | af32bfb | 2021-07-19 09:20:17 -0600 | [diff] [blame] | 162 | return builds_service_pb2.ScheduleBuildRequest(builder=self.builder, |
| 163 | properties=recipe_struct, |
| 164 | request_id=str( |
| 165 | uuid.uuid1()), |
| 166 | tags=tags) |
Xinan Lin | c61196b | 2019-08-13 10:37:30 -0700 | [diff] [blame] | 167 | |
| 168 | def _gen_service_account_key(self, sa): |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 169 | """Generate credentials to authorize the call. |
Xinan Lin | c61196b | 2019-08-13 10:37:30 -0700 | [diff] [blame] | 170 | |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 171 | Args: |
| 172 | sa: A string of the path to the service account json file. |
Xinan Lin | c61196b | 2019-08-13 10:37:30 -0700 | [diff] [blame] | 173 | |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 174 | Returns: |
| 175 | A service account key. |
| 176 | """ |
Xinan Lin | c61196b | 2019-08-13 10:37:30 -0700 | [diff] [blame] | 177 | service_credentials = service_account.ServiceAccountCredentials |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 178 | key = service_credentials.from_json_keyfile_name(sa, self.scope) |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 179 | return auth.ServiceAccountKey(client_email=key.service_account_email, |
| 180 | private_key=key._private_key_pkcs8_pem, |
| 181 | private_key_id=key._private_key_id) |
Prathmesh Prabhu | 1918a8f | 2019-09-07 21:37:37 -0700 | [diff] [blame] | 182 | |
| 183 | |
Prathmesh Prabhu | 06852fe | 2019-09-09 07:58:43 -0700 | [diff] [blame] | 184 | def _form_test_platform_request(task_params): |
Jacob Kopczynski | 408f3fd | 2020-08-07 13:31:59 -0700 | [diff] [blame] | 185 | """Generates test_platform.Request proto to send to buildbucket. |
Prathmesh Prabhu | 06852fe | 2019-09-09 07:58:43 -0700 | [diff] [blame] | 186 | |
| 187 | Args: |
Jacob Kopczynski | 408f3fd | 2020-08-07 13:31:59 -0700 | [diff] [blame] | 188 | task_params: dict containing the parameters of a task from the suite queue. |
Prathmesh Prabhu | 06852fe | 2019-09-09 07:58:43 -0700 | [diff] [blame] | 189 | |
| 190 | Returns: |
Jacob Kopczynski | 408f3fd | 2020-08-07 13:31:59 -0700 | [diff] [blame] | 191 | A ctp_request.Request instance. |
Prathmesh Prabhu | 06852fe | 2019-09-09 07:58:43 -0700 | [diff] [blame] | 192 | """ |
| 193 | build = _infer_build_from_task_params(task_params) |
| 194 | if build == 'None': |
| 195 | raise BuildbucketRunError('No proper build in task params: %r' % |
| 196 | task_params) |
| 197 | pool = _infer_pool_from_task_params(task_params) |
| 198 | timeout = _infer_timeout_from_task_params(task_params) |
| 199 | |
Jacob Kopczynski | 408f3fd | 2020-08-07 13:31:59 -0700 | [diff] [blame] | 200 | request = ctp_request.Request() |
Prathmesh Prabhu | 06852fe | 2019-09-09 07:58:43 -0700 | [diff] [blame] | 201 | params = request.params |
| 202 | |
Xinan Lin | 4757d6f | 2020-03-24 22:20:31 -0700 | [diff] [blame] | 203 | params.scheduling.CopyFrom(_scheduling_for_pool(pool)) |
Prathmesh Prabhu | 46156ff | 2020-06-20 00:18:52 -0700 | [diff] [blame] | 204 | if task_params.get('qs_account') not in ['None', None]: |
Xinan Lin | 4757d6f | 2020-03-24 22:20:31 -0700 | [diff] [blame] | 205 | params.scheduling.qs_account = task_params.get('qs_account') |
| 206 | # Quota Scheduler has no concept of priority. |
Xinan Lin | 8bb5b4b | 2020-07-21 23:17:55 -0700 | [diff] [blame] | 207 | if (task_params.get('priority') not in ['None', None] |
| 208 | and not params.scheduling.qs_account): |
Prathmesh Prabhu | 06852fe | 2019-09-09 07:58:43 -0700 | [diff] [blame] | 209 | params.scheduling.priority = int(task_params['priority']) |
| 210 | |
| 211 | params.software_dependencies.add().chromeos_build = build |
| 212 | params.software_attributes.build_target.name = task_params['board'] |
Aviv Keshet | c679faf | 2019-11-27 17:52:50 -0800 | [diff] [blame] | 213 | |
| 214 | gs_url = GS_PREFIX + task_params['test_source_build'] |
| 215 | params.metadata.test_metadata_url = gs_url |
| 216 | params.metadata.debug_symbols_archive_url = gs_url |
Azizur Rahman | b720e51 | 2022-05-13 18:30:40 +0000 | [diff] [blame] | 217 | params.metadata.container_metadata_url = os.path.join(gs_url, CONTAINER_METADATA_LOC) |
Aviv Keshet | c679faf | 2019-11-27 17:52:50 -0800 | [diff] [blame] | 218 | |
Prathmesh Prabhu | 06852fe | 2019-09-09 07:58:43 -0700 | [diff] [blame] | 219 | params.time.maximum_duration.FromTimedelta(timeout) |
| 220 | |
| 221 | for key, value in _request_tags(task_params, build, pool).iteritems(): |
| 222 | params.decorations.tags.append('%s:%s' % (key, value)) |
| 223 | |
Xinan Lin | 7bf266a | 2020-06-10 23:54:26 -0700 | [diff] [blame] | 224 | for d in _infer_user_defined_dimensions(task_params): |
| 225 | params.freeform_attributes.swarming_dimensions.append(d) |
Xinan Lin | ba3b932 | 2020-04-24 15:08:12 -0700 | [diff] [blame] | 226 | |
Prathmesh Prabhu | 06852fe | 2019-09-09 07:58:43 -0700 | [diff] [blame] | 227 | if task_params['model'] != 'None': |
| 228 | params.hardware_attributes.model = task_params['model'] |
| 229 | |
Azizur Rahman | ee62a66 | 2022-06-02 18:53:15 +0000 | [diff] [blame] | 230 | if task_params.has_key('job_retry') and task_params['job_retry'] == 'True': |
Prathmesh Prabhu | 06852fe | 2019-09-09 07:58:43 -0700 | [diff] [blame] | 231 | params.retry.allow = True |
| 232 | params.retry.max = constants.Buildbucket.MAX_RETRY |
| 233 | |
Azizur Rahman | ee62a66 | 2022-06-02 18:53:15 +0000 | [diff] [blame] | 234 | params.run_via_cft = (task_params.has_key('run_via_cft') and task_params['run_via_cft'] == 'True') |
Azizur Rahman | b720e51 | 2022-05-13 18:30:40 +0000 | [diff] [blame] | 235 | |
Prathmesh Prabhu | 06852fe | 2019-09-09 07:58:43 -0700 | [diff] [blame] | 236 | fw_rw_build = task_params.get(build_lib.BuildVersionKey.FW_RW_VERSION) |
| 237 | fw_ro_build = task_params.get(build_lib.BuildVersionKey.FW_RO_VERSION) |
Brigit Rossbach | bb08091 | 2020-11-18 13:52:17 -0700 | [diff] [blame] | 238 | |
| 239 | if task_params.has_key('firmware_ro_version'): |
| 240 | build = params.software_dependencies.add() |
| 241 | build.ro_firmware_build = task_params['firmware_ro_version'] |
| 242 | |
Prathmesh Prabhu | 06852fe | 2019-09-09 07:58:43 -0700 | [diff] [blame] | 243 | # Skip firmware field if None(unspecified) or 'None'(no such build). |
| 244 | if fw_ro_build not in (None, 'None'): |
| 245 | build = params.software_dependencies.add() |
| 246 | build.ro_firmware_build = fw_ro_build |
Brigit Rossbach | bb08091 | 2020-11-18 13:52:17 -0700 | [diff] [blame] | 247 | |
| 248 | if task_params.has_key('firmware_rw_version'): |
| 249 | build = params.software_dependencies.add() |
| 250 | build.rw_firmware_build = task_params['firmware_rw_version'] |
| 251 | |
Prathmesh Prabhu | 06852fe | 2019-09-09 07:58:43 -0700 | [diff] [blame] | 252 | if fw_rw_build not in (None, 'None'): |
| 253 | build = params.software_dependencies.add() |
| 254 | build.rw_firmware_build = fw_rw_build |
| 255 | |
Garry Wang | 2104bf5 | 2021-07-19 18:13:12 -0700 | [diff] [blame] | 256 | secondary_targets = task_params.get('secondary_targets') |
Garry Wang | 111a26f | 2021-07-23 15:25:14 -0700 | [diff] [blame] | 257 | secondary_targets = restruct_secondary_targets_from_string(secondary_targets) |
Garry Wang | 2104bf5 | 2021-07-19 18:13:12 -0700 | [diff] [blame] | 258 | if secondary_targets: |
| 259 | for s_target in secondary_targets: |
| 260 | s_device = params.secondary_devices.add() |
| 261 | s_device.software_attributes.build_target.name = s_target.board |
| 262 | if s_target.model: |
| 263 | s_device.hardware_attributes.model = s_target.model |
| 264 | if s_target.cros_build: |
| 265 | s_build = s_device.software_dependencies.add() |
| 266 | s_build.chromeos_build = s_target.cros_build |
| 267 | |
Prathmesh Prabhu | 06852fe | 2019-09-09 07:58:43 -0700 | [diff] [blame] | 268 | request.test_plan.suite.add().name = task_params['suite'] |
Azizur Rahman | a9a189f | 2022-07-29 20:28:28 +0000 | [diff] [blame] | 269 | if params.run_via_cft: |
| 270 | include_tags = task_params.get('include_tags', None) |
| 271 | exclude_tags = task_params.get('exclude_tags', None) |
| 272 | if include_tags or exclude_tags: |
| 273 | request.test_plan.tag_criteria.CopyFrom( |
| 274 | test_suite_pb2.TestSuite.TestCaseTagCriteria( |
| 275 | tags=_get_test_tags(include_tags), |
| 276 | tag_excludes=_get_test_tags(exclude_tags))) |
Prathmesh Prabhu | 06852fe | 2019-09-09 07:58:43 -0700 | [diff] [blame] | 277 | return request |
| 278 | |
Sean McAllister | af32bfb | 2021-07-19 09:20:17 -0600 | [diff] [blame] | 279 | |
Prathmesh Prabhu | 1918a8f | 2019-09-07 21:37:37 -0700 | [diff] [blame] | 280 | def _scheduling_for_pool(pool): |
| 281 | """Assign appropriate pool name to scheduling instance. |
| 282 | |
| 283 | Args: |
Xinan Lin | 1516edb | 2020-07-05 23:13:54 -0700 | [diff] [blame] | 284 | pool: string pool name (e.g. 'MANAGED_POOL_QUOTA', 'wificell'). |
Prathmesh Prabhu | 1918a8f | 2019-09-07 21:37:37 -0700 | [diff] [blame] | 285 | |
| 286 | Returns: |
Jacob Kopczynski | 408f3fd | 2020-08-07 13:31:59 -0700 | [diff] [blame] | 287 | scheduling: A ctp_request.Request.Params.Scheduling instance. |
Prathmesh Prabhu | 1918a8f | 2019-09-07 21:37:37 -0700 | [diff] [blame] | 288 | """ |
Jacob Kopczynski | 408f3fd | 2020-08-07 13:31:59 -0700 | [diff] [blame] | 289 | mp = ctp_request.Request.Params.Scheduling.ManagedPool |
Prathmesh Prabhu | 1918a8f | 2019-09-07 21:37:37 -0700 | [diff] [blame] | 290 | if mp.DESCRIPTOR.values_by_name.get(pool) is not None: |
Jacob Kopczynski | 408f3fd | 2020-08-07 13:31:59 -0700 | [diff] [blame] | 291 | return ctp_request.Request.Params.Scheduling(managed_pool=mp.Value(pool)) |
| 292 | return ctp_request.Request.Params.Scheduling(unmanaged_pool=pool) |
Prathmesh Prabhu | 1918a8f | 2019-09-07 21:37:37 -0700 | [diff] [blame] | 293 | |
| 294 | |
Prathmesh Prabhu | 06852fe | 2019-09-09 07:58:43 -0700 | [diff] [blame] | 295 | # Also see: A copy of this function in swarming_lib.py. |
| 296 | def _infer_build_from_task_params(task_params): |
| 297 | """Infer the build to install on the DUT for the scheduled task. |
Prathmesh Prabhu | 1918a8f | 2019-09-07 21:37:37 -0700 | [diff] [blame] | 298 | |
Prathmesh Prabhu | 06852fe | 2019-09-09 07:58:43 -0700 | [diff] [blame] | 299 | Args: |
| 300 | task_params: The parameters of a task loaded from suite queue. |
Prathmesh Prabhu | 1918a8f | 2019-09-07 21:37:37 -0700 | [diff] [blame] | 301 | |
Prathmesh Prabhu | 06852fe | 2019-09-09 07:58:43 -0700 | [diff] [blame] | 302 | Returns: |
| 303 | A string representing the build to run. |
| 304 | """ |
| 305 | cros_build = task_params[build_lib.BuildVersionKey.CROS_VERSION] |
| 306 | android_build = task_params[build_lib.BuildVersionKey.ANDROID_BUILD_VERSION] |
| 307 | testbed_build = task_params[build_lib.BuildVersionKey.TESTBED_BUILD_VERSION] |
| 308 | return cros_build or android_build or testbed_build |
| 309 | |
| 310 | |
| 311 | def _infer_pool_from_task_params(task_params): |
| 312 | """Infer the pool to use for the scheduled task. |
| 313 | |
| 314 | Args: |
| 315 | task_params: The parameters of a task loaded from suite queue. |
| 316 | |
| 317 | Returns: |
| 318 | A string pool to schedule task in. |
| 319 | """ |
| 320 | if task_params.get('override_qs_account'): |
| 321 | return 'DUT_POOL_QUOTA' |
| 322 | return task_params.get('override_pool') or task_params['pool'] |
| 323 | |
| 324 | |
| 325 | def _infer_timeout_from_task_params(task_params): |
| 326 | """Infer the timeout for the scheduled task. |
| 327 | |
| 328 | Args: |
| 329 | task_params: The parameters of a task loaded from suite queue. |
| 330 | |
| 331 | Returns: |
| 332 | A datetime.timedelta instance for the timeout. |
| 333 | """ |
| 334 | timeout_mins = int(task_params['timeout_mins']) |
| 335 | # timeout's unit is hour. |
| 336 | if task_params.get('timeout'): |
Sean Abraham | ec0d076 | 2020-09-18 17:19:05 +0000 | [diff] [blame] | 337 | timeout_mins = max(int(task_params['timeout']) * 60, timeout_mins) |
Prathmesh Prabhu | 06852fe | 2019-09-09 07:58:43 -0700 | [diff] [blame] | 338 | if timeout_mins > constants.Buildbucket.MAX_BUILDBUCKET_TIMEOUT_MINS: |
| 339 | timeout_mins = constants.Buildbucket.MAX_BUILDBUCKET_TIMEOUT_MINS |
| 340 | return datetime.timedelta(minutes=timeout_mins) |
| 341 | |
Prathmesh Prabhu | 1918a8f | 2019-09-07 21:37:37 -0700 | [diff] [blame] | 342 | |
Xinan Lin | ba3b932 | 2020-04-24 15:08:12 -0700 | [diff] [blame] | 343 | def _infer_user_defined_dimensions(task_params): |
| 344 | """Infer the dimensions defined by users. |
| 345 | |
| 346 | Args: |
| 347 | task_params: The parameters of a task loaded from suite queue. |
| 348 | |
| 349 | Returns: |
| 350 | A list of strings; an empty list if no dimensions set. |
| 351 | |
| 352 | Raises: |
| 353 | ValueError: if dimension is not valid. |
| 354 | """ |
| 355 | result = [] |
| 356 | if task_params.get('dimensions') in (None, 'None'): |
| 357 | return result |
Patrick Meiring | d4f6077 | 2021-03-04 12:09:28 +1100 | [diff] [blame] | 358 | dims = [d.lstrip() for d in task_params.get('dimensions').split(',')] |
| 359 | for d in dims: |
Xinan Lin | ba3b932 | 2020-04-24 15:08:12 -0700 | [diff] [blame] | 360 | if len(d.split(':')) != 2: |
| 361 | raise ValueError( |
| 362 | 'Job %s has invalid dimensions: %s' % |
| 363 | (task_params.get('name'), task_params.get('dimensions'))) |
| 364 | result.append(d) |
| 365 | return result |
| 366 | |
| 367 | |
Prathmesh Prabhu | 2382a18 | 2019-09-07 21:18:10 -0700 | [diff] [blame] | 368 | def _request_tags(task_params, build, pool): |
| 369 | """Infer tags to include in cros_test_platform request. |
| 370 | |
| 371 | Args: |
| 372 | task_params: suite task parameters. |
| 373 | build: The build included in the request. Must not be None. |
| 374 | pool: The DUT pool used for the request. Must not be None. |
| 375 | |
| 376 | Returns: |
| 377 | A dict of tags. |
| 378 | """ |
| 379 | tags = { |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 380 | 'build': build, |
| 381 | 'label-pool': pool, |
Taylor Clark | e12ec9a | 2021-02-18 22:22:19 +0000 | [diff] [blame] | 382 | 'ctp-fwd-task-name': task_params.get('name') |
Prathmesh Prabhu | 2382a18 | 2019-09-07 21:18:10 -0700 | [diff] [blame] | 383 | } |
Xinan Lin | fb63d57 | 2019-09-24 15:49:04 -0700 | [diff] [blame] | 384 | if task_params.get('board') not in (None, 'None'): |
Prathmesh Prabhu | 2382a18 | 2019-09-07 21:18:10 -0700 | [diff] [blame] | 385 | tags['label-board'] = task_params['board'] |
Xinan Lin | fb63d57 | 2019-09-24 15:49:04 -0700 | [diff] [blame] | 386 | if task_params.get('model') not in (None, 'None'): |
Prathmesh Prabhu | 2382a18 | 2019-09-07 21:18:10 -0700 | [diff] [blame] | 387 | tags['label-model'] = task_params['model'] |
Xinan Lin | fb63d57 | 2019-09-24 15:49:04 -0700 | [diff] [blame] | 388 | if task_params.get('suite') not in (None, 'None'): |
Prathmesh Prabhu | 2382a18 | 2019-09-07 21:18:10 -0700 | [diff] [blame] | 389 | tags['suite'] = task_params['suite'] |
Taylor Clark | e12ec9a | 2021-02-18 22:22:19 +0000 | [diff] [blame] | 390 | if task_params.get('analytics_name') not in (None, 'None'): |
| 391 | tags['analytics_name'] = task_params['analytics_name'] |
Prathmesh Prabhu | 2382a18 | 2019-09-07 21:18:10 -0700 | [diff] [blame] | 392 | return tags |
| 393 | |
| 394 | |
Azizur Rahman | a9a189f | 2022-07-29 20:28:28 +0000 | [diff] [blame] | 395 | def _get_test_tags(tags_list_str): |
| 396 | """parse test tags properly and return them. |
| 397 | |
| 398 | Args: |
| 399 | tags_list_str: comma separated list of tags. |
| 400 | |
| 401 | Returns: |
| 402 | A list of strings; an empty list if no tags set. |
| 403 | """ |
| 404 | if tags_list_str not in (None, '', 'None'): |
| 405 | return [d.lstrip() for d in tags_list_str.split(',')] |
| 406 | return [] |
| 407 | |
| 408 | |
Xinan Lin | c864711 | 2020-02-04 16:45:56 -0800 | [diff] [blame] | 409 | def _get_key_val_from_label(label): |
| 410 | """A helper to get key and value from the label. |
| 411 | |
| 412 | Args: |
| 413 | label: A string of label, should be in the form of |
| 414 | key:value, e.g. 'pool:ChromeOSSkylab'. |
| 415 | """ |
| 416 | res = label.split(':') |
| 417 | if len(res) == 2: |
| 418 | return res[0], res[1] |
| 419 | logging.warning('Failed to parse the label, %s', label) |
| 420 | |
| 421 | |
Dhanya Ganesh | edb78cb | 2020-07-20 19:40:50 +0000 | [diff] [blame] | 422 | def _bb_tags(suite): |
| 423 | """Get all the tags required for Buildbucket. |
Prathmesh Prabhu | 2382a18 | 2019-09-07 21:18:10 -0700 | [diff] [blame] | 424 | |
| 425 | Args: |
Dhanya Ganesh | edb78cb | 2020-07-20 19:40:50 +0000 | [diff] [blame] | 426 | suite: A string of suite name being scheduled. |
Prathmesh Prabhu | 2382a18 | 2019-09-07 21:18:10 -0700 | [diff] [blame] | 427 | |
| 428 | Returns: |
| 429 | [bb_common_pb2.StringPair] tags to include the buildbucket request. |
| 430 | """ |
Sean McAllister | af32bfb | 2021-07-19 09:20:17 -0600 | [diff] [blame] | 431 | return [ |
Jared Loucks | 438bf3a | 2022-04-06 13:43:15 -0600 | [diff] [blame] | 432 | bb_common_pb2.StringPair(key='label-suite', value=suite), |
Sean McAllister | af32bfb | 2021-07-19 09:20:17 -0600 | [diff] [blame] | 433 | bb_common_pb2.StringPair(key='suite', value=suite), |
| 434 | bb_common_pb2.StringPair(key='user_agent', value='suite_scheduler') |
| 435 | ] |
Xinan Lin | c54a746 | 2020-04-17 15:39:01 -0700 | [diff] [blame] | 436 | |
| 437 | |
| 438 | def _should_skip(params): |
| 439 | """Decide whether to skip a task based on env and pool. |
| 440 | |
| 441 | Suite request from staging may still have a small chance to run |
| 442 | in production. However, for unmanaged pools(e.g. wificell), which |
| 443 | usually are small, dev traffic is unacceptable. |
| 444 | |
| 445 | Args: |
| 446 | params: dict containing the parameters of a task got from suite |
| 447 | queue. |
| 448 | |
| 449 | Returns: |
| 450 | A boolean; true for suite targetting non-default pools from staging |
| 451 | env. |
| 452 | """ |
| 453 | if constants.application_id() == constants.AppID.PROD_APP: |
| 454 | return False |
Xinan Lin | 0d7910d | 2020-07-21 11:06:45 -0700 | [diff] [blame] | 455 | return params['pool'] != 'MANAGED_POOL_QUOTA' |