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

Unified Diff: build/android/test_runner.py

Issue 23784006: [Android] Add --no-timeout to perf sharder. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Pass retries through Created 7 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « build/android/pylib/perf/test_runner.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/android/test_runner.py
diff --git a/build/android/test_runner.py b/build/android/test_runner.py
index 2c87df901f8499afdecc314bf046a87cd563ba32..e2505ab5a35bae2a3c66d22e1a2dd42ec02faae5 100755
--- a/build/android/test_runner.py
+++ b/build/android/test_runner.py
@@ -4,11 +4,7 @@
# Use of this source code is governed by a BSD-style license that can be
# found in the LICENSE file.
-"""Runs all types of tests from one unified interface.
-
-TODO(gkanwar):
-* Add options to run Monkey tests.
-"""
+"""Runs all types of tests from one unified interface."""
import collections
import logging
@@ -433,14 +429,20 @@ def AddPerfTestOptions(option_parser):
option_parser.command_list = []
option_parser.example = ('%prog perf --steps perf_steps.json')
- option_parser.add_option('--steps', help='JSON file containing the list '
- 'of perf steps to run.')
- option_parser.add_option('--flaky-steps',
- help='A JSON file containing steps that are flaky '
- 'and will have its exit code ignored.')
- option_parser.add_option('--print-step', help='The name of a previously '
- 'executed perf step to print.')
-
+ option_parser.add_option(
+ '--steps',
+ help='JSON file containing the list of perf steps to run.')
+ option_parser.add_option(
+ '--flaky-steps',
+ help=('A JSON file containing steps that are flaky '
+ 'and will have its exit code ignored.'))
+ option_parser.add_option(
+ '--print-step',
+ help='The name of a previously executed perf step to print.')
+ option_parser.add_option(
+ '--no-timeout', action='store_true',
+ help=('Do not impose a timeout. Each perf step is responsible for '
+ 'implementing the timeout logic.'))
AddCommonOptions(option_parser)
@@ -458,7 +460,8 @@ def ProcessPerfTestOptions(options, error_func):
if not options.steps and not options.print_step:
error_func('Please specify --steps or --print-step')
return perf_test_options.PerfOptions(
- options.steps, options.flaky_steps, options.print_step)
+ options.steps, options.flaky_steps, options.print_step,
+ options.no_timeout)
def _RunGTests(options, error_func, devices):
@@ -571,7 +574,8 @@ def _RunMonkeyTests(options, error_func, devices):
runner_factory, tests = monkey_setup.Setup(monkey_options)
results, exit_code = test_dispatcher.RunTests(
- tests, runner_factory, devices, shard=False, test_timeout=None)
+ tests, runner_factory, devices, shard=False, test_timeout=None,
+ num_retries=options.num_retries)
report_results.LogFull(
results=results,
@@ -591,7 +595,8 @@ def _RunPerfTests(options, error_func, devices):
runner_factory, tests = perf_setup.Setup(perf_options)
results, _ = test_dispatcher.RunTests(
- tests, runner_factory, devices, shard=True, test_timeout=None)
+ tests, runner_factory, devices, shard=True, test_timeout=None,
+ num_retries=options.num_retries)
report_results.LogFull(
results=results,
« no previous file with comments | « build/android/pylib/perf/test_runner.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698