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

Unified Diff: infra/bots/recipes/swarm_test.py

Issue 2263323002: Apply gerrit ref if it is a Gerrit patch (Closed) Base URL: https://skia.googlesource.com/skia@master
Patch Set: Align Created 4 years, 4 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: 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)
+ )

Powered by Google App Engine
This is Rietveld 408576698