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

Unified Diff: net/PRESUBMIT.py

Issue 194903007: Convert most Chromium presubmit files to use new multiple-tryserver-compatible protocol (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 9 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 side-by-side diff with in-line comments
Download patch
Index: net/PRESUBMIT.py
diff --git a/net/PRESUBMIT.py b/net/PRESUBMIT.py
index 3a2f21323b6c7f844a6044305c539b01391194d3..b3dce0115afb814afeb0c4b418b63c302fca26f2 100644
--- a/net/PRESUBMIT.py
+++ b/net/PRESUBMIT.py
@@ -8,14 +8,17 @@ See http://dev.chromium.org/developers/how-tos/depottools/presubmit-scripts
for more details on the presubmit API built into gcl.
"""
-def GetPreferredTrySlaves(project, change):
- slaves = [
- 'linux_rel:sync_integration_tests',
- 'mac_rel:sync_integration_tests',
- 'win_rel:sync_integration_tests',
- ]
+def GetPreferredTryMasters(project, change):
+ masters = {
+ 'tryserver.chromium': {
+ 'linux_rel': set(['sync_integration_tests']),
+ 'mac_rel': set(['sync_integration_tests']),
+ 'win_rel': set(['sync_integration_tests']),
+ }
+ }
# Changes that touch NSS files will likely need a corresponding OpenSSL edit.
# Conveniently, this one glob also matches _openssl.* changes too.
if any('nss' in f.LocalPath() for f in change.AffectedFiles()):
- slaves.append('linux_redux')
- return slaves
+ masters['tryserver.chromium'].getdefault(
M-A Ruel 2014/03/11 20:54:49 you probably mean setdefault() but it's not necess
Paweł Hajdan Jr. 2014/03/11 21:25:41 Used setdefault.
+ 'linux_redux', set()).add('defaulttests')
+ return masters

Powered by Google App Engine
This is Rietveld 408576698