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

Side by Side Diff: scripts/slave/recipes/chromium_trybot.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, 1 month 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 DEPS = [ 5 DEPS = [
6 'bot_update', 6 'bot_update',
7 'chromium', 7 'chromium',
8 'chromium_tests', 8 'chromium_tests',
9 'gclient', 9 'gclient',
10 'isolate', 10 'isolate',
(...skipping 875 matching lines...) Expand 10 before | Expand all | Expand 10 after
886 886
887 return tests, bot_update_step 887 return tests, bot_update_step
888 888
889 mastername = api.properties.get('mastername') 889 mastername = api.properties.get('mastername')
890 buildername = api.properties.get('buildername') 890 buildername = api.properties.get('buildername')
891 bot_config = get_bot_config(mastername, buildername) 891 bot_config = get_bot_config(mastername, buildername)
892 api.swarming.set_default_dimension('pool', 'Chrome') 892 api.swarming.set_default_dimension('pool', 'Chrome')
893 api.swarming.default_priority = build_to_priority(api.properties) 893 api.swarming.default_priority = build_to_priority(api.properties)
894 api.swarming.add_default_tag('project:chromium') 894 api.swarming.add_default_tag('project:chromium')
895 api.swarming.add_default_tag('purpose:pre-commit') 895 api.swarming.add_default_tag('purpose:pre-commit')
896 # TODO(maruel): Soon! 896 api.swarming.default_idempotent = True
897 # api.swarming.default_idempotent = True
898 897
899 # Differentiate between try jobs and CQ jobs. Always find out who made the CL. 898 # Differentiate between try jobs and CQ jobs. Always find out who made the CL.
900 requestor = api.properties.get('requestor') 899 requestor = api.properties.get('requestor')
901 if requestor == 'commit-bot@chromium.org': 900 if requestor == 'commit-bot@chromium.org':
902 api.swarming.add_default_tag('purpose:CQ') 901 api.swarming.add_default_tag('purpose:CQ')
903 blamelist = api.properties.get('blamelist') 902 blamelist = api.properties.get('blamelist')
904 if len(blamelist) == 1: 903 if len(blamelist) == 1:
905 requestor = blamelist[0] 904 requestor = blamelist[0]
906 else: 905 else:
907 api.swarming.add_default_tag('purpose:ManualTS') 906 api.swarming.add_default_tag('purpose:ManualTS')
(...skipping 638 matching lines...) Expand 10 before | Expand all | Expand 10 after
1546 }, 1545 },
1547 ], 1546 ],
1548 }) 1547 })
1549 ) + 1548 ) +
1550 api.override_step_data( 1549 api.override_step_data(
1551 'analyze', 1550 'analyze',
1552 api.json.output({'status': 'Found dependency', 1551 api.json.output({'status': 'Found dependency',
1553 'targets': ['browser_tests', 'base_unittests'], 1552 'targets': ['browser_tests', 'base_unittests'],
1554 'build_targets': ['base_unittests']})) 1553 'build_targets': ['base_unittests']}))
1555 ) 1554 )
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698