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

Side by Side Diff: tools/perf/core/perf_data_generator.py

Issue 2879993003: [Telemetry] Increase sharding timeout to 2.5 hours from 2 hours. (Closed)
Patch Set: Fix unit tests Created 3 years, 7 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 #!/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 # pylint: disable=too-many-lines 6 # pylint: disable=too-many-lines
7 7
8 """Script to generate chromium.perf.json and chromium.perf.fyi.json in 8 """Script to generate chromium.perf.json and chromium.perf.fyi.json in
9 the src/testing/buildbot directory and benchmark.csv in the src/tools/perf 9 the src/testing/buildbot directory and benchmark.csv in the src/tools/perf
10 directory. Maintaining these files by hand is too unwieldy. 10 directory. Maintaining these files by hand is too unwieldy.
(...skipping 522 matching lines...) Expand 10 before | Expand all | Expand 10 after
533 'name': step_name, 533 'name': step_name,
534 } 534 }
535 if override_compile_targets: 535 if override_compile_targets:
536 result['override_compile_targets'] = override_compile_targets 536 result['override_compile_targets'] = override_compile_targets
537 if swarming_dimensions: 537 if swarming_dimensions:
538 result['swarming'] = { 538 result['swarming'] = {
539 # Always say this is true regardless of whether the tester 539 # Always say this is true regardless of whether the tester
540 # supports swarming. It doesn't hurt. 540 # supports swarming. It doesn't hurt.
541 'can_use_on_swarming_builders': True, 541 'can_use_on_swarming_builders': True,
542 'expiration': 10 * 60 * 60, # 10 hour timeout for now (crbug.com/699312) 542 'expiration': 10 * 60 * 60, # 10 hour timeout for now (crbug.com/699312)
543 'hard_timeout': swarming_timeout if swarming_timeout else 7200, 543 'hard_timeout': swarming_timeout if swarming_timeout else 9000, # 2.5hrs
544 'ignore_task_failure': ignore_task_failure, 544 'ignore_task_failure': ignore_task_failure,
545 'io_timeout': 3600, 545 'io_timeout': 3600,
546 'dimension_sets': swarming_dimensions, 546 'dimension_sets': swarming_dimensions,
547 } 547 }
548 return result 548 return result
549 549
550 550
551 def generate_telemetry_test(swarming_dimensions, benchmark_name, browser): 551 def generate_telemetry_test(swarming_dimensions, benchmark_name, browser):
552 # The step name must end in 'test' or 'tests' in order for the 552 # The step name must end in 'test' or 'tests' in order for the
553 # results to automatically show up on the flakiness dashboard. 553 # results to automatically show up on the flakiness dashboard.
(...skipping 420 matching lines...) Expand 10 before | Expand all | Expand 10 after
974 return 0 974 return 0
975 else: 975 else:
976 print ('The perf JSON config files are not up-to-date. Please run %s ' 976 print ('The perf JSON config files are not up-to-date. Please run %s '
977 'without --validate-only flag to update the perf JSON ' 977 'without --validate-only flag to update the perf JSON '
978 'configs and benchmark.csv.') % sys.argv[0] 978 'configs and benchmark.csv.') % sys.argv[0]
979 return 1 979 return 1
980 else: 980 else:
981 update_all_tests([fyi_waterfall, waterfall]) 981 update_all_tests([fyi_waterfall, waterfall])
982 update_benchmark_csv() 982 update_benchmark_csv()
983 return 0 983 return 0
OLDNEW
« no previous file with comments | « testing/buildbot/chromium.perf.fyi.json ('k') | tools/perf/core/perf_data_generator_unittest.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698