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

Unified Diff: scripts/slave/recipes/gpu/download_and_test.py

Issue 1518533003: (Re-land) Switch remaining bots to new Telemetry GPU isolate. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Rebased. Undid TODO. 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
Index: scripts/slave/recipes/gpu/download_and_test.py
diff --git a/scripts/slave/recipes/gpu/download_and_test.py b/scripts/slave/recipes/gpu/download_and_test.py
index 4d15a5d0eb0db16a84305d0df5281bb825ba0633..b179bdd69232829840fb41d2602e1ccbf12b547f 100644
--- a/scripts/slave/recipes/gpu/download_and_test.py
+++ b/scripts/slave/recipes/gpu/download_and_test.py
@@ -88,8 +88,9 @@ def GenTests(api):
api.platform.name('linux') +
api.step_data('content_gl_tests', retcode=1) +
api.step_data('maps_pixel_test',
- api.test_utils.canned_telemetry_gpu_output(
- passing=False, is_win=False, swarming=False))
+ api.test_utils.canned_isolated_script_output(
+ passing=False, is_win=False, swarming=False,
+ isolated_script_passing=False))
)
yield (
@@ -97,19 +98,14 @@ def GenTests(api):
props('linux', 'Release') +
api.platform.name('linux') +
api.step_data('maps_pixel_test',
- api.test_utils.canned_telemetry_gpu_output(
- passing=False, is_win=False, swarming=False,
- empty_per_page_values=True),
+ api.test_utils.canned_isolated_script_output(
+ passing=False, is_win=False, swarming=False),
retcode=255)
)
- linux_fyi_hashes = api.gpu.dummy_linux_fyi_swarm_hashes
-
yield (
- api.test('telemetry_gpu_fyi_test_harness_failure') +
- props('linux', 'Release',
- hashes=linux_fyi_hashes,
- mastername='chromium.gpu.fyi') +
+ api.test('telemetry_gpu_test_harness_failure_zero_retcode') +
+ props('linux', 'Release') +
api.platform.name('linux') +
api.override_step_data('maps_pixel_test',
api.test_utils.canned_isolated_script_output(
@@ -126,6 +122,8 @@ def GenTests(api):
api.platform.name('win')
)
+ linux_fyi_hashes = api.gpu.dummy_linux_fyi_swarm_hashes
+
yield (
api.test('linux_release_deqp') +
named_props('linux', 'Release', 'linux release dEQP tester',

Powered by Google App Engine
This is Rietveld 408576698