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

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

Issue 2279953002: lightweight builds archiving for mac and win64 (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: lightweight builds archiving for mac and win64 Created 4 years, 4 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/chromium_tests/api.py
diff --git a/scripts/slave/recipe_modules/chromium_tests/api.py b/scripts/slave/recipe_modules/chromium_tests/api.py
index d40e13c77be12dde3f4df8cf767ac7311f8d2fa4..20dd0e3b1098490c32b1d8d25f05a82cdcffe6a6 100644
--- a/scripts/slave/recipe_modules/chromium_tests/api.py
+++ b/scripts/slave/recipe_modules/chromium_tests/api.py
@@ -399,17 +399,23 @@ class ChromiumTestsApi(recipe_api.RecipeApi):
def archive_build(self, mastername, buildername, update_step, bot_db):
bot_config = bot_db.get_bot_config(mastername, buildername)
-
if bot_config.get('bot_type') == 'builder':
if not bot_config.get('cf_archive_build'):
master_config = bot_db.get_master_settings(mastername)
build_revision = update_step.presentation.properties['got_revision']
-
# For archiving 'chromium.perf', the builder also archives a version
# without perf test files for manual bisect.
# (https://bugs.chromium.org/p/chromium/issues/detail?id=604452)
if (master_config.get('bisect_builders') and
buildername in master_config.get('bisect_builders')):
+ include_list, strip_list, whitelist = None, None, None
+ if buildername in master_config.get('bisect_include_files'):
+ include_list = (master_config.get('bisect_include_files')
+ [buildername])
+ if buildername in master_config.get('bisect_strip_files'):
+ strip_list = master_config.get('bisect_strip_files')[buildername]
+ if buildername in master_config.get('bisect_whitelist'):
+ whitelist = master_config.get('bisect_whitelist')[buildername]
self.m.archive.zip_and_upload_build(
'package build for bisect',
self.m.chromium.c.build_config_fs,
@@ -418,6 +424,9 @@ class ChromiumTestsApi(recipe_api.RecipeApi):
cros_board=self.m.chromium.c.TARGET_CROS_BOARD,
update_properties=update_step.presentation.properties,
exclude_perf_test_files=True,
+ include_bisect_file_list=include_list,
+ include_bisect_strip_list=strip_list,
+ include_bisect_whitelist=whitelist,
store_by_hash=False
)

Powered by Google App Engine
This is Rietveld 408576698