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

Side by Side Diff: tools/perf/generate_perf_json.py

Issue 2295803002: Fixing syntax for --upload-results flag (Closed)
Patch Set: Created 4 years, 3 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 | « testing/buildbot/chromium.perf.fyi.json ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 #!/usr/bin/env python 1 #!/usr/bin/env python
2 # Copyright 2016 The Chromium Authors. All rights reserved. 2 # Copyright 2016 The Chromium Authors. All rights reserved.
3 # Use of this source code is governed by a BSD-style license that can be 3 # Use of this source code is governed by a BSD-style license that can be
4 # found in the LICENSE file. 4 # found in the LICENSE file.
5 5
6 """Script to generate chromium.perf.json and chromium.perf.fyi.json in 6 """Script to generate chromium.perf.json and chromium.perf.fyi.json in
7 the src/testing/buildbot directory. Maintaining these files by hand is 7 the src/testing/buildbot directory. Maintaining these files by hand is
8 too unwieldy. 8 too unwieldy.
9 """ 9 """
10 10
(...skipping 272 matching lines...) Expand 10 before | Expand all | Expand 10 after
283 283
284 def generate_telemetry_test(tester_config, benchmark_name, browser): 284 def generate_telemetry_test(tester_config, benchmark_name, browser):
285 # The step name must end in 'test' or 'tests' in order for the 285 # The step name must end in 'test' or 'tests' in order for the
286 # results to automatically show up on the flakiness dashboard. 286 # results to automatically show up on the flakiness dashboard.
287 # (At least, this was true some time ago.) Continue to use this 287 # (At least, this was true some time ago.) Continue to use this
288 # naming convention for the time being to minimize changes. 288 # naming convention for the time being to minimize changes.
289 289
290 test_args = [ 290 test_args = [
291 benchmark_name, 291 benchmark_name,
292 '-v', 292 '-v',
293 '--upload_results', 293 '--upload-results',
294 '--output-format=chartjson', 294 '--output-format=chartjson',
295 '--browser=%s' % browser 295 '--browser=%s' % browser
296 ] 296 ]
297 # When this is enabled on more than just windows machines we will need 297 # When this is enabled on more than just windows machines we will need
298 # --device=android 298 # --device=android
299 299
300 step_name = benchmark_name 300 step_name = benchmark_name
301 if browser == 'reference': 301 if browser == 'reference':
302 test_args.append('--output-trace-tag=_ref') 302 test_args.append('--output-trace-tag=_ref')
303 step_name += '.reference' 303 step_name += '.reference'
(...skipping 103 matching lines...) Expand 10 before | Expand all | Expand 10 after
407 407
408 def main(): 408 def main():
409 waterfall = get_waterfall_config() 409 waterfall = get_waterfall_config()
410 fyi_waterfall = get_fyi_waterfall_config() 410 fyi_waterfall = get_fyi_waterfall_config()
411 generate_all_tests(fyi_waterfall, True) 411 generate_all_tests(fyi_waterfall, True)
412 generate_all_tests(waterfall, False) 412 generate_all_tests(waterfall, False)
413 return 0 413 return 0
414 414
415 if __name__ == "__main__": 415 if __name__ == "__main__":
416 sys.exit(main()) 416 sys.exit(main())
OLDNEW
« no previous file with comments | « testing/buildbot/chromium.perf.fyi.json ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698