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

Side by Side Diff: scripts/slave/recipes/android/builder.py

Issue 2275563003: Fix android recipe build upload names (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: 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 unified diff | Download patch
OLDNEW
1 # Copyright 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 from contextlib import contextmanager 5 from contextlib import contextmanager
6 from recipe_engine import recipe_api 6 from recipe_engine import recipe_api
7 from recipe_engine.types import freeze 7 from recipe_engine.types import freeze
8 8
9 DEPS = [ 9 DEPS = [
10 'chromium', 10 'chromium',
(...skipping 34 matching lines...) Expand 10 before | Expand all | Expand 10 after
45 }, 45 },
46 'chromium.perf': { 46 'chromium.perf': {
47 'Android Builder': { 47 'Android Builder': {
48 'recipe_config': 'main_builder_rel_mb', 48 'recipe_config': 'main_builder_rel_mb',
49 'gclient_apply_config': ['android', 'perf'], 49 'gclient_apply_config': ['android', 'perf'],
50 'kwargs': { 50 'kwargs': {
51 'BUILD_CONFIG': 'Release', 51 'BUILD_CONFIG': 'Release',
52 }, 52 },
53 'upload': { 53 'upload': {
54 'bucket': 'chrome-perf', 54 'bucket': 'chrome-perf',
55 'path': lambda api: ('android_perf_rel/full-build-linux_%s.zip' 55 'path': lambda api: ('Android Builder/full-build-linux_%s.zip'
56 % api.properties['revision']), 56 % api.properties['revision']),
57 }, 57 },
58 'run_mb': True, 58 'run_mb': True,
59 'targets': [ 59 'targets': [
60 'android_tools', 60 'android_tools',
61 'cc_perftests', 61 'cc_perftests',
62 'chrome_public_apk', 62 'chrome_public_apk',
63 'gpu_perftests', 63 'gpu_perftests',
64 'push_apps_to_background_apk', 64 'push_apps_to_background_apk',
65 'system_webview_apk', 65 'system_webview_apk',
(...skipping 12 matching lines...) Expand all
78 }, 78 },
79 'Android arm64 Builder': { 79 'Android arm64 Builder': {
80 'recipe_config': 'arm64_builder_rel_mb', 80 'recipe_config': 'arm64_builder_rel_mb',
81 'gclient_apply_config': ['android', 'perf'], 81 'gclient_apply_config': ['android', 'perf'],
82 'kwargs': { 82 'kwargs': {
83 'BUILD_CONFIG': 'Release', 83 'BUILD_CONFIG': 'Release',
84 }, 84 },
85 'upload': { 85 'upload': {
86 'bucket': 'chrome-perf', 86 'bucket': 'chrome-perf',
87 'path': lambda api: ( 87 'path': lambda api: (
88 'android_perf_rel_arm64/full-build-linux_%s.zip' 88 'android_perf_rel_arm64/full-build-linux_%s.zip'
dtu 2016/08/24 00:58:36 64-bit also :)
89 % api.properties['revision']), 89 % api.properties['revision']),
90 }, 90 },
91 'run_mb': True, 91 'run_mb': True,
92 'targets': [ 92 'targets': [
93 'android_tools', 93 'android_tools',
94 'cc_perftests', 94 'cc_perftests',
95 'chrome_public_apk', 95 'chrome_public_apk',
96 'gpu_perftests', 96 'gpu_perftests',
97 'push_apps_to_background_apk', 97 'push_apps_to_background_apk',
98 'system_webview_apk', 98 'system_webview_apk',
(...skipping 165 matching lines...) Expand 10 before | Expand all | Expand 10 after
264 '_'.join(_sanitize_nonalpha(step) for step in steps))) + 264 '_'.join(_sanitize_nonalpha(step) for step in steps))) +
265 props(mastername=mastername, buildername=buildername) + 265 props(mastername=mastername, buildername=buildername) +
266 reduce(lambda a, b: a + b, 266 reduce(lambda a, b: a + b,
267 (api.step_data(step, retcode=1) for step in steps)) 267 (api.step_data(step, retcode=1) for step in steps))
268 ) 268 )
269 269
270 yield step_failure(mastername='chromium.android', 270 yield step_failure(mastername='chromium.android',
271 buildername='Android x64 Builder (dbg)', 271 buildername='Android x64 Builder (dbg)',
272 steps=['check licenses']) 272 steps=['check licenses'])
273 273
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698