Double Builds for Blimp Integration Test Infrastructure on FYI bot

scripts/slave/recipes/blimp/integration.py

+# 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 Android Tester': {
+        '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': {
+          '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')

[jbudorick, 2016/10/06 13:34:30]

ANDROID_BUILD and LINUX_BUILD should not be capitalized

[shenghuazhang, 2016/10/07 02:13:26]

Done.

+  LINUX_BUILD = api.path['slave_build'].join('src', 'out-linux', 'Debug')
+  mastername = api.properties.get('mastername')
+  buildername, bot_config = api.chromium.configure_bot(BUILDERS)

[jbudorick, 2016/10/06 13:34:30]

After looking at api.chromium.configure_bot, I'm not sure that this will work w/
the configuration split between client_config and engine_config.

+  api.chromium.ensure_goma()
+  api.chromium.runhooks()
+  api.chromium.run_mb(mastername="chromium.fyi",

[jbudorick, 2016/10/06 13:34:30]

nit:  separate the android build and the linux build. run mb for linux, then
build linux, then run mb for android, then build android.

[shenghuazhang, 2016/10/08 00:21:08]

Done.

+                      buildername="Blimp Android Test - Android",
+                      build_dir=ANDROID_BUILD)
+  api.chromium.run_mb(mastername="chromium.fyi",
+                      buildername="Blimp Android Test - Linux",
+                      build_dir=LINUX_BUILD)
+  api.chromium.compile(targets=['blimp', 'chrome_public_apk'], out_dir=ANDROID_BUILD)

[jbudorick, 2016/10/06 13:34:30]

80 chars, please

[shenghuazhang, 2016/10/07 02:13:26]

Done.

+  api.chromium.compile(targets=['blimp'], out_dir=LINUX_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 Android Tester',
+        mastername='chromium.fyi')
+  )