CrOS: Switch tryserver over to recipes, Gitiles.

scripts/slave/recipes/cros/cbuildbot_tryjob.py

 # Copyright 2013 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 common import cros_chromite
 
 DEPS = [
   'chromite',
   'gitiles',
   'properties',
 ]
 
 # Map master name to 'chromite' configuration name.
 _MASTER_CONFIG_MAP = {
     'chromiumos.tryserver': {
-      'master_config': 'external',
+      'master_config': 'chromiumos_tryserver',
       'variants': {
-        'etc': ['chromeos_tryserver_etc'],
+        'etc': ['chromiumos_tryserver_etc'],
       },
     },
 }
 
 
 # Testing: Tryjob data file JSON.
 _TRYJOB_DATA = """
 {
   "name": "12345",
   "email": "testauthor@fake.chromium.org",
   "extra_args": [
     "--timeout",
     "14400",
     "--remote-trybot",
     "--remote-version=4"
   ]
 }
 """
 
 
 def RunSteps(api):
   # The 'cbuildbot' config name to build is the name of the builder.
-  cbb_config_name = api.properties.get('buildername')
+  #
+  # TODO(dnj): After we fully switch to BuildBucket scheduling, load the config
+  #            name from the BuildBucket job instead of `cbb_config` build
+  #            property. We can't do this yet b/c the job description can
+  #            specify multiple configs in one tryjob, so there's no way for us
+  #            to know which one we are.
+  cbb_config_name = api.properties.get('cbb_config')
+  assert cbb_config_name, "No configuration name specified."
 
   cbb = cros_chromite.Get()
   cbb_config = cbb.get(cbb_config_name)
 
   # Apply our generic configuration.
   api.chromite.configure(
       api.properties,
       _MASTER_CONFIG_MAP)
   api.chromite.c.cbb.config = cbb_config_name
 
-  # Determine our build directory name.
-  namebase = cbb_config_name
-  if cbb_config:
-    namebase = 'internal' if cbb_config.get('internal') else 'external'
-  api.chromite.c.cbb.builddir = '%s_master' % (namebase,)
+  repository = api.properties.get('repository')
+  revision = api.properties.get('revision')
+  assert repository, "A repository must be specified."
+  assert revision, "A revision must be specified."
+  assert api.chromite.check_repository('tryjob', repository), (
+      "Refusing to query unknown tryjob repository: %s" % (repository,))
+  # If we are a tryjob, add parameters specified in the description.

[martiniss, 2015/09/15 17:58:58]

Why does it say "if"? This is a tryjob recipe, correct?

[dnj, 2015/09/15 18:33:27]

Copy/pasted comment; fixed/done.

+  tryjob_args = api.chromite.load_try_job(repository, revision)
+
+  # Determine our build directory name based on whether this build is internal
+  # or external.
+  if not api.chromite.c.cbb.builddir:
+    namebase = cbb_config_name
+    if cbb_config:
+      namebase = 'internal' if cbb_config.get('internal') else 'external'
+    api.chromite.c.cbb.builddir = '%s_master' % (namebase,)
 
   # Run our 'cbuildbot'.
-  api.chromite.run_cbuildbot(tryjob=True)
+  api.chromite.run_cbuildbot(args=tryjob_args)
 
 
 def GenTests(api):
   # Test a CrOS tryjob.
   yield (
       api.test('basic')
       + api.properties(
           mastername='chromiumos.tryserver',
-          buildername='x86-generic-full',
+          buildername='full',
           slavename='test',
           repository='https://chromium.googlesource.com/chromiumos/tryjobs.git',
           revision=api.gitiles.make_hash('test'),
+          cbb_config='x86-generic-full',
       )
       + api.step_data(
           'Fetch tryjob commit',
           api.gitiles.make_commit_test_data(
               'test',
               '\n'.join([
                   'Commit message!',
               ]),
               new_files=['user/user.12345'],
           ),
       )
       + api.step_data(
           'Fetch tryjob descriptor (user/user.12345)',
           api.gitiles.make_encoded_file(_TRYJOB_DATA)
       )
   )
 
   # Test an 'etc' job (no Chromite config).
   yield (
       api.test('etc')
       + api.properties(
           mastername='chromiumos.tryserver',
           buildername='etc',
           slavename='test',
-          cbb_variant='etc',
           repository='https://chromium.googlesource.com/chromiumos/tryjobs.git',
           revision=api.gitiles.make_hash('test'),
+          cbb_config='fakeboard-paladin',
+          cbb_variant='etc',
       )
       + api.step_data(
           'Fetch tryjob commit',
           api.gitiles.make_commit_test_data(
               'test',
               '\n'.join([
                   'Commit message!',
               ]),
               new_files=['user/user.12345'],
           ),
       )
       + api.step_data(
           'Fetch tryjob descriptor (user/user.12345)',
           api.gitiles.make_encoded_file(_TRYJOB_DATA)
       )
   )
 
   # Test an invalid CrOS tryjob (no files in commit).
   yield (
       api.test('basic_no_files_in_commit')
       + api.properties(
           mastername='chromiumos.tryserver',
-          buildername='x86-generic-full',
+          buildername='full',
           slavename='test',
           repository='https://chromium.googlesource.com/chromiumos/tryjobs.git',
           revision=api.gitiles.make_hash('test'),
+          cbb_config='x86-generic-full',
       )
       + api.step_data(
           'Fetch tryjob commit',
           api.gitiles.make_commit_test_data(
               'test',
               '\n'.join([
                   'Commit message!',
               ]),
           ),
       )
   )