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

Unified Diff: scripts/slave/recipe_modules/archive/api.py

Issue 2128613005: Archive Linux perf builds for manual bisect (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Code style change Created 4 years, 5 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/recipe_modules/archive/api.py
diff --git a/scripts/slave/recipe_modules/archive/api.py b/scripts/slave/recipe_modules/archive/api.py
index f5fe781719417860feeb2319b48333b6e82bc493..10a966c2c3b0ab7e949c7d75cfb87dd9defbc35c 100644
--- a/scripts/slave/recipe_modules/archive/api.py
+++ b/scripts/slave/recipe_modules/archive/api.py
@@ -5,6 +5,7 @@
import re
from recipe_engine import recipe_api
+from . import perf_test_files
# TODO(machenbach): Chromium specific data should move out of the archive
@@ -116,6 +117,19 @@ class ArchiveApi(recipe_api.RecipeApi):
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.CHROME_REQUIRED_FILES)
+ args.extend(['--include-files', inclusions])
+ args.extend(['--ignore_regex'])
+ 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,

Powered by Google App Engine
This is Rietveld 408576698