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

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

Issue 2455523004: Fix recipes for Gerrit property rename (Closed)
Patch Set: Created 4 years, 2 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_compile.py
diff --git a/infra/bots/recipes/swarm_compile.py b/infra/bots/recipes/swarm_compile.py
index cf4f46fa593c4a40dfd7e888c9f2b50ab1894a22..a85e90f8e58ebde7555a319fcaa3724846da6b82 100644
--- a/infra/bots/recipes/swarm_compile.py
+++ b/infra/bots/recipes/swarm_compile.py
@@ -175,12 +175,6 @@ def GenTests(api):
api.platform('win', 64)
)
- 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(
@@ -191,7 +185,12 @@ def GenTests(api):
path_config='kitchen',
swarm_out_dir='[SWARM_OUT_DIR]',
revision='abc123',
- **gerrit_kwargs) +
+ patch_storage='gerrit') +
+ api.properties.tryserver(
+ buildername=buildername + '-Trybot',
+ gerrit_project='skia',
+ gerrit_url='https://skia-review.googlesource.com/',
+ ) +
api.platform('win', 64)
)
@@ -226,9 +225,13 @@ def GenTests(api):
swarm_out_dir='[SWARM_OUT_DIR]',
revision='abc123',
nobuildbot='True',
- issue=500,
- patchset=1,
- patch_storage='gerrit',
- gerrit='https://skia-review.googlesource.com') +
+ patch_issue=500,
+ patch_set=1,
+ patch_storage='gerrit') +
+ api.properties.tryserver(
+ buildername=buildername,
+ gerrit_project='skia',
+ gerrit_url='https://skia-review.googlesource.com/',
+ ) +
api.platform('win', 64)
)
« no previous file with comments | « infra/bots/recipe_modules/vars/api.py ('k') | infra/bots/recipes/swarm_compile.expected/buildbotless_trybot_gerrit.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698