Chromium Code Reviews| Index: infra/bots/recipes/swarm_test.py |
| diff --git a/infra/bots/recipes/swarm_test.py b/infra/bots/recipes/swarm_test.py |
| index 1ceba3a1a26e26d5b7efcd5457da9bca3649e684..38793491f4c1540bd56e76dc0814b8af7cc61ab7 100644 |
| --- a/infra/bots/recipes/swarm_test.py |
| +++ b/infra/bots/recipes/swarm_test.py |
| @@ -430,10 +430,18 @@ def test_steps(api): |
| 'build_number', api.vars.build_number, |
| ] |
| if api.vars.is_trybot: |
| - properties.extend([ |
| - 'issue', api.vars.issue, |
| - 'patchset', api.vars.patchset, |
| - ]) |
| + if api.vars.patch_storage == 'gerrit': |
| + properties.extend([ |
| + 'patch_storage', api.vars.patch_storage, |
| + 'repository', api.vars.gerrit_repository, |
| + 'event.patchSet.ref', api.vars.gerrit_ref, |
| + 'event.change.number', api.vars.gerrit_issue, |
|
rmistry
2016/08/23 14:00:41
Actually I do not think this is right. This needs
|
| + ]) |
| + else: |
| + properties.extend([ |
| + 'issue', api.vars.issue, |
| + 'patchset', api.vars.patchset, |
| + ]) |
| args = [ |
| 'dm', |
| @@ -860,3 +868,23 @@ def GenTests(api): |
| ) + |
| api.platform('win', 64) |
| ) |
| + |
| + builder = 'Test-Ubuntu-GCC-GCE-CPU-AVX2-x86-Debug-Trybot' |
| + gerrit_kwargs = { |
| + 'patch_storage': 'gerrit', |
| + 'repository': 'skia', |
| + 'event.patchSet.ref': 'refs/changes/00/2100/2', |
| + 'event.change.number': '2100', |
| + } |
| + yield ( |
| + api.test('recipe_with_gerrit_patch') + |
| + api.properties( |
| + buildername=builder, |
| + mastername='client.skia', |
| + slavename='skiabot-linux-swarm-000', |
| + buildnumber=5, |
| + path_config='kitchen', |
| + swarm_out_dir='[SWARM_OUT_DIR]', |
| + revision='abc123', |
| + **gerrit_kwargs) |
| + ) |