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

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

Issue 954743003: Make step status EXCEPTION when test runner has infrastructure failure. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Moved test runner exit codes to central location. Created 5 years, 9 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 2015 The Chromium Authors. All rights reserved. 1 # Copyright 2015 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 infra.libs.infra_types import freeze 5 from infra.libs.infra_types import freeze
6 from slave import recipe_api 6 from slave import recipe_api
7 7
8 DEPS = [ 8 DEPS = [
9 'amp', 9 'amp',
10 'bot_update', 10 'bot_update',
(...skipping 78 matching lines...) Expand 10 before | Expand all | Expand 10 after
89 }) 89 })
90 90
91 REPO_URL = 'svn://svn-mirror.golo.chromium.org/chrome/trunk/src' 91 REPO_URL = 'svn://svn-mirror.golo.chromium.org/chrome/trunk/src'
92 92
93 AMP_INSTANCE_ADDRESS = '172.22.21.180' 93 AMP_INSTANCE_ADDRESS = '172.22.21.180'
94 AMP_INSTANCE_PORT = '80' 94 AMP_INSTANCE_PORT = '80'
95 AMP_INSTANCE_PROTOCOL = 'http' 95 AMP_INSTANCE_PROTOCOL = 'http'
96 AMP_RESULTS_BUCKET = 'chrome-amp-results' 96 AMP_RESULTS_BUCKET = 'chrome-amp-results'
97 97
98 def GenSteps(api): 98 def GenSteps(api):
99 api.amp.setup()
100
99 mastername = api.properties['mastername'] 101 mastername = api.properties['mastername']
100 buildername = api.properties['buildername'] 102 buildername = api.properties['buildername']
101 builder = BUILDERS[mastername][buildername] 103 builder = BUILDERS[mastername][buildername]
102 api.chromium_android.configure_from_properties( 104 api.chromium_android.configure_from_properties(
103 builder['config'], 105 builder['config'],
104 REPO_NAME='src', 106 REPO_NAME='src',
105 REPO_URL=REPO_URL, 107 REPO_URL=REPO_URL,
106 INTERNAL=False, 108 INTERNAL=False,
107 BUILD_CONFIG=builder['target']) 109 BUILD_CONFIG=builder['target'])
108 110
(...skipping 130 matching lines...) Expand 10 before | Expand all | Expand 10 after
239 revision='4f4b02f6b7fa20a3a25682c457bbc8ad589c8a00', 241 revision='4f4b02f6b7fa20a3a25682c457bbc8ad589c8a00',
240 buildername=buildername, 242 buildername=buildername,
241 slavename='slavename', 243 slavename='slavename',
242 mastername=mastername) + 244 mastername=mastername) +
243 api.override_step_data( 245 api.override_step_data(
244 'analyze', 246 'analyze',
245 api.json.output({ 247 api.json.output({
246 'status': 'Found dependency', 248 'status': 'Found dependency',
247 'targets': ['android_webview_unittests'], 249 'targets': ['android_webview_unittests'],
248 'build_targets': ['android_webview_unittests_apk']})) + 250 'build_targets': ['android_webview_unittests_apk']})) +
249 api.step_data('[collect] android_webview_unittests', retcode=1) 251 api.step_data('[collect] android_webview_unittests', retcode=1) +
252 # Infrastructure failure
253 api.step_data('[collect] base_unittests', retcode=87)
250 ) 254 )
251 255
252 yield ( 256 yield (
253 api.test('analyze_no_compilation') + 257 api.test('analyze_no_compilation') +
254 api.properties.generic( 258 api.properties.generic(
255 mastername='tryserver.chromium.linux', 259 mastername='tryserver.chromium.linux',
256 buildername='android_amp_rel_tests_recipe', 260 buildername='android_amp_rel_tests_recipe',
257 slavename='slavename') + 261 slavename='slavename') +
258 api.override_step_data( 262 api.override_step_data(
259 'analyze', api.json.output({'status': 'No compile necessary'}))) 263 'analyze', api.json.output({'status': 'No compile necessary'})))
260 264
261 yield ( 265 yield (
262 api.test('analyze_no_tests') + 266 api.test('analyze_no_tests') +
263 api.properties.generic( 267 api.properties.generic(
264 mastername='tryserver.chromium.linux', 268 mastername='tryserver.chromium.linux',
265 buildername='android_amp_rel_tests_recipe', 269 buildername='android_amp_rel_tests_recipe',
266 slavename='slavename') + 270 slavename='slavename') +
267 api.override_step_data( 271 api.override_step_data(
268 'analyze', 272 'analyze',
269 api.json.output({ 273 api.json.output({
270 'status': 'Found dependency', 274 'status': 'Found dependency',
271 'targets': [], 275 'targets': [],
272 'build_targets': ['base_unittests']}))) 276 'build_targets': ['base_unittests']})))
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698