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

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

Issue 1777863005: Recipe changes to move webview perf bots to the android/perf recipe (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Rebase. Created 4 years, 8 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 a29fa30561a1917be4de3723d1da0aca94ce93e8..5fad5b98d21768454e6c5198a4ecce42a2ecf56b 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',
]
@contextmanager
@@ -64,6 +65,20 @@ BUILDERS = freeze({
'system_webview_apk',
'system_webview_shell_apk',
],
+ 'triggers': [
+ {
+ 'buildername': 'WebView Nexus5 Perf (1)',
+ 'mastername': 'master.chromium.perf.fyi',
+ },
+ {
+ 'buildername': 'WebView Nexus5X Perf (1)',
+ 'mastername': 'master.chromium.perf.fyi',
+ },
+ {
+ 'buildername': 'WebView Nexus5X Perf (2)',
+ 'mastername': 'master.chromium.perf.fyi',
+ }
+ ]
},
'Android arm64 Builder': {
'recipe_config': 'arm64_builder_rel_mb',
@@ -199,6 +214,13 @@ def _RunStepsInternal(api, mastername, buildername, revision):
if upload_config:
droid.zip_and_upload_build(upload_config['bucket'])
+ # TODO(mikecase): Remove this logic once WebView bots are on chromium.perf
+ if bot_config.get('triggers'):
+ api.trigger(*[{'bucket': b['mastername'],
+ 'buildername': b['buildername'],
+ 'mastername': b['mastername'],
+ 'revision': api.properties['revision']}
nodir 2016/04/20 18:03:40 The structure of this dict should be something lik
+ 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