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

Side by Side Diff: scripts/slave/recipes/chromium_trybot.py

Issue 1372723002: Empty per_page_values shouldn't imply test success (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Remove dead code Created 5 years, 2 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 | Annotate | Revision Log
OLDNEW
1 # Copyright 2013 The Chromium Authors. All rights reserved. 1 # Copyright 2013 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 collections 5 import collections
6 6
7 from recipe_engine.types import freeze 7 from recipe_engine.types import freeze
8 8
9 DEPS = [ 9 DEPS = [
10 'amp', 10 'amp',
(...skipping 1348 matching lines...) Expand 10 before | Expand all | Expand 10 after
1359 api.platform.name('mac') + 1359 api.platform.name('mac') +
1360 api.override_step_data('pixel_test on Intel GPU on Mac (with patch)', 1360 api.override_step_data('pixel_test on Intel GPU on Mac (with patch)',
1361 api.raw_io.output_dir({'0/results.json': ''})) + 1361 api.raw_io.output_dir({'0/results.json': ''})) +
1362 api.override_step_data('analyze', 1362 api.override_step_data('analyze',
1363 api.json.output({'status': 'Found dependency', 1363 api.json.output({'status': 'Found dependency',
1364 'targets': gpu_targets, 1364 'targets': gpu_targets,
1365 'build_targets': gpu_targets})) 1365 'build_targets': gpu_targets}))
1366 ) 1366 )
1367 1367
1368 yield ( 1368 yield (
1369 api.test('telemetry_gpu_harness_failure') +
1370 props(
1371 mastername='tryserver.chromium.linux',
1372 buildername='linux_chromium_rel_ng',
1373 ) +
1374 api.platform.name('linux') +
1375 api.override_step_data(
1376 'maps_pixel_test on NVIDIA GPU on Linux (with patch) on Linux',
1377 api.test_utils.canned_telemetry_gpu_output(
1378 passing=False, is_win=False, swarming=True,
1379 empty_per_page_values=True),
1380 retcode=255) +
1381 api.override_step_data('analyze',
1382 api.json.output({'status': 'Found dependency',
1383 'targets': gpu_targets,
1384 'build_targets': gpu_targets}))
1385 )
1386
1387 yield (
1369 api.test('telemetry_gpu_swarming_error') + 1388 api.test('telemetry_gpu_swarming_error') +
1370 props( 1389 props(
1371 mastername='tryserver.chromium.mac', 1390 mastername='tryserver.chromium.mac',
1372 buildername='mac_chromium_rel_ng', 1391 buildername='mac_chromium_rel_ng',
1373 ) + 1392 ) +
1374 api.platform.name('mac') + 1393 api.platform.name('mac') +
1375 api.override_step_data( 1394 api.override_step_data(
1376 'pixel_test on Intel GPU on Mac (with patch)', 1395 'pixel_test on Intel GPU on Mac (with patch)',
1377 api.raw_io.output_dir({ 1396 api.raw_io.output_dir({
1378 '0/results.json': '', 1397 '0/results.json': '',
(...skipping 206 matching lines...) Expand 10 before | Expand all | Expand 10 after
1585 api.test_utils.canned_test_output(passing=True)) 1604 api.test_utils.canned_test_output(passing=True))
1586 ) 1605 )
1587 1606
1588 yield ( 1607 yield (
1589 api.test('use_v8_patch_on_blink_trybot') + 1608 api.test('use_v8_patch_on_blink_trybot') +
1590 props(mastername='tryserver.blink', 1609 props(mastername='tryserver.blink',
1591 buildername='mac_blink_rel', 1610 buildername='mac_blink_rel',
1592 patch_project='v8') + 1611 patch_project='v8') +
1593 api.platform.name('mac') 1612 api.platform.name('mac')
1594 ) 1613 )
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698