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

Side by Side Diff: scripts/slave/recipe_modules/bisect_tester/api.py

Issue 2112553005: Revert of Redo test after step failure if multiple android devices are connected to lin… (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Created 4 years, 5 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 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 json 5 import json
6 import os 6 import os
7 7
8 from recipe_engine import recipe_api 8 from recipe_engine import recipe_api
9 from . import perf_test 9 from . import perf_test
10 10
11 BUCKET = 'chrome-perf' 11 BUCKET = 'chrome-perf'
12 RESULTS_GS_DIR = 'bisect-results' 12 RESULTS_GS_DIR = 'bisect-results'
13 13
14 14
15 class BisectTesterApi(recipe_api.RecipeApi): 15 class BisectTesterApi(recipe_api.RecipeApi):
16 """A module for the bisect tester bot using the chromium recipe.""" 16 """A module for the bisect tester bot using the chromium recipe."""
17 17
18 def __init__(self, **kwargs): 18 def __init__(self, **kwargs):
19 super(BisectTesterApi, self).__init__(**kwargs) 19 super(BisectTesterApi, self).__init__(**kwargs)
20 self.device_to_test = None
21 self.devices_tested = []
22 20
23 def local_test_enabled(self): 21 def local_test_enabled(self):
24 buildername = os.environ.get('BUILDBOT_BUILDERNAME') 22 buildername = os.environ.get('BUILDBOT_BUILDERNAME')
25 cr_config = self.m.chromium.c 23 cr_config = self.m.chromium.c
26 if buildername and buildername.endswith('_bisect') and cr_config: 24 if buildername and buildername.endswith('_bisect') and cr_config:
27 return True # pragma: no cover 25 return True # pragma: no cover
28 return False 26 return False
29 27
30 def load_config_from_dict(self, bisect_config): 28 def load_config_from_dict(self, bisect_config):
31 """Copies the required configuration keys to a new dict.""" 29 """Copies the required configuration keys to a new dict."""
(...skipping 53 matching lines...) Expand 10 before | Expand all | Expand 10 after
85 self.m.properties['mastername'], 83 self.m.properties['mastername'],
86 self.m.properties['buildername'], 84 self.m.properties['buildername'],
87 self.m.properties['buildnumber']) 85 self.m.properties['buildnumber'])
88 local_save_results = self.m.python('saving url to temp file', 86 local_save_results = self.m.python('saving url to temp file',
89 self.resource('put_temp.py'), 87 self.resource('put_temp.py'),
90 stdout=self.m.raw_io.output(), 88 stdout=self.m.raw_io.output(),
91 stdin=self.m.raw_io.input(url)) 89 stdin=self.m.raw_io.input(url))
92 local_file = local_save_results.stdout.splitlines()[0].strip() 90 local_file = local_save_results.stdout.splitlines()[0].strip()
93 self.m.gsutil.upload( 91 self.m.gsutil.upload(
94 local_file, BUCKET, gs_filename, name=str(gs_filename)) 92 local_file, BUCKET, gs_filename, name=str(gs_filename))
OLDNEW
« no previous file with comments | « scripts/slave/recipe_modules/auto_bisect/local_bisect.py ('k') | scripts/slave/recipe_modules/bisect_tester/perf_test.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698