Added swarming step to the Blink trybots

scripts/slave/recipe_modules/chromium_tests/api.py

 # Copyright 2014 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.
 
 import copy
 
 from slave import recipe_api
 
 
 # Different types of builds this recipe module can do.
@@ skipping 449 matching lines @@
     # but adding |matching_exes| makes determing if conditional tests are
     # necessary easier. For example, if we didn't do this we could end up
     # with chrome_run as a compile_target and not chrome (since chrome_run
     # depends upon chrome). This results in not picking up
     # NaclIntegrationTest as it depends upon chrome not chrome_run.
     compile_targets = list(set(self.m.filter.matching_exes +
                                self.m.filter.compile_targets))
 
     return True, self.m.filter.matching_exes, compile_targets
 
+  def configure_swarming(self, project_name, precommit):
+    """Configures default swarming dimensions and tags.
+
+    Args:
+      project_name: Lowercase name of the project, e.g. "blink", "chromium".
+      precommit: Boolean flag to indicate whether the tests are running before
+          the changes are commited.
+    """
+    self.m.swarming.set_default_dimension('pool', 'Chrome')
+    self.m.swarming.add_default_tag('project:%s' % project_name)
+    self.m.swarming.default_idempotent = True
+
+    if precommit:
+      self.m.swarming.add_default_tag('purpose:pre-commit')
+      requester = self.m.properties.get('requester')

[Ken Russell (switch to Gerrit), 2014/10/29 19:16:16]

Wow. One of the tests of the "requester" build property has been broken all
along due to a typo ("requestor").

[Sergiy Byelozyorov, 2014/10/29 20:04:24]

Yeah. I've actually grepped whole repo for "requestor" to make sure there are no
other places with this typo.

+      if requester == 'commit-bot@chromium.org':
+        self.m.swarming.default_priority = 30
+        self.m.swarming.add_default_tag('purpose:CQ')
+        blamelist = self.m.properties.get('blamelist')
+        if len(blamelist) == 1:
+          requester = blamelist[0]
+      else:
+        self.m.swarming.default_priority = 50
+        self.m.swarming.add_default_tag('purpose:ManualTS')
+      self.m.swarming.default_user = requester
+    else:
+      self.m.swarming.add_default_tag('purpose:post-commit')
+      self.m.swarming.add_default_tag('purpose:CI')
+      self.m.swarming.default_priority = 25
+
   # Used to build the Google Storage archive url.
   #
   # We need to special-case the logic for composing the archive url for a couple
   # of masters. That has been moved outside of the compile method.
   #
   # Special-cased masters:
   #   'chromium.perf':
   #     exclude the name of the master from the url.
   #   'tryserver.chromium.perf':
   #     return nothing so that the archive url specified in factory_properties
   #     (as set on the master's configuration) is used instead.
   def _build_gs_archive_url(self, mastername, master_config):
     if mastername == 'chromium.perf':
       return self.m.archive.legacy_upload_url(
           master_config.get('build_gs_bucket'),
           extra_url_components=None)
     elif mastername == 'tryserver.chromium.perf':
       return  None
     else:
       return self.m.archive.legacy_upload_url(
           master_config.get('build_gs_bucket'),
           extra_url_components=self.m.properties['mastername'])