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

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

Issue 1823183002: Run Sizes on Windows perf builders (take 2) (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Created 4 years, 9 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
« no previous file with comments | « no previous file | scripts/slave/recipe_modules/chromium_tests/chromium_perf.py » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2013 The Chromium Authors. All rights reserved. 1 # Copyright 2013 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 re 5 import re
6 6
7 from recipe_engine import recipe_api 7 from recipe_engine import recipe_api
8 from recipe_engine import util as recipe_util 8 from recipe_engine import util as recipe_util
9 9
10 class TestLauncherFilterFileInputPlaceholder(recipe_util.Placeholder): 10 class TestLauncherFilterFileInputPlaceholder(recipe_util.Placeholder):
(...skipping 378 matching lines...) Expand 10 before | Expand all | Expand 10 after
389 if platform: 389 if platform:
390 sizes_args.extend(['--platform', platform]) 390 sizes_args.extend(['--platform', platform])
391 else: 391 else:
392 sizes_args.extend(['--platform', self.c.TARGET_PLATFORM]) 392 sizes_args.extend(['--platform', self.c.TARGET_PLATFORM])
393 393
394 run_tests_args = ['--target', self.c.build_config_fs, 394 run_tests_args = ['--target', self.c.build_config_fs,
395 '--no-xvfb'] 395 '--no-xvfb']
396 properties_json = self.m.json.dumps(self.m.properties.legacy()) 396 properties_json = self.m.json.dumps(self.m.properties.legacy())
397 run_tests_args.extend(['--factory-properties', properties_json, 397 run_tests_args.extend(['--factory-properties', properties_json,
398 '--build-properties', properties_json]) 398 '--build-properties', properties_json])
399 run_tests_args.extend(['--annotate=graphing', 399 run_tests_args.extend(['--test-type=sizes',
400 '--test-type=sizes',
401 '--builder-name=%s' % self.m.properties['buildername' ], 400 '--builder-name=%s' % self.m.properties['buildername' ],
402 '--slave-name=%s' % self.m.properties['slavename'], 401 '--slave-name=%s' % self.m.properties['slavename'],
403 '--build-number=%s' % self.m.properties['buildnumber' ], 402 '--build-number=%s' % self.m.properties['buildnumber' ],
404 '--run-python-script']) 403 '--run-python-script'])
405 404
406 if perf_id: 405 if perf_id:
407 assert results_url is not None 406 assert results_url is not None
408 run_tests_args.extend(['--results-url=%s' % results_url, 407 run_tests_args.extend(['--annotate=graphing',
408 '--results-url=%s' % results_url,
409 '--perf-dashboard-id=sizes', 409 '--perf-dashboard-id=sizes',
410 '--perf-id=%s' % perf_id]) 410 '--perf-id=%s' % perf_id])
411 411
412 # If we have a clang revision, add that to the perf data point. 412 # If we have a clang revision, add that to the perf data point.
413 # TODO(hans): We want this for all perf data, not just sizes. 413 # TODO(hans): We want this for all perf data, not just sizes.
414 if hasattr(self, '_clang_version'): 414 if hasattr(self, '_clang_version'):
415 clang_rev = re.match(r'(\d+)(-\d+)?', self._clang_version).group(1) 415 clang_rev = re.match(r'(\d+)(-\d+)?', self._clang_version).group(1)
416 run_tests_args.append( 416 run_tests_args.append(
417 "--perf-config={'r_clang_rev': '%s'}" % clang_rev) 417 "--perf-config={'r_clang_rev': '%s'}" % clang_rev)
418 418
(...skipping 296 matching lines...) Expand 10 before | Expand all | Expand 10 after
715 })) 715 }))
716 716
717 def get_annotate_by_test_name(self, test_name): 717 def get_annotate_by_test_name(self, test_name):
718 return 'graphing' 718 return 'graphing'
719 719
720 def download_lto_plugin(self): 720 def download_lto_plugin(self):
721 return self.m.python( 721 return self.m.python(
722 name='download LTO plugin', 722 name='download LTO plugin',
723 script=self.m.path['checkout'].join( 723 script=self.m.path['checkout'].join(
724 'build', 'download_gold_plugin.py')) 724 'build', 'download_gold_plugin.py'))
OLDNEW
« no previous file with comments | « no previous file | scripts/slave/recipe_modules/chromium_tests/chromium_perf.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698