blob: 9eb4993e9d9b366386f3ec551ba464ee2131cb81 [file] [log] [blame]
Xinan Lin3ba18a02019-08-13 15:44:55 -07001# Copyright 2019 The Chromium OS Authors. All rights reserved.
Xinan Linc61196b2019-08-13 10:37:30 -07002# Use of this source code is governed by a BSD-style license that can be
3# found in the LICENSE file.
Xinan Linc61196b2019-08-13 10:37:30 -07004"""Module for interacting with Buildbucket."""
Xinan Linc61196b2019-08-13 10:37:30 -07005
Xinan Lin9e4917d2019-11-04 10:58:47 -08006import collections
Xinan Lin3ba18a02019-08-13 15:44:55 -07007import datetime
linxinane5eb4552019-08-26 05:44:45 +00008import logging
Xinan Lin3ba18a02019-08-13 15:44:55 -07009import re
10import string
Jacob Kopczynskie56bf492020-08-07 13:20:12 -070011import types
Xinan Lin081b5d32020-03-23 17:37:55 -070012import uuid
Xinan Lin3ba18a02019-08-13 15:44:55 -070013
Xinan Lindf0698a2020-02-05 22:38:11 -080014import analytics
Xinan Lin3ba18a02019-08-13 15:44:55 -070015import build_lib
Xinan Linc61196b2019-08-13 10:37:30 -070016import constants
17import file_getter
Garry Wang111a26f2021-07-23 15:25:14 -070018from multi_duts_lib import restruct_secondary_targets_from_string
Xinan Linc61196b2019-08-13 10:37:30 -070019
Jacob Kopczynski408f3fd2020-08-07 13:31:59 -070020from chromite.api.gen.test_platform import request_pb2 as ctp_request
Xinan Linc61196b2019-08-13 10:37:30 -070021from components import auth
Xinan Lin3ba18a02019-08-13 15:44:55 -070022from components.prpc import client as prpc_client
Sean McAllisteraf32bfb2021-07-19 09:20:17 -060023from chromite.third_party.infra_libs.buildbucket.proto import common_pb2 as bb_common_pb2
24from chromite.third_party.infra_libs.buildbucket.proto import builds_service_pb2, builder_pb2, build_pb2
25from chromite.third_party.infra_libs.buildbucket.proto.builds_service_prpc_pb2 import BuildsServiceDescription
Xinan Linc61196b2019-08-13 10:37:30 -070026
27from oauth2client import service_account
Xinan Lin3ba18a02019-08-13 15:44:55 -070028
Sean McAllister66bf7e92021-07-16 18:46:04 +000029from google.protobuf import json_format, struct_pb2
Xinan Linc61196b2019-08-13 10:37:30 -070030
Xinan Lin6e097382019-08-27 18:43:35 -070031GS_PREFIX = 'gs://chromeos-image-archive/'
32
Xinan Lin57e2d962020-03-30 17:24:53 -070033# 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)
37PRPC_TIMEOUT_SEC = 55
38
39
Xinan Linc61196b2019-08-13 10:37:30 -070040def _get_client(address):
41 """Create a prpc client instance for given address."""
42 return prpc_client.Client(address, BuildsServiceDescription)
43
44
Xinan Lin3ba18a02019-08-13 15:44:55 -070045class BuildbucketRunError(Exception):
46 """Raised when interactions with buildbucket server fail."""
47
48
Xinan Linc61196b2019-08-13 10:37:30 -070049class TestPlatformClient(object):
50 """prpc client for cros_test_platform, aka frontdoor."""
Xinan Linc61196b2019-08-13 10:37:30 -070051 def __init__(self, address, project, bucket, builder):
52 self.client = _get_client(address)
Sean McAllisteraf32bfb2021-07-19 09:20:17 -060053 self.builder = builder_pb2.BuilderID(project=project,
54 bucket=bucket,
55 builder=builder)
Xinan Linc61196b2019-08-13 10:37:30 -070056 self.scope = 'https://www.googleapis.com/auth/userinfo.email'
57 self.running_env = constants.environment()
58
Xinan Lin9e4917d2019-11-04 10:58:47 -080059 def multirequest_run(self, tasks, suite):
Xinan Lin1e8e7912020-07-31 09:52:16 -070060 """Call cros_test_platform Builder to schedule a batch of suite tests.
Xinan Lin3ba18a02019-08-13 15:44:55 -070061
62 Args:
Xinan Lin9e4917d2019-11-04 10:58:47 -080063 tasks: The suite tasks to run.
64 suite: suite name for the batch request.
65
66 Returns:
67 List of executed tasks.
Xinan Lin3ba18a02019-08-13 15:44:55 -070068
69 Raises:
70 BuildbucketRunError: if failed to get build info from task parameters.
71 """
Xinan Lin9e4917d2019-11-04 10:58:47 -080072 requests = []
73 executed_tasks = []
74 counter = collections.defaultdict(int)
Xinan Lindf0698a2020-02-05 22:38:11 -080075 task_executions = []
Xinan Lin9e4917d2019-11-04 10:58:47 -080076 for task in tasks:
77 try:
78 params = task.extract_params()
Xinan Linc54a7462020-04-17 15:39:01 -070079 if _should_skip(params):
80 continue
Xinan Lin8bb5b4b2020-07-21 23:17:55 -070081 req = _form_test_platform_request(params)
Xinan Lin9e4917d2019-11-04 10:58:47 -080082 req_json = json_format.MessageToJson(req)
83 counter_key = params['board']
Prathmesh Prabhuaf0857f2020-06-20 00:16:32 -070084 counter_key += '' if params['model'] in [None, 'None'
85 ] else ('_' + params['model'])
Xinan Lin9e4917d2019-11-04 10:58:47 -080086 counter[counter_key] += 1
87 req_name = counter_key
88 if counter[counter_key] > 1:
89 req_name += '_' + str(counter[counter_key] - 1)
90 requests.append('"%s": %s' % (req_name, req_json))
91 executed_tasks.append(task)
Xinan Lin083ba8f2020-02-06 13:55:18 -080092 if params.get('task_id'):
Xinan Lin9b17c5b2020-08-06 10:43:30 -070093 task_executions.append(
94 analytics.ExecutionTask(params['task_id'], req_name))
Xinan Lin9e4917d2019-11-04 10:58:47 -080095 except (ValueError, BuildbucketRunError):
Xinan Linba3b9322020-04-24 15:08:12 -070096 logging.error('Failed to process task: %r', params)
Xinan Lin9e4917d2019-11-04 10:58:47 -080097 if not requests:
98 return []
99 try:
100 requests_json = '{ "requests": { %s } }' % ', '.join(requests)
Dhanya Ganeshedb78cb2020-07-20 19:40:50 +0000101 req_build = self._build_request(requests_json, _bb_tags(suite))
Xinan Lin9e4917d2019-11-04 10:58:47 -0800102 logging.debug('Raw request to buildbucket: %r', req_build)
linxinane5eb4552019-08-26 05:44:45 +0000103
Prathmesh Prabhuaf0857f2020-06-20 00:16:32 -0700104 if (self.running_env == constants.RunningEnv.ENV_STANDALONE
105 or self.running_env == constants.RunningEnv.ENV_DEVELOPMENT_SERVER):
Xinan Lin9e4917d2019-11-04 10:58:47 -0800106 # If running locally, use the staging service account.
107 sa_key = self._gen_service_account_key(
108 file_getter.STAGING_CLIENT_SECRETS_FILE)
109 cred = prpc_client.service_account_credentials(
110 service_account_key=sa_key)
111 else:
112 cred = prpc_client.service_account_credentials()
Prathmesh Prabhuaf0857f2020-06-20 00:16:32 -0700113 resp = self.client.ScheduleBuild(req_build,
114 credentials=cred,
115 timeout=PRPC_TIMEOUT_SEC)
Xinan Lin9e4917d2019-11-04 10:58:47 -0800116 logging.debug('Response from buildbucket: %r', resp)
Xinan Lindf0698a2020-02-05 22:38:11 -0800117 for t in task_executions:
118 t.update_result(resp)
119 try:
120 if not t.upload():
121 logging.warning('Failed to insert row: %r', t)
122 # For any exceptions from BQ, only log it.
Prathmesh Prabhuaf0857f2020-06-20 00:16:32 -0700123 except Exception as e: #pylint: disable=broad-except
124 logging.exception('Failed to insert row: %r, got error: %s', t,
125 str(e))
Xinan Lin9e4917d2019-11-04 10:58:47 -0800126 return executed_tasks
127 except Exception as e:
128 logging.debug('Failed to process tasks: %r', tasks)
129 logging.exception(str(e))
130 return []
Xinan Lin3ba18a02019-08-13 15:44:55 -0700131
Xinan Linc61196b2019-08-13 10:37:30 -0700132 def dummy_run(self):
133 """Perform a dummy run of prpc call to cros_test_platform-dev."""
134
Xinan Lin9e4917d2019-11-04 10:58:47 -0800135 requests_json = '{ "requests": { "dummy": {} } }'
136 req_build = self._build_request(requests_json, tags=None)
Xinan Linc61196b2019-08-13 10:37:30 -0700137 # Use the staging service account to authorize the request.
138 sa_key = self._gen_service_account_key(
139 file_getter.STAGING_CLIENT_SECRETS_FILE)
140 cred = prpc_client.service_account_credentials(service_account_key=sa_key)
Prathmesh Prabhuaf0857f2020-06-20 00:16:32 -0700141 return self.client.ScheduleBuild(req_build,
142 credentials=cred,
143 timeout=PRPC_TIMEOUT_SEC)
Xinan Linc61196b2019-08-13 10:37:30 -0700144
Xinan Lin9e4917d2019-11-04 10:58:47 -0800145 def _build_request(self, reqs_json, tags):
Xinan Lin3ba18a02019-08-13 15:44:55 -0700146 """Generate ScheduleBuildRequest for calling buildbucket.
Xinan Linc61196b2019-08-13 10:37:30 -0700147
Xinan Lin3ba18a02019-08-13 15:44:55 -0700148 Args:
Xinan Lin9e4917d2019-11-04 10:58:47 -0800149 reqs_json: A json string of requests.
Prathmesh Prabhu2382a182019-09-07 21:18:10 -0700150 tags: A list of tags for the buildbucket build.
Xinan Linc61196b2019-08-13 10:37:30 -0700151
Xinan Lin3ba18a02019-08-13 15:44:55 -0700152 Returns:
153 A ScheduleBuildRequest instance.
154 """
Xinan Lin9e4917d2019-11-04 10:58:47 -0800155 requests_struct = struct_pb2.Struct()
156 recipe_struct = json_format.Parse(reqs_json, requests_struct)
Sean McAllisteraf32bfb2021-07-19 09:20:17 -0600157 return builds_service_pb2.ScheduleBuildRequest(builder=self.builder,
158 properties=recipe_struct,
159 request_id=str(
160 uuid.uuid1()),
161 tags=tags)
Xinan Linc61196b2019-08-13 10:37:30 -0700162
163 def _gen_service_account_key(self, sa):
Xinan Lin3ba18a02019-08-13 15:44:55 -0700164 """Generate credentials to authorize the call.
Xinan Linc61196b2019-08-13 10:37:30 -0700165
Xinan Lin3ba18a02019-08-13 15:44:55 -0700166 Args:
167 sa: A string of the path to the service account json file.
Xinan Linc61196b2019-08-13 10:37:30 -0700168
Xinan Lin3ba18a02019-08-13 15:44:55 -0700169 Returns:
170 A service account key.
171 """
Xinan Linc61196b2019-08-13 10:37:30 -0700172 service_credentials = service_account.ServiceAccountCredentials
Xinan Lin3ba18a02019-08-13 15:44:55 -0700173 key = service_credentials.from_json_keyfile_name(sa, self.scope)
Prathmesh Prabhuaf0857f2020-06-20 00:16:32 -0700174 return auth.ServiceAccountKey(client_email=key.service_account_email,
175 private_key=key._private_key_pkcs8_pem,
176 private_key_id=key._private_key_id)
Prathmesh Prabhu1918a8f2019-09-07 21:37:37 -0700177
178
Prathmesh Prabhu06852fe2019-09-09 07:58:43 -0700179def _form_test_platform_request(task_params):
Jacob Kopczynski408f3fd2020-08-07 13:31:59 -0700180 """Generates test_platform.Request proto to send to buildbucket.
Prathmesh Prabhu06852fe2019-09-09 07:58:43 -0700181
182 Args:
Jacob Kopczynski408f3fd2020-08-07 13:31:59 -0700183 task_params: dict containing the parameters of a task from the suite queue.
Prathmesh Prabhu06852fe2019-09-09 07:58:43 -0700184
185 Returns:
Jacob Kopczynski408f3fd2020-08-07 13:31:59 -0700186 A ctp_request.Request instance.
Prathmesh Prabhu06852fe2019-09-09 07:58:43 -0700187 """
188 build = _infer_build_from_task_params(task_params)
189 if build == 'None':
190 raise BuildbucketRunError('No proper build in task params: %r' %
191 task_params)
192 pool = _infer_pool_from_task_params(task_params)
193 timeout = _infer_timeout_from_task_params(task_params)
194
Jacob Kopczynski408f3fd2020-08-07 13:31:59 -0700195 request = ctp_request.Request()
Prathmesh Prabhu06852fe2019-09-09 07:58:43 -0700196 params = request.params
197
Xinan Lin4757d6f2020-03-24 22:20:31 -0700198 params.scheduling.CopyFrom(_scheduling_for_pool(pool))
Prathmesh Prabhu46156ff2020-06-20 00:18:52 -0700199 if task_params.get('qs_account') not in ['None', None]:
Xinan Lin4757d6f2020-03-24 22:20:31 -0700200 params.scheduling.qs_account = task_params.get('qs_account')
201 # Quota Scheduler has no concept of priority.
Xinan Lin8bb5b4b2020-07-21 23:17:55 -0700202 if (task_params.get('priority') not in ['None', None]
203 and not params.scheduling.qs_account):
Prathmesh Prabhu06852fe2019-09-09 07:58:43 -0700204 params.scheduling.priority = int(task_params['priority'])
205
206 params.software_dependencies.add().chromeos_build = build
207 params.software_attributes.build_target.name = task_params['board']
Aviv Keshetc679faf2019-11-27 17:52:50 -0800208
209 gs_url = GS_PREFIX + task_params['test_source_build']
210 params.metadata.test_metadata_url = gs_url
211 params.metadata.debug_symbols_archive_url = gs_url
212
Prathmesh Prabhu06852fe2019-09-09 07:58:43 -0700213 params.time.maximum_duration.FromTimedelta(timeout)
214
215 for key, value in _request_tags(task_params, build, pool).iteritems():
216 params.decorations.tags.append('%s:%s' % (key, value))
217
Xinan Lin7bf266a2020-06-10 23:54:26 -0700218 for d in _infer_user_defined_dimensions(task_params):
219 params.freeform_attributes.swarming_dimensions.append(d)
Xinan Linba3b9322020-04-24 15:08:12 -0700220
Prathmesh Prabhu06852fe2019-09-09 07:58:43 -0700221 if task_params['model'] != 'None':
222 params.hardware_attributes.model = task_params['model']
223
224 if task_params['job_retry'] == 'True':
225 params.retry.allow = True
226 params.retry.max = constants.Buildbucket.MAX_RETRY
227
228 fw_rw_build = task_params.get(build_lib.BuildVersionKey.FW_RW_VERSION)
229 fw_ro_build = task_params.get(build_lib.BuildVersionKey.FW_RO_VERSION)
Brigit Rossbachbb080912020-11-18 13:52:17 -0700230
231 if task_params.has_key('firmware_ro_version'):
232 build = params.software_dependencies.add()
233 build.ro_firmware_build = task_params['firmware_ro_version']
234
Prathmesh Prabhu06852fe2019-09-09 07:58:43 -0700235 # Skip firmware field if None(unspecified) or 'None'(no such build).
236 if fw_ro_build not in (None, 'None'):
237 build = params.software_dependencies.add()
238 build.ro_firmware_build = fw_ro_build
Brigit Rossbachbb080912020-11-18 13:52:17 -0700239
240 if task_params.has_key('firmware_rw_version'):
241 build = params.software_dependencies.add()
242 build.rw_firmware_build = task_params['firmware_rw_version']
243
Prathmesh Prabhu06852fe2019-09-09 07:58:43 -0700244 if fw_rw_build not in (None, 'None'):
245 build = params.software_dependencies.add()
246 build.rw_firmware_build = fw_rw_build
247
Garry Wang2104bf52021-07-19 18:13:12 -0700248 secondary_targets = task_params.get('secondary_targets')
Garry Wang111a26f2021-07-23 15:25:14 -0700249 secondary_targets = restruct_secondary_targets_from_string(secondary_targets)
Garry Wang2104bf52021-07-19 18:13:12 -0700250 if secondary_targets:
251 for s_target in secondary_targets:
252 s_device = params.secondary_devices.add()
253 s_device.software_attributes.build_target.name = s_target.board
254 if s_target.model:
255 s_device.hardware_attributes.model = s_target.model
256 if s_target.cros_build:
257 s_build = s_device.software_dependencies.add()
258 s_build.chromeos_build = s_target.cros_build
259
Prathmesh Prabhu06852fe2019-09-09 07:58:43 -0700260 request.test_plan.suite.add().name = task_params['suite']
261 return request
262
Sean McAllisteraf32bfb2021-07-19 09:20:17 -0600263
Prathmesh Prabhu1918a8f2019-09-07 21:37:37 -0700264def _scheduling_for_pool(pool):
265 """Assign appropriate pool name to scheduling instance.
266
267 Args:
Xinan Lin1516edb2020-07-05 23:13:54 -0700268 pool: string pool name (e.g. 'MANAGED_POOL_QUOTA', 'wificell').
Prathmesh Prabhu1918a8f2019-09-07 21:37:37 -0700269
270 Returns:
Jacob Kopczynski408f3fd2020-08-07 13:31:59 -0700271 scheduling: A ctp_request.Request.Params.Scheduling instance.
Prathmesh Prabhu1918a8f2019-09-07 21:37:37 -0700272 """
Jacob Kopczynski408f3fd2020-08-07 13:31:59 -0700273 mp = ctp_request.Request.Params.Scheduling.ManagedPool
Prathmesh Prabhu1918a8f2019-09-07 21:37:37 -0700274 if mp.DESCRIPTOR.values_by_name.get(pool) is not None:
Jacob Kopczynski408f3fd2020-08-07 13:31:59 -0700275 return ctp_request.Request.Params.Scheduling(managed_pool=mp.Value(pool))
276 return ctp_request.Request.Params.Scheduling(unmanaged_pool=pool)
Prathmesh Prabhu1918a8f2019-09-07 21:37:37 -0700277
278
Prathmesh Prabhu06852fe2019-09-09 07:58:43 -0700279# Also see: A copy of this function in swarming_lib.py.
280def _infer_build_from_task_params(task_params):
281 """Infer the build to install on the DUT for the scheduled task.
Prathmesh Prabhu1918a8f2019-09-07 21:37:37 -0700282
Prathmesh Prabhu06852fe2019-09-09 07:58:43 -0700283 Args:
284 task_params: The parameters of a task loaded from suite queue.
Prathmesh Prabhu1918a8f2019-09-07 21:37:37 -0700285
Prathmesh Prabhu06852fe2019-09-09 07:58:43 -0700286 Returns:
287 A string representing the build to run.
288 """
289 cros_build = task_params[build_lib.BuildVersionKey.CROS_VERSION]
290 android_build = task_params[build_lib.BuildVersionKey.ANDROID_BUILD_VERSION]
291 testbed_build = task_params[build_lib.BuildVersionKey.TESTBED_BUILD_VERSION]
292 return cros_build or android_build or testbed_build
293
294
295def _infer_pool_from_task_params(task_params):
296 """Infer the pool to use for the scheduled task.
297
298 Args:
299 task_params: The parameters of a task loaded from suite queue.
300
301 Returns:
302 A string pool to schedule task in.
303 """
304 if task_params.get('override_qs_account'):
305 return 'DUT_POOL_QUOTA'
306 return task_params.get('override_pool') or task_params['pool']
307
308
309def _infer_timeout_from_task_params(task_params):
310 """Infer the timeout for the scheduled task.
311
312 Args:
313 task_params: The parameters of a task loaded from suite queue.
314
315 Returns:
316 A datetime.timedelta instance for the timeout.
317 """
318 timeout_mins = int(task_params['timeout_mins'])
319 # timeout's unit is hour.
320 if task_params.get('timeout'):
Sean Abrahamec0d0762020-09-18 17:19:05 +0000321 timeout_mins = max(int(task_params['timeout']) * 60, timeout_mins)
Prathmesh Prabhu06852fe2019-09-09 07:58:43 -0700322 if timeout_mins > constants.Buildbucket.MAX_BUILDBUCKET_TIMEOUT_MINS:
323 timeout_mins = constants.Buildbucket.MAX_BUILDBUCKET_TIMEOUT_MINS
324 return datetime.timedelta(minutes=timeout_mins)
325
Prathmesh Prabhu1918a8f2019-09-07 21:37:37 -0700326
Xinan Linba3b9322020-04-24 15:08:12 -0700327def _infer_user_defined_dimensions(task_params):
328 """Infer the dimensions defined by users.
329
330 Args:
331 task_params: The parameters of a task loaded from suite queue.
332
333 Returns:
334 A list of strings; an empty list if no dimensions set.
335
336 Raises:
337 ValueError: if dimension is not valid.
338 """
339 result = []
340 if task_params.get('dimensions') in (None, 'None'):
341 return result
Patrick Meiringd4f60772021-03-04 12:09:28 +1100342 dims = [d.lstrip() for d in task_params.get('dimensions').split(',')]
343 for d in dims:
Xinan Linba3b9322020-04-24 15:08:12 -0700344 if len(d.split(':')) != 2:
345 raise ValueError(
346 'Job %s has invalid dimensions: %s' %
347 (task_params.get('name'), task_params.get('dimensions')))
348 result.append(d)
349 return result
350
351
Prathmesh Prabhu2382a182019-09-07 21:18:10 -0700352def _request_tags(task_params, build, pool):
353 """Infer tags to include in cros_test_platform request.
354
355 Args:
356 task_params: suite task parameters.
357 build: The build included in the request. Must not be None.
358 pool: The DUT pool used for the request. Must not be None.
359
360 Returns:
361 A dict of tags.
362 """
363 tags = {
Prathmesh Prabhuaf0857f2020-06-20 00:16:32 -0700364 'build': build,
365 'label-pool': pool,
Taylor Clarke12ec9a2021-02-18 22:22:19 +0000366 'ctp-fwd-task-name': task_params.get('name')
Prathmesh Prabhu2382a182019-09-07 21:18:10 -0700367 }
Xinan Linfb63d572019-09-24 15:49:04 -0700368 if task_params.get('board') not in (None, 'None'):
Prathmesh Prabhu2382a182019-09-07 21:18:10 -0700369 tags['label-board'] = task_params['board']
Xinan Linfb63d572019-09-24 15:49:04 -0700370 if task_params.get('model') not in (None, 'None'):
Prathmesh Prabhu2382a182019-09-07 21:18:10 -0700371 tags['label-model'] = task_params['model']
Xinan Linfb63d572019-09-24 15:49:04 -0700372 if task_params.get('suite') not in (None, 'None'):
Prathmesh Prabhu2382a182019-09-07 21:18:10 -0700373 tags['suite'] = task_params['suite']
Taylor Clarke12ec9a2021-02-18 22:22:19 +0000374 if task_params.get('analytics_name') not in (None, 'None'):
375 tags['analytics_name'] = task_params['analytics_name']
Prathmesh Prabhu2382a182019-09-07 21:18:10 -0700376 return tags
377
378
Xinan Linc8647112020-02-04 16:45:56 -0800379def _get_key_val_from_label(label):
380 """A helper to get key and value from the label.
381
382 Args:
383 label: A string of label, should be in the form of
384 key:value, e.g. 'pool:ChromeOSSkylab'.
385 """
386 res = label.split(':')
387 if len(res) == 2:
388 return res[0], res[1]
389 logging.warning('Failed to parse the label, %s', label)
390
391
Dhanya Ganeshedb78cb2020-07-20 19:40:50 +0000392def _bb_tags(suite):
393 """Get all the tags required for Buildbucket.
Prathmesh Prabhu2382a182019-09-07 21:18:10 -0700394
395 Args:
Dhanya Ganeshedb78cb2020-07-20 19:40:50 +0000396 suite: A string of suite name being scheduled.
Prathmesh Prabhu2382a182019-09-07 21:18:10 -0700397
398 Returns:
399 [bb_common_pb2.StringPair] tags to include the buildbucket request.
400 """
Sean McAllisteraf32bfb2021-07-19 09:20:17 -0600401 return [
402 bb_common_pb2.StringPair(key='suite', value=suite),
403 bb_common_pb2.StringPair(key='user_agent', value='suite_scheduler')
404 ]
Xinan Linc54a7462020-04-17 15:39:01 -0700405
406
407def _should_skip(params):
408 """Decide whether to skip a task based on env and pool.
409
410 Suite request from staging may still have a small chance to run
411 in production. However, for unmanaged pools(e.g. wificell), which
412 usually are small, dev traffic is unacceptable.
413
414 Args:
415 params: dict containing the parameters of a task got from suite
416 queue.
417
418 Returns:
419 A boolean; true for suite targetting non-default pools from staging
420 env.
421 """
422 if constants.application_id() == constants.AppID.PROD_APP:
423 return False
Xinan Lin0d7910d2020-07-21 11:06:45 -0700424 return params['pool'] != 'MANAGED_POOL_QUOTA'