Commit ff9d46ff authored by Stephen Martinis's avatar Stephen Martinis Committed by Commit Bot

//tools/perf: Add compile target to linux builder

Also creates cleaner way to add additional compile targets to builders.

NOTRY=true # win chromium rel ng seems flaky

Bug: 758630
Change-Id: I03f2b599d8f953f21d3a6d0e57d8db90d8d13999
Reviewed-on: https://chromium-review.googlesource.com/691065
Commit-Queue: Stephen Martinis <martiniss@chromium.org>
Reviewed-by: default avatarNed Nguyen <nednguyen@google.com>
Cr-Commit-Position: refs/heads/master@{#505251}
parent 7a49b8cd
...@@ -29834,6 +29834,11 @@ ...@@ -29834,6 +29834,11 @@
"microdump_stackwalk" "microdump_stackwalk"
] ]
}, },
"Linux Builder": {
"additional_compile_targets": [
"telemetry_perf_tests_new"
]
},
"Linux Perf": { "Linux Perf": {
"isolated_scripts": [ "isolated_scripts": [
{ {
...@@ -122,17 +122,21 @@ def get_fyi_waterfall_config(): ...@@ -122,17 +122,21 @@ def get_fyi_waterfall_config():
return waterfall return waterfall
# Additional compile targets to add to builders.
BUILDER_ADDITIONAL_COMPILE_TARGETS = {
'Android Compile': ['microdump_stackwalk'],
'Android arm64 Compile': ['microdump_stackwalk'],
# crbug.com/758630. Test builder, will be removed.
'Linux Builder': ['telemetry_perf_tests_new'],
}
def get_waterfall_config(): def get_waterfall_config():
waterfall = {'builders':{}, 'testers': {}} waterfall = {'builders':{}, 'testers': {}}
waterfall = add_builder( for builder, targets in BUILDER_ADDITIONAL_COMPILE_TARGETS.items():
waterfall, 'Android Compile', additional_compile_targets=[ waterfall = add_builder(
'microdump_stackwalk' waterfall, builder, additional_compile_targets=targets)
])
waterfall = add_builder(
waterfall, 'Android arm64 Compile', additional_compile_targets=[
'microdump_stackwalk'
])
# These configurations are taken from chromium_perf.py in # These configurations are taken from chromium_perf.py in
# build/scripts/slave/recipe_modules/chromium_tests and must be kept in sync # build/scripts/slave/recipe_modules/chromium_tests and must be kept in sync
...@@ -1049,9 +1053,8 @@ def verify_all_tests_in_benchmark_csv(tests, benchmark_metadata): ...@@ -1049,9 +1053,8 @@ def verify_all_tests_in_benchmark_csv(tests, benchmark_metadata):
elif 'scripts' in tests[t]: elif 'scripts' in tests[t]:
scripts = tests[t]['scripts'] scripts = tests[t]['scripts']
else: else:
assert('Android Compile' == t assert(t in BUILDER_ADDITIONAL_COMPILE_TARGETS
or 'Android arm64 Compile' == t or t.startswith('AAAAA')), 'Unknown test data %s' % t
or t.startswith('AAAAA')), 'Unknown test data %s' % t
for s in scripts: for s in scripts:
name = s['name'] name = s['name']
name = re.sub('\\.reference$', '', name) name = re.sub('\\.reference$', '', name)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment