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

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

Issue 1502313004: (Re-land) Switch GPU FYI bots to new Telemetry GPU isolate. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Created 5 years 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 | « no previous file | scripts/slave/recipes/gpu/build_and_test.expected/killall_gnome_keyring_failure.json » ('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 b9ff5c5594b6fc0ffa86979db22c58298ee6b455..d59f12251d7aa8d75e61c736dabcb320dd8e3416 100644
--- a/scripts/slave/recipe_modules/gpu/api.py
+++ b/scripts/slave/recipe_modules/gpu/api.py
@@ -600,9 +600,30 @@ class GpuApi(recipe_api.RecipeApi):
extra_browser_args_string += ' ' + ' '.join(extra_browser_args)
test_args.append(extra_browser_args_string)
- return self.m.chromium_tests.steps.TelemetryGPUTest(
- name, chrome_revision, webkit_revision, args=test_args,
- target_name=target_name, enable_swarming=enable_swarming,
- swarming_dimensions=swarming_dimensions,
- master_class_name=self._master_class_name_for_testing,
- swarming_extra_suffix=self._get_gpu_suffix(swarming_dimensions))
+ # Run the new Telemetry GPU test isolate on the FYI waterfall.
+ # Once it's running well on all platforms, the old isolate, and
+ # the code which launches it, will be removed.
+ if self.is_fyi_waterfall:
+ # The step name must end in 'test' or 'tests' in order for the
+ # results to automatically show up on the flakiness dashboard.
+ # (At least, this was true some time ago.) Continue to use this
+ # naming convention for the time being to minimize changes.
+ step_name = name
+ if not (step_name.endswith('test') or step_name.endswith('tests')):
+ step_name = '%s_tests' % step_name
+ # Prepend Telemetry GPU-specific flags.
+ benchmark_name = target_name or name
+ prefix_args = [
+ benchmark_name, '--show-stdout',
+ '--browser=%s' % self.m.chromium.c.build_config_fs.lower() ]
+ return self.m.chromium_tests.steps.LocalIsolatedScriptTest(
+ step_name, args=prefix_args + test_args,
+ target_name='telemetry_gpu_new_test',
+ override_compile_targets=['telemetry_gpu_new_test_run'])
+ else:
+ return self.m.chromium_tests.steps.TelemetryGPUTest(
+ name, chrome_revision, webkit_revision, args=test_args,
+ target_name=target_name, enable_swarming=enable_swarming,
+ swarming_dimensions=swarming_dimensions,
+ master_class_name=self._master_class_name_for_testing,
+ swarming_extra_suffix=self._get_gpu_suffix(swarming_dimensions))
« no previous file with comments | « no previous file | scripts/slave/recipes/gpu/build_and_test.expected/killall_gnome_keyring_failure.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698