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

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

Issue 653793003: Mark all chromium test swarming task requests as idempotent. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Fix for skia Created 6 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
« no previous file with comments | « no previous file | scripts/slave/recipes/chromium.expected/full_chromium_chromiumos_Linux_ChromiumOS_Tests__1_.json » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 DEPS = [ 5 DEPS = [
6 'adb', 6 'adb',
7 'chromium', 7 'chromium',
8 'chromium_android', 8 'chromium_android',
9 'chromium_tests', 9 'chromium_tests',
10 'isolate', 10 'isolate',
(...skipping 19 matching lines...) Expand all
30 tests = api.chromium_tests.tests_for_builder( 30 tests = api.chromium_tests.tests_for_builder(
31 mastername, buildername, update_step, master_dict) 31 mastername, buildername, update_step, master_dict)
32 32
33 if not tests: 33 if not tests:
34 return 34 return
35 35
36 api.swarming.default_priority = 25 36 api.swarming.default_priority = 25
37 api.swarming.set_default_dimension('pool', 'Chrome') 37 api.swarming.set_default_dimension('pool', 'Chrome')
38 api.swarming.add_default_tag('project:chromium') 38 api.swarming.add_default_tag('project:chromium')
39 api.swarming.add_default_tag('purpose:CI') 39 api.swarming.add_default_tag('purpose:CI')
40 # TODO(maruel): Soon! 40 api.swarming.default_idempotent = True
41 # api.swarming.default_idempotent = True
42 41
43 def test_runner(): 42 def test_runner():
44 failed_tests = [] 43 failed_tests = []
45 #TODO(martiniss) convert loop 44 #TODO(martiniss) convert loop
46 for t in tests: 45 for t in tests:
47 try: 46 try:
48 t.pre_run(api, '') 47 t.pre_run(api, '')
49 except api.step.StepFailure: # pragma: no cover 48 except api.step.StepFailure: # pragma: no cover
50 failed_tests.append(t) 49 failed_tests.append(t)
51 for t in tests: 50 for t in tests:
(...skipping 301 matching lines...) Expand 10 before | Expand all | Expand 10 after
353 yield ( 352 yield (
354 api.test('perf_test_profile_failure') + 353 api.test('perf_test_profile_failure') +
355 api.properties.generic(mastername='chromium.perf', 354 api.properties.generic(mastername='chromium.perf',
356 buildername='Linux Perf (1)', 355 buildername='Linux Perf (1)',
357 parent_buildername='Linux Builder', 356 parent_buildername='Linux Builder',
358 buildnumber=0) + 357 buildnumber=0) +
359 api.platform('linux', 64) + 358 api.platform('linux', 64) +
360 api.override_step_data( 359 api.override_step_data(
361 'blink_perf.all.release', retcode=1) 360 'blink_perf.all.release', retcode=1)
362 ) 361 )
OLDNEW
« no previous file with comments | « no previous file | scripts/slave/recipes/chromium.expected/full_chromium_chromiumos_Linux_ChromiumOS_Tests__1_.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698