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

Side by Side Diff: scripts/slave/recipe_modules/auto_bisect/bisector.py

Issue 1062583003: Adding config step to output the bisect config dictionary. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@hax
Patch Set: Created 5 years, 8 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/auto_bisect/example.expected/bad_deps_syntax.json » ('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 2015 The Chromium Authors. All rights reserved. 1 # Copyright 2015 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 . import bisect_results 7 from . import bisect_results
8 from . import depot_config 8 from . import depot_config
9 9
10 _DEPS_SHA_PATCH = """ 10 _DEPS_SHA_PATCH = """
(...skipping 11 matching lines...) Expand all
22 22
23 class Bisector(object): 23 class Bisector(object):
24 """This class abstracts an ongoing bisect (or n-sect) job.""" 24 """This class abstracts an ongoing bisect (or n-sect) job."""
25 25
26 def __init__(self, api, bisect_config, revision_class, init_revisions=True): 26 def __init__(self, api, bisect_config, revision_class, init_revisions=True):
27 """Initializes the state of a new bisect job from a dictionary. 27 """Initializes the state of a new bisect job from a dictionary.
28 28
29 Note that the initial good_rev and bad_rev MUST resolve to a commit position 29 Note that the initial good_rev and bad_rev MUST resolve to a commit position
30 in the chromium repo. 30 in the chromium repo.
31 """ 31 """
32 api.m.step('config', ['echo', str(dict(bisect_config))])
qyearsley 2015/04/03 23:20:22 Config step text would be easier to visually parse
RobertoCN 2015/04/04 00:40:48 Done.
32 super(Bisector, self).__init__() 33 super(Bisector, self).__init__()
33 self._api = api 34 self._api = api
34 self.bisect_config = bisect_config 35 self.bisect_config = bisect_config
35 self.revision_class = revision_class 36 self.revision_class = revision_class
36 37
37 # Test-only properties. 38 # Test-only properties.
38 # TODO: Replace these with proper mod_test_data 39 # TODO: Replace these with proper mod_test_data
39 self.dummy_regression_confidence = bisect_config.get( 40 self.dummy_regression_confidence = bisect_config.get(
40 'dummy_regression_confidence') 41 'dummy_regression_confidence')
41 self.dummy_builds = bisect_config.get('dummy_builds', False) 42 self.dummy_builds = bisect_config.get('dummy_builds', False)
(...skipping 439 matching lines...) Expand 10 before | Expand all | Expand 10 after
481 return 'linux_perf_tester' 482 return 'linux_perf_tester'
482 483
483 def get_builder_bot_for_this_platform(self): 484 def get_builder_bot_for_this_platform(self):
484 # TODO: Actually look at the current platform. 485 # TODO: Actually look at the current platform.
485 return 'linux_perf_bisect_builder' 486 return 'linux_perf_bisect_builder'
486 487
487 def get_platform_gs_prefix(self): 488 def get_platform_gs_prefix(self):
488 # TODO: Actually check the current platform 489 # TODO: Actually check the current platform
489 return 'gs://chrome-perf/Linux Builder/full-build-linux_' 490 return 'gs://chrome-perf/Linux Builder/full-build-linux_'
490 491
OLDNEW
« no previous file with comments | « no previous file | scripts/slave/recipe_modules/auto_bisect/example.expected/bad_deps_syntax.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698