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

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

Issue 919113002: Removed tryserver.chromium.gpu and related references (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Rebase Created 5 years, 10 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 | « scripts/slave/recipe_modules/chromium/steps.py ('k') | scripts/slave/recipe_modules/isolate/api.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: scripts/slave/recipe_modules/gpu/api.py
diff --git a/scripts/slave/recipe_modules/gpu/api.py b/scripts/slave/recipe_modules/gpu/api.py
index d79b15d184b633da8c9dc1f4a13357b2dfe02a5e..c8820eb051a4957dfc13fb3deb41a6bceb7f70ca 100644
--- a/scripts/slave/recipe_modules/gpu/api.py
+++ b/scripts/slave/recipe_modules/gpu/api.py
@@ -442,18 +442,6 @@ class GpuApi(recipe_api.RecipeApi):
return tests
- def _should_run_test(self, isolate_name):
- # Check to see if analyze.py excluded this isolate from the
- # compile targets. If so, skip running it. (It isn't possible to
- # run analyze.py on the testers because they don't check out the
- # Chromium workspace.) We prefer to not run this logic on the main
- # waterfall bots so that forgetting an isolate's hash there will
- # cause a hard failure.
- if self.m.properties['mastername'] == 'tryserver.chromium.gpu':
- return isolate_name in self.m.isolate.isolated_tests
- else:
- return True
-
def _get_gpu_suffix(self, dimensions):
if dimensions is None:
return None
@@ -496,8 +484,6 @@ class GpuApi(recipe_api.RecipeApi):
target_name = target_name or name
assert target_name.endswith('test') or target_name.endswith('tests')
- if not self._should_run_test(name):
- return
results_directory = self.m.path['slave_build'].join('gtest-results', name)
return self.m.chromium.steps.GPUGTestTest(
@@ -519,9 +505,6 @@ class GpuApi(recipe_api.RecipeApi):
enable_swarming, swarming_dimensions,
args=None, target_name=None,
extra_browser_args=None):
- if not self._should_run_test('telemetry_gpu_test'):
- return
-
test_args = ['-v', '--use-devtools-active-port']
if args:
test_args.extend(args)
« no previous file with comments | « scripts/slave/recipe_modules/chromium/steps.py ('k') | scripts/slave/recipe_modules/isolate/api.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698