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

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

Issue 873403002: Add support for cc_perftests and other non-telemetry gtest based tests. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Rebase again + coverage. Created 5 years, 10 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 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 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 from . import steps 5 from . import steps
6 6
7 7
8 def _GetTargetName(platform, target_bits): 8 def _GetTargetName(platform, target_bits):
9 return ('Release_x64' if platform is 'win' and target_bits is 64 9 return ('Release_x64' if platform is 'win' and target_bits is 64
10 else 'Release') 10 else 'Release')
11 11
12 def _Spec(platform, parent_builder, perf_id, index, num_shards, target_bits): 12 def _Spec(platform, parent_builder, perf_id, index, num_shards, target_bits):
13 return { 13 return {
14 'disable_tests': True, 14 'disable_tests': False,
15 'bot_type': 'tester', 15 'bot_type': 'tester',
16 'chromium_config_kwargs': { 16 'chromium_config_kwargs': {
17 'BUILD_CONFIG': 'Release', 17 'BUILD_CONFIG': 'Release',
18 'TARGET_BITS': target_bits, 18 'TARGET_BITS': target_bits,
19 }, 19 },
20 'parent_buildername': parent_builder, 20 'parent_buildername': parent_builder,
21 'recipe_config': 'perf', 21 'recipe_config': 'perf',
22 'testing': { 22 'testing': {
23 'platform': platform, 23 'platform': platform,
24 }, 24 },
25 'test_spec_file': 'chromium.perf.json',
26 'test_generators': [
27 steps.generate_script,
28 ],
25 'tests': [ 29 'tests': [
26 steps.DynamicPerfTests( 30 steps.DynamicPerfTests(
27 _GetTargetName(platform, target_bits).lower(), 31 _GetTargetName(platform, target_bits).lower(),
28 perf_id, index, num_shards), 32 perf_id, index, num_shards),
29 ], 33 ],
34 'perf-id': perf_id,
35 'results-url': 'https://chromeperf.appspot.com',
30 } 36 }
31 37
32 38
33 def _AddBotSpec(name, platform, parent_builder, perf_id, target_bits, 39 def _AddBotSpec(name, platform, parent_builder, perf_id, target_bits,
34 num_shards): 40 num_shards):
35 if num_shards > 1: 41 if num_shards > 1:
36 for i in range(0, num_shards): 42 for i in range(0, num_shards):
37 builder_name = "%s (%d)" % (name, i + 1) 43 builder_name = "%s (%d)" % (name, i + 1)
38 SPEC['builders'][builder_name] = _Spec(platform, parent_builder, perf_id, 44 SPEC['builders'][builder_name] = _Spec(platform, parent_builder, perf_id,
39 i, num_shards, target_bits) 45 i, num_shards, target_bits)
(...skipping 202 matching lines...) Expand 10 before | Expand all | Expand 10 after
242 'gclient_apply_config': ['android'], 248 'gclient_apply_config': ['android'],
243 'parent_buildername': 'Android Builder', 249 'parent_buildername': 'Android Builder',
244 'recipe_config': 'perf', 250 'recipe_config': 'perf',
245 'android_config': 'perf', 251 'android_config': 'perf',
246 'testing': { 252 'testing': {
247 'platform': 'linux', 253 'platform': 'linux',
248 }, 254 },
249 'tests': [ 255 'tests': [
250 steps.AndroidPerfTests(v['perf_id'], v['num_device_shards']), 256 steps.AndroidPerfTests(v['perf_id'], v['num_device_shards']),
251 ], 257 ],
258 'perf-id': v['perf_id'],
259 'results-url': 'https://chromeperf.appspot.com',
252 } 260 }
OLDNEW
« no previous file with comments | « scripts/slave/recipe_modules/chromium/api.py ('k') | scripts/slave/recipe_modules/chromium/steps.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698