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

Unified Diff: scripts/slave/recipes/android/builder.py

Issue 1989613002: New chromium.perf.fyi bots! Mac Retina and Android Galaxy S5. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Created 4 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
Index: scripts/slave/recipes/android/builder.py
diff --git a/scripts/slave/recipes/android/builder.py b/scripts/slave/recipes/android/builder.py
index a5d36c426363b5280a88d745ab76f152a77506dc..a5aed40f0df9ec137e88b15fe0f78b9c07c37a66 100644
--- a/scripts/slave/recipes/android/builder.py
+++ b/scripts/slave/recipes/android/builder.py
@@ -15,6 +15,7 @@ DEPS = [
'recipe_engine/properties',
'recipe_engine/step',
'depot_tools/tryserver',
+ 'trigger',
jbudorick 2016/05/21 12:30:44 nit: this should precede tryserver
dtu 2016/05/25 21:25:33 Done.
]
@contextmanager
@@ -64,6 +65,12 @@ BUILDERS = freeze({
'system_webview_apk',
'system_webview_shell_apk',
],
+ 'triggers': [
+ {
+ 'buildername': 'Android Galaxy S5 Perf (1)',
+ 'mastername': 'chromium.perf.fyi',
+ },
+ ],
},
'Android arm64 Builder': {
'recipe_config': 'arm64_builder_rel_mb',
@@ -209,6 +216,14 @@ def _RunStepsInternal(api, mastername, buildername, revision):
if upload_config:
droid.zip_and_upload_build(upload_config['bucket'])
+ if bot_config.get('triggers'):
jbudorick 2016/05/21 12:30:44 nit: either do trigger_config = bot_config.get
dtu 2016/05/25 21:25:33 Done.
+ api.trigger(*[{'bucket': 'master.' + b['mastername'],
+ 'builder_name': b['buildername'],
+ 'properties': {
+ 'revision': api.properties['revision'],
+ 'parent_revision': api.properties['revision']}
+ } for b in bot_config['triggers']])
+
def RunSteps(api, mastername, buildername, revision):
with api.tryserver.set_failure_hash():

Powered by Google App Engine
This is Rietveld 408576698