Archive Linux perf builds for manual bisect

scripts/slave/recipe_modules/archive/api.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.
 
 import re
 
 from recipe_engine import recipe_api
+from . import perf_test_files
 
 
 # TODO(machenbach): Chromium specific data should move out of the archive
 # module, into e.g. the chromium test configs.
 EXCLUDED_FILES_ALL_PLATFORMS = [
   '.landmines',
   '.ninja_deps',
   '.ninja_log',
   'gen',
   'obj',
@@ skipping 76 matching lines @@
       build_revision=None, cros_board=None, package_dsym_files=False,
       exclude_files=None, **kwargs):
     """Returns a step invoking zip_build.py to zip up a Chromium build.
        If build_url is specified, also uploads the build."""
     args = [
         '--show-path',
         'python',
         self.package_repo_resource('scripts', 'slave', 'zip_build.py'),
         '--target', target,
     ]
+

[dimu1, 2016/07/11 17:47:09]

nit: remove empty line

[miimnk, 2016/07/11 22:08:47]

Done.

     if build_url or 'build_archive_url' in self.m.properties:
       args.extend(['--build-url',
                    build_url or self.m.properties['build_archive_url']])
     if build_revision:
       args.extend(['--build_revision', build_revision])
     elif src_dir:
       args.extend(['--src-dir', src_dir])
     if cros_board:
       args.extend(['--cros-board', cros_board])
     if package_dsym_files:
       args.append('--package-dsym-files')
     if exclude_files:
       args.extend(['--exclude-files', exclude_files])
     if 'gs_acl' in self.m.properties:
       args.extend(['--gs-acl', self.m.properties['gs_acl']])
+    if not kwargs.pop("includePerfTestFiles", True):
+      inclusions = ",".join(perf_test_files.FILES)

[stgao, 2016/07/11 19:04:27]

style nit: single quote instead of double quote.

[miimnk, 2016/07/11 22:08:47]

Done.

+      args.extend(['--include-files', inclusions])
+      args.extend(['--exclusive_include'])
+      args.extend(['--strip_symbol'])
+      # If update_properties is passed in and store_by_hash is False,
+      # we store it with commit position number instead of a hash
+      update_properties = kwargs.pop("update_properties", None)
+      if update_properties and not kwargs.pop("store_by_hash", True):
+        commit_position = self._get_commit_position(
+                            update_properties, None)
+        cp_branch, cp_number = self.m.commit_position.parse(commit_position)
+        args.extend(['--build_revision', cp_number])
 
     properties_json = self.m.json.dumps(self.m.properties.legacy())
     args.extend(['--factory-properties', properties_json,
                  '--build-properties', properties_json])
 
     kwargs['allow_subannotations'] = True
     self.m.python(
       step_name,
       self.package_repo_resource('scripts', 'tools', 'runit.py'),
       args,
@@ skipping 274 matching lines @@
   def legacy_download_url(self, gs_bucket_name, extra_url_components=None):
     """Returns a url suitable for downloading a Chromium build from
     Google Storage.
 
     extra_url_components, if specified, should be a string without a
     trailing '/' which is inserted in the middle of the URL.
 
     The builder_name, or parent_buildername, is always automatically
     inserted into the URL."""
     return self._legacy_url(True, gs_bucket_name, extra_url_components)