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

Unified Diff: scripts/slave/recipes/blimp/integration.py

Issue 2390753005: Double Builds for Blimp Integration Test Infrastructure on FYI bot (Closed)
Patch Set: Dirk comment 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: scripts/slave/recipes/blimp/integration.py
diff --git a/scripts/slave/recipes/blimp/integration.py b/scripts/slave/recipes/blimp/integration.py
new file mode 100644
index 0000000000000000000000000000000000000000..b4bf95ffe071a29bb6eb7ddfbd386cb4ac6a7df8
--- /dev/null
+++ b/scripts/slave/recipes/blimp/integration.py
@@ -0,0 +1,86 @@
+# Copyright 2016 The Chromium Authors. All rights reserved.
+# Use of this source code is governed by a BSD-style license that can be
+# found in the LICENSE file.
+
+from recipe_engine import recipe_api
+from recipe_engine.recipe_api import Property
+from recipe_engine.types import freeze
+
+DEPS = [
+ 'depot_tools/bot_update',
+ 'chromium',
+ 'chromium_android',
+ 'chromium_swarming',
+ 'chromium_tests',
+ 'recipe_engine/json',
+ 'recipe_engine/path',
+ 'recipe_engine/platform',
+ 'recipe_engine/properties',
+ 'recipe_engine/python',
+ 'recipe_engine/step',
+ 'swarming',
+ 'test_results',
+ 'test_utils',
+]
+
+BUILDERS = freeze({
+ 'chromium.fyi': {
+ 'builders':{
+ 'Blimp Client Engine Integration': {
+ 'client_config': {
+ 'android_config': 'main_builder_rel_mb',
+ 'chromium_config': 'android',
+ 'chromium_config_kwargs': {
+ 'BUILD_CONFIG': 'Debug',
+ 'TARGET_BITS': 64,
+ 'TARGET_PLATFORM': 'android',
+ },
+ 'gclient_apply_config': ['android'],
+ 'gclient_config': 'chromium',
+ },
+ 'engine_config': {
mikecase (-- gone --) 2016/10/10 18:15:53 It doesnt look like these configs are being used.
Dirk Pranke 2016/10/11 00:18:23 Right, I don't think you can nest configs like thi
+ 'chromium_apply_config': [ 'mb' ],
+ 'chromium_config': 'chromium',
+ 'chromium_config_kwargs': {
+ 'BUILD_CONFIG': 'Debug',
+ 'TARGET_BITS': 64,
+ 'TARGET_PLATFORM': 'linux',
+ },
+ 'gclient_config': 'chromium',
+ },
+ },
+ },
+ },
+})
+
+
+def RunSteps(api):
+ android_build = api.path['slave_build'].join('src', 'out-android', 'Debug')
+ linux_build = api.path['slave_build'].join('src', 'out-linux', 'Debug')
+ mastername = api.properties.get('mastername')
+ buildername, bot_config = api.chromium.configure_bot(BUILDERS)
mikecase (-- gone --) 2016/10/10 18:15:53 woah, cool. Didn't know this API existed.
+ api.chromium.ensure_goma()
+ api.chromium.runhooks()
+ api.chromium.run_mb(mastername=mastername,
+ buildername=buildername,
+ build_dir=linux_build,
+ phase='engine')
+ api.chromium.compile(targets=['blimp'],
+ out_dir=linux_build)
+ api.chromium.run_mb(mastername=mastername,
+ buildername=buildername,
+ build_dir=android_build,
+ phase='client')
+ api.chromium.compile(targets=['blimp', 'chrome_public_apk'],
+ out_dir=android_build)
+
+
+def GenTests(api):
+ sanitize = lambda s: ''.join(c if c.isalnum() else '_' for c in s)
+
+ yield (
+ api.test('%s_test_pass' % sanitize('Blimp Android Tester')) +
+ api.properties.generic(
+ buildername='Blimp Client Engine Integration',
+ mastername='chromium.fyi')
+ )

Powered by Google App Engine
This is Rietveld 408576698