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

Unified Diff: git_cl.py

Issue 1143573002: Allow properties to be overridden in buildbucket tries. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/depot_tools.git@master
Patch Set: Review Created 5 years, 7 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: git_cl.py
diff --git a/git_cl.py b/git_cl.py
index fd80fb701cfd2822aef83ecb1aa34db5b08280c5..f3aa2ab5ae972872605c01d26bfa22e1f742a101 100755
--- a/git_cl.py
+++ b/git_cl.py
@@ -222,7 +222,8 @@ def _prefix_master(master):
return '%s%s' % (prefix, master)
-def trigger_try_jobs(auth_config, changelist, options, masters, category):
+def trigger_try_jobs(auth_config, changelist, options, masters, category,
+ override_properties=None):
rietveld_url = settings.GetDefaultServerUrl()
rietveld_host = urlparse.urlparse(rietveld_url).hostname
authenticator = auth.get_authenticator_for_host(rietveld_host, auth_config)
@@ -266,6 +267,8 @@ def trigger_try_jobs(auth_config, changelist, options, masters, category):
'testfilter': tests,
},
}
+ if override_properties:
+ parameters['properties'].update(override_properties)
if options.clobber:
parameters['properties']['clobber'] = True
batch_req_body['builds'].append(
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698