Aviv Keshet | c679faf | 2019-11-27 17:52:50 -0800 | [diff] [blame] | 1 | #!/usr/bin/env python2 |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 2 | # Copyright 2019 The Chromium OS Authors. All rights reserved. |
| 3 | # Use of this source code is governed by a BSD-style license that can be |
| 4 | # found in the LICENSE file. |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 5 | """Module for buildbucket unittests.""" |
| 6 | |
Xinan Lin | df0698a | 2020-02-05 22:38:11 -0800 | [diff] [blame] | 7 | import json |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 8 | import mock |
| 9 | import os |
| 10 | import unittest |
| 11 | |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 12 | import constants |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 13 | import build_lib |
| 14 | import buildbucket |
| 15 | import task_executor |
| 16 | |
Jacob Kopczynski | cefd50c | 2020-07-31 10:05:10 -0700 | [diff] [blame] | 17 | from chromite.api.gen.test_platform import request_pb2 as ctp_request |
Xinan Lin | df0698a | 2020-02-05 22:38:11 -0800 | [diff] [blame] | 18 | from chromite.api.gen.test_platform.suite_scheduler import analytics_pb2 |
Sean McAllister | af32bfb | 2021-07-19 09:20:17 -0600 | [diff] [blame] | 19 | from chromite.third_party.infra_libs.buildbucket.proto import build_pb2 |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 20 | from google.appengine.api import taskqueue |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 21 | from google.appengine.ext import testbed |
Sean McAllister | 66bf7e9 | 2021-07-16 18:46:04 +0000 | [diff] [blame] | 22 | from google.protobuf import json_format |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 23 | |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 24 | ADDR = u'http://localhost:1' |
Xinan Lin | df0698a | 2020-02-05 22:38:11 -0800 | [diff] [blame] | 25 | FAKE_UUID = 'c78e0bf3-4142-11ea-bc66-88e9fe4c5349' |
| 26 | FAKE_BUILD_ID = 8890493019851395280 |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 27 | |
| 28 | |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 29 | def _get_suite_params(board='fake_board', |
| 30 | model='fake_model', |
| 31 | suite='fake_suite', |
Xinan Lin | 1516edb | 2020-07-05 23:13:54 -0700 | [diff] [blame] | 32 | pool='MANAGED_POOL_QUOTA', |
Taylor Clark | e12ec9a | 2021-02-18 22:22:19 +0000 | [diff] [blame] | 33 | cros_build='fake_cros_build', |
| 34 | analytics_name=None): |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 35 | return { |
Prathmesh Prabhu | 2382a18 | 2019-09-07 21:18:10 -0700 | [diff] [blame] | 36 | 'suite': suite, |
| 37 | 'board': board, |
| 38 | 'model': model, |
| 39 | build_lib.BuildVersionKey.CROS_VERSION: cros_build, |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 40 | build_lib.BuildVersionKey.FW_RW_VERSION: 'fake_firmware_rw_build', |
| 41 | build_lib.BuildVersionKey.FW_RO_VERSION: 'fake_firmware_ro_build', |
| 42 | build_lib.BuildVersionKey.ANDROID_BUILD_VERSION: 'fake_android_build', |
| 43 | build_lib.BuildVersionKey.TESTBED_BUILD_VERSION: 'fake_testbed_build', |
Brigit Rossbach | bb08091 | 2020-11-18 13:52:17 -0700 | [diff] [blame] | 44 | 'firmware_ro_version': 'fake_firmware_ro_build', |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 45 | 'num': 1, |
Prathmesh Prabhu | 2382a18 | 2019-09-07 21:18:10 -0700 | [diff] [blame] | 46 | 'pool': pool, |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 47 | 'priority': 10, |
Xinan Lin | 6e09738 | 2019-08-27 18:43:35 -0700 | [diff] [blame] | 48 | 'timeout': 12, |
| 49 | 'timeout_mins': 4320, |
| 50 | 'max_runtime_mins': 4320, |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 51 | 'no_wait_for_results': True, |
| 52 | 'test_source_build': 'fake_test_source_build', |
| 53 | 'job_retry': False, |
| 54 | 'no_delay': False, |
| 55 | 'force': False, |
| 56 | 'run_prod_code': True, |
| 57 | 'is_skylab': False, |
Taylor Clark | e12ec9a | 2021-02-18 22:22:19 +0000 | [diff] [blame] | 58 | 'analytics_name': analytics_name, |
Azizur Rahman | b720e51 | 2022-05-13 18:30:40 +0000 | [diff] [blame] | 59 | 'secondary_targets': '', |
Azizur Rahman | a9a189f | 2022-07-29 20:28:28 +0000 | [diff] [blame] | 60 | 'run_via_cft': True, |
| 61 | 'include_tags': 'a, b, c', |
| 62 | 'exclude_tags': 'e, f, g', |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 63 | } |
| 64 | |
| 65 | |
| 66 | class FakeTestPlatformClient(object): |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 67 | def __init__(self, test): |
| 68 | self._test = test |
| 69 | self.called_with_requests = [] |
Xinan Lin | df0698a | 2020-02-05 22:38:11 -0800 | [diff] [blame] | 70 | self.error = None |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 71 | |
Xinan Lin | 57e2d96 | 2020-03-30 17:24:53 -0700 | [diff] [blame] | 72 | def ScheduleBuild(self, req, credentials=None, timeout=10): |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 73 | self.called_with_requests.append(req) |
Xinan Lin | df0698a | 2020-02-05 22:38:11 -0800 | [diff] [blame] | 74 | if self.error: |
| 75 | return self.error |
| 76 | return build_pb2.Build(id=FAKE_BUILD_ID) |
| 77 | |
| 78 | |
| 79 | class FakeBigqueryRestClient(object): |
Xinan Lin | df0698a | 2020-02-05 22:38:11 -0800 | [diff] [blame] | 80 | def __init__(self, rest_client, project=None, dataset=None, table=None): |
| 81 | """Initialize the mock class.""" |
| 82 | self.table = table |
| 83 | self.rows = [] |
| 84 | |
| 85 | def insert(self, rows): |
| 86 | self.rows = rows |
| 87 | return True |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 88 | |
| 89 | |
| 90 | class TestPlatformClientTestCase(unittest.TestCase): |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 91 | def setUp(self): |
| 92 | super(TestPlatformClientTestCase, self).setUp() |
| 93 | self.fake_client = FakeTestPlatformClient(self) |
| 94 | patcher = mock.patch('buildbucket._get_client', |
| 95 | return_value=self._get_client(ADDR)) |
| 96 | patcher.start() |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 97 | self.client = buildbucket.TestPlatformClient(ADDR, 'foo-proj', |
| 98 | 'foo-bucket', 'foo-builder') |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 99 | self.addCleanup(patcher.stop) |
| 100 | self.testbed = testbed.Testbed() |
| 101 | self.testbed.activate() |
| 102 | self.addCleanup(self.testbed.deactivate) |
| 103 | self.testbed.init_taskqueue_stub( |
| 104 | root_path=os.path.join(os.path.dirname(__file__))) |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 105 | self.taskqueue_stub = self.testbed.get_stub(testbed.TASKQUEUE_SERVICE_NAME) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 106 | self.queue = taskqueue.Queue(task_executor.SUITES_QUEUE) |
Xinan Lin | c54a746 | 2020-04-17 15:39:01 -0700 | [diff] [blame] | 107 | _mock_application_id = mock.patch('constants.application_id') |
| 108 | self.mock_application_id = _mock_application_id.start() |
| 109 | self.addCleanup(_mock_application_id.stop) |
| 110 | self.mock_application_id.return_value = constants.AppID.PROD_APP |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 111 | |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 112 | def testFormTestPlatformMultiRequestSuccessfully(self): |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 113 | suite_kwargs = [_get_suite_params(board=b) for b in ['foo', 'goo']] |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 114 | for suite in suite_kwargs: |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 115 | task_executor.push(task_executor.SUITES_QUEUE, tag='fake_suite', **suite) |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 116 | |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 117 | tasks = self.queue.lease_tasks_by_tag( |
| 118 | 3600, constants.Buildbucket.MULTIREQUEST_SIZE, deadline=60) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 119 | |
| 120 | self.client.multirequest_run(tasks, 'fake_suite') |
Prathmesh Prabhu | 9a8060a | 2019-08-30 14:13:23 -0700 | [diff] [blame] | 121 | self._assert_bb_client_called() |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 122 | |
| 123 | request_map = self._extract_request_map_from_bb_client() |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 124 | request_list = [ |
Jacob Kopczynski | 408f3fd | 2020-08-07 13:31:59 -0700 | [diff] [blame] | 125 | _struct_to_ctp_request(v) for v in request_map.values() |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 126 | ] |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 127 | request_list.sort( |
| 128 | key=lambda req: req.params.software_attributes.build_target.name) |
| 129 | for i, req in enumerate(request_list): |
| 130 | self.assertEqual(req.params.scheduling.priority, 10) |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 131 | self.assertEqual( |
| 132 | req.params.scheduling.managed_pool, |
Jacob Kopczynski | 408f3fd | 2020-08-07 13:31:59 -0700 | [diff] [blame] | 133 | ctp_request.Request.Params.Scheduling.MANAGED_POOL_QUOTA) |
Prathmesh Prabhu | 9adfa63 | 2020-02-21 10:39:24 -0800 | [diff] [blame] | 134 | # Don't check for exact max timeout to stay DRY. |
| 135 | # But do check that the maximum is sane. |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 136 | self.assertLess(req.params.time.maximum_duration.seconds, |
| 137 | 2 * 24 * 60 * 60) |
Aviv Keshet | c679faf | 2019-11-27 17:52:50 -0800 | [diff] [blame] | 138 | gs_url = ('gs://chromeos-image-archive/%s' % |
| 139 | suite_kwargs[i]['test_source_build']) |
| 140 | self.assertEqual(req.params.metadata.test_metadata_url, gs_url) |
| 141 | self.assertEqual(req.params.metadata.debug_symbols_archive_url, gs_url) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 142 | self.assertEqual(req.test_plan.suite[0].name, suite_kwargs[i]['suite']) |
| 143 | self.assertEqual(req.params.software_attributes.build_target.name, |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 144 | suite_kwargs[i]['board']) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 145 | |
| 146 | def testPoolName(self): |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 147 | suite_kwargs = [_get_suite_params(board=b) for b in ['foo', 'goo', 'hoo']] |
Xinan Lin | 1516edb | 2020-07-05 23:13:54 -0700 | [diff] [blame] | 148 | suite_kwargs[0]['pool'] = 'MANAGED_POOL_CTS' |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 149 | suite_kwargs[1]['pool'] = 'wifi' |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 150 | for suite in suite_kwargs: |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 151 | task_executor.push(task_executor.SUITES_QUEUE, tag='some_suite', **suite) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 152 | |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 153 | tasks = self.queue.lease_tasks_by_tag( |
| 154 | 3600, constants.Buildbucket.MULTIREQUEST_SIZE, deadline=60) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 155 | self.client.multirequest_run(tasks, 'some_suite') |
| 156 | self._assert_bb_client_called() |
| 157 | request_map = self._extract_request_map_from_bb_client() |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 158 | request_list = [ |
Jacob Kopczynski | 408f3fd | 2020-08-07 13:31:59 -0700 | [diff] [blame] | 159 | _struct_to_ctp_request(v) for v in request_map.values() |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 160 | ] |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 161 | self.assertEqual(len(request_list), 3) |
| 162 | request_list.sort( |
| 163 | key=lambda req: req.params.software_attributes.build_target.name) |
| 164 | self.assertEqual(request_list[0].params.scheduling.managed_pool, |
Jacob Kopczynski | 408f3fd | 2020-08-07 13:31:59 -0700 | [diff] [blame] | 165 | ctp_request.Request.Params.Scheduling.MANAGED_POOL_CTS) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 166 | self.assertEqual(request_list[1].params.scheduling.unmanaged_pool, |
| 167 | suite_kwargs[1]['pool']) |
Prathmesh Prabhu | 88409f6 | 2019-08-30 14:32:28 -0700 | [diff] [blame] | 168 | |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 169 | def testNoFinalBuild(self): |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 170 | suite_kwargs = _get_suite_params() |
| 171 | suite_kwargs[build_lib.BuildVersionKey.CROS_VERSION] = None |
| 172 | suite_kwargs[build_lib.BuildVersionKey.ANDROID_BUILD_VERSION] = None |
| 173 | suite_kwargs[build_lib.BuildVersionKey.TESTBED_BUILD_VERSION] = None |
| 174 | |
| 175 | task_executor.push(task_executor.SUITES_QUEUE, |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 176 | tag='fake_suite', |
| 177 | **suite_kwargs) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 178 | |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 179 | tasks = self.queue.lease_tasks_by_tag( |
| 180 | 3600, constants.Buildbucket.MULTIREQUEST_SIZE, deadline=60) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 181 | executed = self.client.multirequest_run(tasks, 'fake_suite') |
| 182 | self.assertEqual(len(executed), 0) |
Prathmesh Prabhu | 9a8060a | 2019-08-30 14:13:23 -0700 | [diff] [blame] | 183 | self._assert_bb_client_not_called() |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 184 | |
Prathmesh Prabhu | 46156ff | 2020-06-20 00:18:52 -0700 | [diff] [blame] | 185 | def testShouldSetQsAccountForUnmanagedPool(self): |
Xinan Lin | 4757d6f | 2020-03-24 22:20:31 -0700 | [diff] [blame] | 186 | suite_kwargs = _get_suite_params(pool='foo') |
| 187 | suite_kwargs['priority'] = '30' |
| 188 | suite_kwargs['qs_account'] = 'qs-foo' |
| 189 | |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 190 | task_executor.push(task_executor.SUITES_QUEUE, |
| 191 | tag='fake_suite', |
| 192 | **suite_kwargs) |
Xinan Lin | 4757d6f | 2020-03-24 22:20:31 -0700 | [diff] [blame] | 193 | |
| 194 | tasks = self.queue.lease_tasks_by_tag( |
| 195 | 3600, constants.Buildbucket.MULTIREQUEST_SIZE, deadline=60) |
| 196 | self.client.multirequest_run(tasks, 'fake_suite') |
Xinan Lin | 4757d6f | 2020-03-24 22:20:31 -0700 | [diff] [blame] | 197 | self._assert_bb_client_called() |
Jacob Kopczynski | f464c8f | 2020-08-20 17:54:35 +0000 | [diff] [blame] | 198 | request_map = self._extract_request_map_from_bb_client() |
Xinan Lin | 4757d6f | 2020-03-24 22:20:31 -0700 | [diff] [blame] | 199 | self.assertEqual(len(request_map), 1) |
Jacob Kopczynski | 408f3fd | 2020-08-07 13:31:59 -0700 | [diff] [blame] | 200 | req = _struct_to_ctp_request(request_map['fake_board_fake_model']) |
Xinan Lin | 4757d6f | 2020-03-24 22:20:31 -0700 | [diff] [blame] | 201 | self.assertEqual(req.params.scheduling.qs_account, 'qs-foo') |
| 202 | # If qs_account is set, priority should be 0, the default value |
| 203 | # defined by proto3. |
| 204 | self.assertEqual(req.params.scheduling.priority, 0) |
| 205 | |
Prathmesh Prabhu | 46156ff | 2020-06-20 00:18:52 -0700 | [diff] [blame] | 206 | def testShouldSetQsAccountForManagedPool(self): |
Xinan Lin | 1516edb | 2020-07-05 23:13:54 -0700 | [diff] [blame] | 207 | suite_kwargs = _get_suite_params(pool='MANAGED_POOL_QUOTA') |
Xinan Lin | 4757d6f | 2020-03-24 22:20:31 -0700 | [diff] [blame] | 208 | suite_kwargs['qs_account'] = 'qs-foo' |
| 209 | |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 210 | task_executor.push(task_executor.SUITES_QUEUE, |
| 211 | tag='fake_suite', |
| 212 | **suite_kwargs) |
Xinan Lin | 4757d6f | 2020-03-24 22:20:31 -0700 | [diff] [blame] | 213 | |
| 214 | tasks = self.queue.lease_tasks_by_tag( |
| 215 | 3600, constants.Buildbucket.MULTIREQUEST_SIZE, deadline=60) |
| 216 | self.client.multirequest_run(tasks, 'fake_suite') |
Xinan Lin | 4757d6f | 2020-03-24 22:20:31 -0700 | [diff] [blame] | 217 | self._assert_bb_client_called() |
Jacob Kopczynski | f464c8f | 2020-08-20 17:54:35 +0000 | [diff] [blame] | 218 | request_map = self._extract_request_map_from_bb_client() |
Xinan Lin | 4757d6f | 2020-03-24 22:20:31 -0700 | [diff] [blame] | 219 | self.assertEqual(len(request_map), 1) |
Jacob Kopczynski | 408f3fd | 2020-08-07 13:31:59 -0700 | [diff] [blame] | 220 | req = _struct_to_ctp_request(request_map['fake_board_fake_model']) |
Prathmesh Prabhu | 46156ff | 2020-06-20 00:18:52 -0700 | [diff] [blame] | 221 | self.assertEqual(req.params.scheduling.qs_account, 'qs-foo') |
| 222 | # If qs_account is set, priority should be 0, the default value |
| 223 | # defined by proto3. |
| 224 | self.assertEqual(req.params.scheduling.priority, 0) |
Xinan Lin | 4757d6f | 2020-03-24 22:20:31 -0700 | [diff] [blame] | 225 | |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 226 | def testShouldSetPriority(self): |
| 227 | suite_kwargs = _get_suite_params() |
| 228 | suite_kwargs['priority'] = '30' |
Xinan Lin | fb63d57 | 2019-09-24 15:49:04 -0700 | [diff] [blame] | 229 | |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 230 | task_executor.push(task_executor.SUITES_QUEUE, |
| 231 | tag='fake_suite', |
| 232 | **suite_kwargs) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 233 | |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 234 | tasks = self.queue.lease_tasks_by_tag( |
| 235 | 3600, constants.Buildbucket.MULTIREQUEST_SIZE, deadline=60) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 236 | self.client.multirequest_run(tasks, 'fake_suite') |
| 237 | request_map = self._extract_request_map_from_bb_client() |
Xinan Lin | fb63d57 | 2019-09-24 15:49:04 -0700 | [diff] [blame] | 238 | self._assert_bb_client_called() |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 239 | self.assertEqual(len(request_map), 1) |
Jacob Kopczynski | 408f3fd | 2020-08-07 13:31:59 -0700 | [diff] [blame] | 240 | req = _struct_to_ctp_request(request_map['fake_board_fake_model']) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 241 | self.assertEqual(req.params.scheduling.priority, 30) |
| 242 | |
| 243 | def testRequestWithInvalidTags(self): |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 244 | suite_kwargs = [_get_suite_params(board=b) for b in ['foo', 'foo']] |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 245 | |
| 246 | # test request having None String Tag |
| 247 | suite_kwargs[0]['model'] = 'None' |
| 248 | |
| 249 | # test request having None Tag |
| 250 | suite_kwargs[1]['model'] = None |
| 251 | |
| 252 | for suite in suite_kwargs: |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 253 | task_executor.push(task_executor.SUITES_QUEUE, tag='fake_suite', **suite) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 254 | |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 255 | tasks = self.queue.lease_tasks_by_tag( |
| 256 | 3600, constants.Buildbucket.MULTIREQUEST_SIZE, deadline=60) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 257 | executed = self.client.multirequest_run(tasks, 'fake_suite') |
Jacob Kopczynski | f464c8f | 2020-08-20 17:54:35 +0000 | [diff] [blame] | 258 | self._assert_bb_client_called() |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 259 | self.assertEqual(len(executed), 2) |
| 260 | |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 261 | request_map = self._extract_request_map_from_bb_client() |
| 262 | self.assertEqual(len(request_map), 2) |
Xinan Lin | fb63d57 | 2019-09-24 15:49:04 -0700 | [diff] [blame] | 263 | want = { |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 264 | 'suite:fake_suite', |
| 265 | 'build:fake_cros_build', |
Xinan Lin | 1516edb | 2020-07-05 23:13:54 -0700 | [diff] [blame] | 266 | 'label-pool:MANAGED_POOL_QUOTA', |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 267 | 'label-board:foo', |
Taylor Clark | e12ec9a | 2021-02-18 22:22:19 +0000 | [diff] [blame] | 268 | 'ctp-fwd-task-name:None', |
Xinan Lin | fb63d57 | 2019-09-24 15:49:04 -0700 | [diff] [blame] | 269 | } |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 270 | want_bb = { |
Jared Loucks | 438bf3a | 2022-04-06 13:43:15 -0600 | [diff] [blame] | 271 | 'label-image:fake_cros_build', |
| 272 | 'label-suite:fake_suite', |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 273 | 'suite:fake_suite', |
Dhanya Ganesh | edb78cb | 2020-07-20 19:40:50 +0000 | [diff] [blame] | 274 | 'user_agent:suite_scheduler', |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 275 | } |
| 276 | for req_name in ['foo', 'foo_1']: |
Jacob Kopczynski | 408f3fd | 2020-08-07 13:31:59 -0700 | [diff] [blame] | 277 | req = _struct_to_ctp_request(request_map[req_name]) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 278 | req_tags = set([t for t in req.params.decorations.tags]) |
| 279 | self.assertEqual(want, req_tags) |
| 280 | bb_tags = set([t for t in self._extract_tags_from_bb_client()]) |
| 281 | self.assertEqual(want_bb, bb_tags) |
Xinan Lin | fb63d57 | 2019-09-24 15:49:04 -0700 | [diff] [blame] | 282 | |
Prathmesh Prabhu | 2382a18 | 2019-09-07 21:18:10 -0700 | [diff] [blame] | 283 | def testRequestTags(self): |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 284 | suite_kwargs = _get_suite_params( |
| 285 | board='fake_board', |
| 286 | model='fake_model', |
Xinan Lin | 1516edb | 2020-07-05 23:13:54 -0700 | [diff] [blame] | 287 | pool='DUT_POOL_QUOTA', |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 288 | suite='fake_suite', |
| 289 | cros_build='fake_build', |
Taylor Clark | e12ec9a | 2021-02-18 22:22:19 +0000 | [diff] [blame] | 290 | analytics_name='fake_analytics_name', |
Prathmesh Prabhu | 2382a18 | 2019-09-07 21:18:10 -0700 | [diff] [blame] | 291 | ) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 292 | task_executor.push(task_executor.SUITES_QUEUE, |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 293 | tag='fake_suite', |
| 294 | **suite_kwargs) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 295 | |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 296 | tasks = self.queue.lease_tasks_by_tag( |
| 297 | 3600, constants.Buildbucket.MULTIREQUEST_SIZE, deadline=60) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 298 | self.client.multirequest_run(tasks, 'fake_suite') |
Prathmesh Prabhu | 2382a18 | 2019-09-07 21:18:10 -0700 | [diff] [blame] | 299 | self._assert_bb_client_called() |
| 300 | want = { |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 301 | 'label-board:fake_board', |
| 302 | 'label-model:fake_model', |
Xinan Lin | 1516edb | 2020-07-05 23:13:54 -0700 | [diff] [blame] | 303 | 'label-pool:DUT_POOL_QUOTA', |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 304 | 'suite:fake_suite', |
| 305 | 'build:fake_build', |
Taylor Clark | e12ec9a | 2021-02-18 22:22:19 +0000 | [diff] [blame] | 306 | 'analytics_name:fake_analytics_name', |
| 307 | 'ctp-fwd-task-name:None', |
Prathmesh Prabhu | 2382a18 | 2019-09-07 21:18:10 -0700 | [diff] [blame] | 308 | } |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 309 | want_bb = { |
Jared Loucks | 438bf3a | 2022-04-06 13:43:15 -0600 | [diff] [blame] | 310 | 'label-suite:fake_suite', |
| 311 | 'label-image:fake_build', |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 312 | 'suite:fake_suite', |
Dhanya Ganesh | edb78cb | 2020-07-20 19:40:50 +0000 | [diff] [blame] | 313 | 'user_agent:suite_scheduler', |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 314 | } |
| 315 | request_map = self._extract_request_map_from_bb_client() |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 316 | request_list = [ |
Jacob Kopczynski | 408f3fd | 2020-08-07 13:31:59 -0700 | [diff] [blame] | 317 | _struct_to_ctp_request(v) for v in request_map.values() |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 318 | ] |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 319 | self.assertEqual(len(request_list), 1) |
| 320 | req = request_list[0] |
Prathmesh Prabhu | 2382a18 | 2019-09-07 21:18:10 -0700 | [diff] [blame] | 321 | req_tags = set([t for t in req.params.decorations.tags]) |
| 322 | self.assertEqual(want, req_tags) |
| 323 | bb_tags = set([t for t in self._extract_tags_from_bb_client()]) |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 324 | self.assertEqual(want_bb, bb_tags) |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 325 | |
Xinan Lin | ba3b932 | 2020-04-24 15:08:12 -0700 | [diff] [blame] | 326 | def testRequestUserDefinedDimensions(self): |
| 327 | suite_kwargs = _get_suite_params( |
| 328 | board='fake_board', |
| 329 | model='fake_model', |
Xinan Lin | 1516edb | 2020-07-05 23:13:54 -0700 | [diff] [blame] | 330 | pool='MANAGED_POOL_QUOTA', |
Xinan Lin | ba3b932 | 2020-04-24 15:08:12 -0700 | [diff] [blame] | 331 | suite='fake_suite', |
| 332 | cros_build='fake_build', |
| 333 | ) |
Patrick Meiring | d4f6077 | 2021-03-04 12:09:28 +1100 | [diff] [blame] | 334 | suite_kwargs['dimensions'] = 'label-wifi:foo, label-bt:hoo' |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 335 | task_executor.push(task_executor.SUITES_QUEUE, |
| 336 | tag='fake_suite', |
| 337 | **suite_kwargs) |
Xinan Lin | ba3b932 | 2020-04-24 15:08:12 -0700 | [diff] [blame] | 338 | |
| 339 | tasks = self.queue.lease_tasks_by_tag( |
| 340 | 3600, constants.Buildbucket.MULTIREQUEST_SIZE, deadline=60) |
| 341 | self.client.multirequest_run(tasks, 'fake_suite') |
| 342 | self._assert_bb_client_called() |
| 343 | request_map = self._extract_request_map_from_bb_client() |
| 344 | request_list = [ |
Jacob Kopczynski | 408f3fd | 2020-08-07 13:31:59 -0700 | [diff] [blame] | 345 | _struct_to_ctp_request(v) for v in request_map.values() |
Xinan Lin | ba3b932 | 2020-04-24 15:08:12 -0700 | [diff] [blame] | 346 | ] |
| 347 | self.assertEqual(len(request_list), 1) |
| 348 | req = request_list[0] |
| 349 | free_dimensions = req.params.freeform_attributes.swarming_dimensions |
| 350 | self.assertTrue(len(free_dimensions), 2) |
| 351 | self.assertTrue('label-wifi:foo' in free_dimensions) |
| 352 | self.assertTrue('label-bt:hoo' in free_dimensions) |
| 353 | |
| 354 | def testRequestInvalidUserDefinedDimensions(self): |
| 355 | suite_kwargs = _get_suite_params( |
| 356 | board='fake_board', |
| 357 | model='fake_model', |
Xinan Lin | 1516edb | 2020-07-05 23:13:54 -0700 | [diff] [blame] | 358 | pool='MANAGED_POOL_QUOTA', |
Xinan Lin | ba3b932 | 2020-04-24 15:08:12 -0700 | [diff] [blame] | 359 | suite='fake_suite', |
| 360 | cros_build='fake_build', |
| 361 | ) |
| 362 | suite_kwargs['dimensions'] = 'invalid-dimension' |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 363 | task_executor.push(task_executor.SUITES_QUEUE, |
| 364 | tag='fake_suite', |
| 365 | **suite_kwargs) |
Xinan Lin | ba3b932 | 2020-04-24 15:08:12 -0700 | [diff] [blame] | 366 | |
| 367 | tasks = self.queue.lease_tasks_by_tag( |
| 368 | 3600, constants.Buildbucket.MULTIREQUEST_SIZE, deadline=60) |
| 369 | self.client.multirequest_run(tasks, 'fake_suite') |
| 370 | self._assert_bb_client_not_called() |
| 371 | |
Prathmesh Prabhu | 73801e1 | 2019-08-30 14:09:31 -0700 | [diff] [blame] | 372 | def _get_client(self, addr): |
| 373 | self.assertEqual(ADDR, addr) |
| 374 | return self.fake_client |
| 375 | |
Jacob Kopczynski | f464c8f | 2020-08-20 17:54:35 +0000 | [diff] [blame] | 376 | def _assert_bb_client_called(self, times=1): |
| 377 | actual = len(self.fake_client.called_with_requests) |
| 378 | self.assertEqual(actual, times, |
| 379 | "BB client called %s times, expected %s" % (actual, times)) |
Prathmesh Prabhu | 9a8060a | 2019-08-30 14:13:23 -0700 | [diff] [blame] | 380 | |
| 381 | def _assert_bb_client_not_called(self): |
Jacob Kopczynski | f464c8f | 2020-08-20 17:54:35 +0000 | [diff] [blame] | 382 | actual = len(self.fake_client.called_with_requests) |
| 383 | self.assertEqual(actual, 0, |
| 384 | "BB client called %s times, expected 0"% actual) |
Prathmesh Prabhu | 9a8060a | 2019-08-30 14:13:23 -0700 | [diff] [blame] | 385 | |
Xinan Lin | 9e4917d | 2019-11-04 10:58:47 -0800 | [diff] [blame] | 386 | def _extract_request_map_from_bb_client(self): |
| 387 | return self.fake_client.called_with_requests[0].properties['requests'] |
Prathmesh Prabhu | 6348ea8 | 2019-08-30 14:15:21 -0700 | [diff] [blame] | 388 | |
Prathmesh Prabhu | 2382a18 | 2019-09-07 21:18:10 -0700 | [diff] [blame] | 389 | def _extract_tags_from_bb_client(self): |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 390 | return [ |
| 391 | '%s:%s' % (t.key, t.value) |
| 392 | for t in self.fake_client.called_with_requests[0].tags |
| 393 | ] |
Prathmesh Prabhu | 2382a18 | 2019-09-07 21:18:10 -0700 | [diff] [blame] | 394 | |
Prathmesh Prabhu | 9a8060a | 2019-08-30 14:13:23 -0700 | [diff] [blame] | 395 | |
Xinan Lin | df0698a | 2020-02-05 22:38:11 -0800 | [diff] [blame] | 396 | class TestTaskExecutions(TestPlatformClientTestCase): |
Xinan Lin | df0698a | 2020-02-05 22:38:11 -0800 | [diff] [blame] | 397 | def setUp(self): |
| 398 | super(TestTaskExecutions, self).setUp() |
| 399 | _mock_bq_client = mock.patch('rest_client.BigqueryRestClient') |
| 400 | mock_bq_client = _mock_bq_client.start() |
| 401 | self.addCleanup(_mock_bq_client.stop) |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 402 | self.mock_bq_client = FakeBigqueryRestClient(None, |
| 403 | project='proj', |
| 404 | dataset='dataset', |
| 405 | table='foo') |
Xinan Lin | df0698a | 2020-02-05 22:38:11 -0800 | [diff] [blame] | 406 | mock_bq_client.return_value = self.mock_bq_client |
| 407 | |
| 408 | def testRecordSuccessfulTaskExecution(self): |
Xinan Lin | 9b17c5b | 2020-08-06 10:43:30 -0700 | [diff] [blame] | 409 | suite = _get_suite_params(board='fake_build', model='fake_model') |
Xinan Lin | 083ba8f | 2020-02-06 13:55:18 -0800 | [diff] [blame] | 410 | suite['task_id'] = FAKE_UUID |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 411 | task_executor.push(task_executor.SUITES_QUEUE, tag='fake_suite', **suite) |
| 412 | tasks = self.queue.lease_tasks_by_tag( |
| 413 | 3600, constants.Buildbucket.MULTIREQUEST_SIZE, deadline=60) |
Xinan Lin | df0698a | 2020-02-05 22:38:11 -0800 | [diff] [blame] | 414 | self.client.multirequest_run(tasks, 'fake_suite') |
| 415 | self._assert_bb_client_called() |
| 416 | task_execution = json_format.Parse( |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 417 | json.dumps(self.mock_bq_client.rows[0]['json']), |
| 418 | analytics_pb2.ExecutionTask()) |
Xinan Lin | df0698a | 2020-02-05 22:38:11 -0800 | [diff] [blame] | 419 | self.assertEqual(task_execution.queued_task_id, FAKE_UUID) |
Xinan Lin | 9b17c5b | 2020-08-06 10:43:30 -0700 | [diff] [blame] | 420 | self.assertEqual(task_execution.request_tag, 'fake_build_fake_model') |
Xinan Lin | df0698a | 2020-02-05 22:38:11 -0800 | [diff] [blame] | 421 | self.assertEqual(task_execution.response.ctp_build_id, str(FAKE_BUILD_ID)) |
| 422 | self.assertEqual(task_execution.error.error_message, '') |
| 423 | |
| 424 | def testRecordFailedTaskExecution(self): |
Xinan Lin | 9b17c5b | 2020-08-06 10:43:30 -0700 | [diff] [blame] | 425 | suite = _get_suite_params(board='fake_build', model='fake_model') |
Xinan Lin | 083ba8f | 2020-02-06 13:55:18 -0800 | [diff] [blame] | 426 | suite['task_id'] = FAKE_UUID |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 427 | task_executor.push(task_executor.SUITES_QUEUE, tag='fake_suite', **suite) |
Xinan Lin | df0698a | 2020-02-05 22:38:11 -0800 | [diff] [blame] | 428 | self.fake_client.error = "cros_test_platform error" |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 429 | tasks = self.queue.lease_tasks_by_tag( |
| 430 | 3600, constants.Buildbucket.MULTIREQUEST_SIZE, deadline=60) |
Xinan Lin | df0698a | 2020-02-05 22:38:11 -0800 | [diff] [blame] | 431 | self.client.multirequest_run(tasks, 'fake_suite') |
| 432 | self._assert_bb_client_called() |
| 433 | task_execution = json_format.Parse( |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 434 | json.dumps(self.mock_bq_client.rows[0]['json']), |
| 435 | analytics_pb2.ExecutionTask()) |
Xinan Lin | df0698a | 2020-02-05 22:38:11 -0800 | [diff] [blame] | 436 | self.assertEqual(task_execution.queued_task_id, FAKE_UUID) |
Xinan Lin | 9b17c5b | 2020-08-06 10:43:30 -0700 | [diff] [blame] | 437 | self.assertEqual(task_execution.request_tag, 'fake_build_fake_model') |
Xinan Lin | df0698a | 2020-02-05 22:38:11 -0800 | [diff] [blame] | 438 | self.assertEqual(task_execution.response.ctp_build_id, '') |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 439 | self.assertEqual(task_execution.error.error_message, |
| 440 | self.fake_client.error) |
Xinan Lin | df0698a | 2020-02-05 22:38:11 -0800 | [diff] [blame] | 441 | |
Xinan Lin | 083ba8f | 2020-02-06 13:55:18 -0800 | [diff] [blame] | 442 | def testIgnoreTaskWithoutTaskID(self): |
| 443 | suite = _get_suite_params(board='foo') |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 444 | task_executor.push(task_executor.SUITES_QUEUE, tag='fake_suite', **suite) |
Xinan Lin | 083ba8f | 2020-02-06 13:55:18 -0800 | [diff] [blame] | 445 | self.fake_client.error = "cros_test_platform error" |
Prathmesh Prabhu | af0857f | 2020-06-20 00:16:32 -0700 | [diff] [blame] | 446 | tasks = self.queue.lease_tasks_by_tag( |
| 447 | 3600, constants.Buildbucket.MULTIREQUEST_SIZE, deadline=60) |
Xinan Lin | 083ba8f | 2020-02-06 13:55:18 -0800 | [diff] [blame] | 448 | self.client.multirequest_run(tasks, 'fake_suite') |
| 449 | self._assert_bb_client_called() |
| 450 | self.assertEqual(len(self.mock_bq_client.rows), 0) |
| 451 | |
Jacob Kopczynski | 408f3fd | 2020-08-07 13:31:59 -0700 | [diff] [blame] | 452 | def testIgnoreSuiteForUnmanagedPoolInStaging(self): |
Xinan Lin | c54a746 | 2020-04-17 15:39:01 -0700 | [diff] [blame] | 453 | self.mock_application_id.return_value = 'suite-scheduler-staging' |
| 454 | suite = _get_suite_params(pool='foo') |
| 455 | task_executor.push(task_executor.SUITES_QUEUE, tag='fake_suite', **suite) |
| 456 | self.fake_client.error = "cros_test_platform error" |
| 457 | tasks = self.queue.lease_tasks_by_tag( |
| 458 | 3600, constants.Buildbucket.MULTIREQUEST_SIZE, deadline=60) |
| 459 | self.client.multirequest_run(tasks, 'fake_suite') |
| 460 | self._assert_bb_client_not_called() |
| 461 | |
Xinan Lin | df0698a | 2020-02-05 22:38:11 -0800 | [diff] [blame] | 462 | |
Jacob Kopczynski | 408f3fd | 2020-08-07 13:31:59 -0700 | [diff] [blame] | 463 | def _struct_to_ctp_request(struct_pb2): |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 464 | """Transform google struct proto to test_platform_request. |
| 465 | |
| 466 | Args: |
| 467 | struct_pb2: A struct_pb2 instance. |
| 468 | |
| 469 | Returns: |
Jacob Kopczynski | 408f3fd | 2020-08-07 13:31:59 -0700 | [diff] [blame] | 470 | A ctp_request instance. |
Xinan Lin | 3ba18a0 | 2019-08-13 15:44:55 -0700 | [diff] [blame] | 471 | """ |
| 472 | json = json_format.MessageToJson(struct_pb2) |
Jacob Kopczynski | 408f3fd | 2020-08-07 13:31:59 -0700 | [diff] [blame] | 473 | return json_format.Parse(json, ctp_request.Request()) |