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

Side by Side Diff: telemetry/telemetry/benchmark_runner.py

Issue 2941123002: [Telemetry] Move discover to common/py_utils (Closed)
Patch Set: fix imports Created 3 years, 6 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 # 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 """Parses the command line, discovers the appropriate benchmarks, and runs them. 5 """Parses the command line, discovers the appropriate benchmarks, and runs them.
6 6
7 Handles benchmark configuration, but all the logic for 7 Handles benchmark configuration, but all the logic for
8 actually running the benchmark is in Benchmark and PageRunner.""" 8 actually running the benchmark is in Benchmark and PageRunner."""
9 9
10 import argparse 10 import argparse
11 import json 11 import json
12 import logging 12 import logging
13 import os 13 import os
14 import sys 14 import sys
15 15
16 from telemetry import benchmark 16 from telemetry import benchmark
17 from telemetry.core import discover
18 from telemetry import decorators 17 from telemetry import decorators
19 from telemetry.internal.browser import browser_finder 18 from telemetry.internal.browser import browser_finder
20 from telemetry.internal.browser import browser_options 19 from telemetry.internal.browser import browser_options
21 from telemetry.internal.util import binary_manager 20 from telemetry.internal.util import binary_manager
22 from telemetry.internal.util import command_line 21 from telemetry.internal.util import command_line
23 from telemetry.internal.util import ps_util 22 from telemetry.internal.util import ps_util
24 from telemetry.util import matching 23 from telemetry.util import matching
25 from telemetry.util import bot_utils 24 from telemetry.util import bot_utils
26 25
26 from py_utils import discover
27 27
28 # Right now, we only have one of each of our power perf bots. This means that 28 # Right now, we only have one of each of our power perf bots. This means that
29 # all eligible Telemetry benchmarks are run unsharded, which results in very 29 # all eligible Telemetry benchmarks are run unsharded, which results in very
30 # long (12h) cycle times. We'd like to reduce the number of tests that we run 30 # long (12h) cycle times. We'd like to reduce the number of tests that we run
31 # on each bot drastically until we get more of the same hardware to shard tests 31 # on each bot drastically until we get more of the same hardware to shard tests
32 # with, but we can't do so until we've verified that the hardware configuration 32 # with, but we can't do so until we've verified that the hardware configuration
33 # is a viable one for Chrome Telemetry tests. This is done by seeing at least 33 # is a viable one for Chrome Telemetry tests. This is done by seeing at least
34 # one all-green test run. As this happens for each bot, we'll add it to this 34 # one all-green test run. As this happens for each bot, we'll add it to this
35 # whitelist, making it eligible to run only BattOr power tests. 35 # whitelist, making it eligible to run only BattOr power tests.
36 GOOD_POWER_PERF_BOT_WHITELIST = [ 36 GOOD_POWER_PERF_BOT_WHITELIST = [
(...skipping 396 matching lines...) Expand 10 before | Expand all | Expand 10 after
433 command.ProcessCommandLineArgs(parser, options, environment) 433 command.ProcessCommandLineArgs(parser, options, environment)
434 434
435 if command == Help: 435 if command == Help:
436 command_instance = command(all_commands) 436 command_instance = command(all_commands)
437 else: 437 else:
438 command_instance = command() 438 command_instance = command()
439 if isinstance(command_instance, command_line.OptparseCommand): 439 if isinstance(command_instance, command_line.OptparseCommand):
440 return command_instance.Run(options) 440 return command_instance.Run(options)
441 else: 441 else:
442 return command_instance.Run(options, args) 442 return command_instance.Run(options, args)
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698