Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(347)

Side by Side Diff: scripts/slave/recipe_modules/chromium_tests/chromium_perf.py

Issue 2172553003: The second solution solution for ToT catapult. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Passing gclient config name to BuildSpec. Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 # Copyright 2015 The Chromium Authors. All rights reserved. 1 # Copyright 2015 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import collections 5 import collections
6 6
7 from . import steps 7 from . import steps
8 8
9 9
10 _builders = collections.defaultdict(dict) 10 _builders = collections.defaultdict(dict)
11 11
12 12
13 SPEC = { 13 SPEC = {
14 'builders': {}, 14 'builders': {},
15 'settings': { 15 'settings': {
16 'build_gs_bucket': 'chrome-perf', 16 'build_gs_bucket': 'chrome-perf',
17 }, 17 },
18 } 18 }
19 19
20 20
21 def _BaseSpec(bot_type, chromium_config, disable_tests, 21 def _BaseSpec(bot_type, chromium_config, disable_tests,
22 platform, target_bits, tests): 22 platform, target_bits, tests, gclient_config='chromium_perf'):
dtu 2016/08/10 01:19:39 Don't think we need a new parameter, we don't have
RobertoCN 2016/08/10 22:59:54 Done.
23 spec = { 23 spec = {
24 'bot_type': bot_type, 24 'bot_type': bot_type,
25 'chromium_config': chromium_config, 25 'chromium_config': chromium_config,
26 'chromium_config_kwargs': { 26 'chromium_config_kwargs': {
27 'BUILD_CONFIG': 'Release', 27 'BUILD_CONFIG': 'Release',
28 'TARGET_BITS': target_bits, 28 'TARGET_BITS': target_bits,
29 }, 29 },
30 'disable_tests': disable_tests, 30 'disable_tests': disable_tests,
31 'gclient_config': 'chromium_perf', 31 'gclient_config': gclient_config,
32 'testing': { 32 'testing': {
33 'platform': 'linux' if platform == 'android' else platform, 33 'platform': 'linux' if platform == 'android' else platform,
34 }, 34 },
35 'tests': tests, 35 'tests': tests,
36 } 36 }
37 37
38 if platform == 'android': 38 if platform == 'android':
39 spec['android_config'] = 'chromium_perf' 39 spec['android_config'] = 'chromium_perf'
40 spec['chromium_apply_config'] = ['android'] 40 spec['chromium_apply_config'] = ['android']
41 spec['chromium_config_kwargs']['TARGET_ARCH'] = 'arm' 41 spec['chromium_config_kwargs']['TARGET_ARCH'] = 'arm'
42 spec['chromium_config_kwargs']['TARGET_PLATFORM'] = 'android' 42 spec['chromium_config_kwargs']['TARGET_PLATFORM'] = 'android'
43 spec['gclient_apply_config'] = ['android'] 43 spec['gclient_apply_config'] = ['android']
44 44
45 return spec 45 return spec
46 46
47 47
48 def BuildSpec(chromium_config, perf_id, platform, target_bits): 48 def BuildSpec(chromium_config, perf_id, platform, target_bits,
49 gclient_config='chromium_perf'):
49 if platform == 'android': 50 if platform == 'android':
50 # TODO: Run sizes on Android. 51 # TODO: Run sizes on Android.
51 tests = [] 52 tests = []
52 else: 53 else:
53 tests = [steps.SizesStep('https://chromeperf.appspot.com', perf_id)] 54 tests = [steps.SizesStep('https://chromeperf.appspot.com', perf_id)]
54 55
55 spec = _BaseSpec( 56 spec = _BaseSpec(
56 bot_type='builder', 57 bot_type='builder',
57 chromium_config=chromium_config, 58 chromium_config=chromium_config,
58 disable_tests=True, 59 disable_tests=True,
59 platform=platform, 60 platform=platform,
60 target_bits=target_bits, 61 target_bits=target_bits,
61 tests=tests) 62 tests=tests,
63 gclient_config=gclient_config,
64 )
62 65
63 spec['compile_targets'] = ['chromium_builder_perf'] 66 spec['compile_targets'] = ['chromium_builder_perf']
64 67
65 return spec 68 return spec
66 69
67 70
68 def _TestSpec(chromium_config, parent_builder, perf_id, platform, target_bits, 71 def _TestSpec(chromium_config, parent_builder, perf_id, platform, target_bits,
69 max_battery_temp, shard_index, num_host_shards, 72 max_battery_temp, shard_index, num_host_shards,
70 num_device_shards): 73 num_device_shards):
71 spec = _BaseSpec( 74 spec = _BaseSpec(
(...skipping 89 matching lines...) Expand 10 before | Expand all | Expand 10 after
161 _AddTestSpec('Mac 10.10 Perf', 'chromium-rel-mac10', 'mac', 164 _AddTestSpec('Mac 10.10 Perf', 'chromium-rel-mac10', 'mac',
162 num_host_shards=5) 165 num_host_shards=5)
163 _AddTestSpec('Mac Retina Perf', 'chromium-rel-mac-retina', 'mac', 166 _AddTestSpec('Mac Retina Perf', 'chromium-rel-mac-retina', 'mac',
164 num_host_shards=5) 167 num_host_shards=5)
165 _AddTestSpec('Mac HDD Perf', 'chromium-rel-mac-hdd', 'mac', 168 _AddTestSpec('Mac HDD Perf', 'chromium-rel-mac-hdd', 'mac',
166 num_host_shards=5) 169 num_host_shards=5)
167 170
168 171
169 _AddTestSpec('Linux Perf', 'linux-release', 'linux', 172 _AddTestSpec('Linux Perf', 'linux-release', 'linux',
170 num_host_shards=5) 173 num_host_shards=5)
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698