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

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

Issue 2112413002: Allowing zip_build to use a file name given in build_url. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Untouching blank line. 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/example.py
diff --git a/scripts/slave/recipe_modules/archive/example.py b/scripts/slave/recipe_modules/archive/example.py
index 45994264feb659f9d45d8094578b17299d05b2b5..599fe10ea9d32823ad0cf91a6a6f871e0f01612d 100644
--- a/scripts/slave/recipe_modules/archive/example.py
+++ b/scripts/slave/recipe_modules/archive/example.py
@@ -18,6 +18,11 @@ TEST_COMMIT_POSITON_COMPONENT='refs/heads/master@{#234}'
def RunSteps(api):
+ if 'build_archive_url' in api.properties:
+ api.archive.zip_and_upload_build(
+ step_name='zip build',
+ target=api.path['checkout'].join('/Release/out'))
+ return
api.archive.clusterfuzz_archive(
build_dir=api.path['slave_build'].join('src', 'out', 'Release'),
update_properties=api.properties.get('update_properties'),
@@ -112,3 +117,10 @@ def GenTests(api):
api.override_step_data(
'listdir build_dir', api.json.output(['chrome']))
)
+
+ yield(
+ api.test('zip_and_upload_custom_location') +
+ api.platform('linux', 64) +
+ api.properties(
+ build_archive_url='gs://dummy-bucket/Linux Release/full-build.zip')
+ )

Powered by Google App Engine
This is Rietveld 408576698