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

Unified Diff: scripts/slave/recipe_modules/bisect_tester/api.py

Issue 2119483003: Reland "Allow multiple devices on bisects hosts " (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Remove irrelavant if statement, rename connected_devices variable 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 side-by-side diff with in-line comments
Download patch
Index: scripts/slave/recipe_modules/bisect_tester/api.py
diff --git a/scripts/slave/recipe_modules/bisect_tester/api.py b/scripts/slave/recipe_modules/bisect_tester/api.py
index 53181308999431813973d751cb20ebcf3688d76a..2766dc8ded69e9df0ee9a7a1512f83870da8658c 100644
--- a/scripts/slave/recipe_modules/bisect_tester/api.py
+++ b/scripts/slave/recipe_modules/bisect_tester/api.py
@@ -17,11 +17,21 @@ class BisectTesterApi(recipe_api.RecipeApi):
def __init__(self, **kwargs):
super(BisectTesterApi, self).__init__(**kwargs)
+ self._device_to_test = None
+
+ @property
+ def device_to_test(self):
+ return self._device_to_test
+
+ @device_to_test.setter
+ def device_to_test(self, value):
+ self._device_to_test = value
def local_test_enabled(self):
buildername = os.environ.get('BUILDBOT_BUILDERNAME')
cr_config = self.m.chromium.c
- if buildername and buildername.endswith('_bisect') and cr_config:
+ if buildername and buildername.endswith('_bisect') and cr_config or (
+ self.m.properties.get('local_test')):
return True # pragma: no cover
return False
@@ -61,6 +71,7 @@ class BisectTesterApi(recipe_api.RecipeApi):
stdout=self.m.raw_io.output(),
stdin=self.m.raw_io.input(
contents_json))
+
local_file = local_save_results.stdout.splitlines()[0].strip()
# TODO(robertocn): Look into using self.m.json.input(contents) instead of
# local_file.
« 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