Index: git_cl.py |
diff --git a/git_cl.py b/git_cl.py |
index 2f88d0b4a85579a31c0af06d831ab06042d4c513..c212300f5918403463246c5fa2f4ceb404b60c3d 100755 |
--- a/git_cl.py |
+++ b/git_cl.py |
@@ -245,7 +245,9 @@ def trigger_luci_job(changelist, masters, options): |
issue = changelist.GetIssue() |
patchset = changelist.GetMostRecentPatchset() |
for builders_and_tests in sorted(masters.itervalues()): |
- for builder in sorted(builders_and_tests.iterkeys()): |
+ # TODO(hinoka et al): add support for other properties. |
+ # Currently, this completely ignores testfilter and other properties. |
+ for builder in sorted(builders_and_tests): |
luci_trigger.trigger( |
builder, 'HEAD', issue, patchset, issue_props['project']) |
@@ -292,9 +294,10 @@ def trigger_try_jobs(auth_config, changelist, options, masters, category): |
'patchset': patchset, |
'reason': options.name, |
'rietveld': rietveld_url, |
- 'testfilter': tests, |
}, |
} |
+ if tests: |
+ parameters['properties']['testfilter'] = tests |
if properties: |
parameters['properties'].update(properties) |
if options.clobber: |
@@ -3216,7 +3219,7 @@ def CMDtry(parser, args): |
elif ',' in bot: |
parser.error('Specify one bot per --bot flag') |
else: |
- builders_and_tests.setdefault(bot, []).append('defaulttests') |
+ builders_and_tests.setdefault(bot, []) |
for bot, tests in new_style: |
builders_and_tests.setdefault(bot, []).extend(tests) |