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

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: Documented test_runner function arguments. 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 137 matching lines...) Expand 10 before | Expand all | Expand 10 after
246 buildername=buildername, 248 buildername=buildername,
247 slavename='slavename', 249 slavename='slavename',
248 mastername=mastername) + 250 mastername=mastername) +
249 api.override_step_data( 251 api.override_step_data(
250 'analyze', 252 'analyze',
251 api.json.output({ 253 api.json.output({
252 'status': 'Found dependency', 254 'status': 'Found dependency',
253 'targets': ['android_webview_unittests'], 255 'targets': ['android_webview_unittests'],
254 'build_targets': ['android_webview_unittests_apk']})) + 256 'build_targets': ['android_webview_unittests_apk']})) +
255 api.step_data('[trigger] components_unittests', retcode=1) + 257 api.step_data('[trigger] components_unittests', retcode=1) +
256 api.step_data('[collect] android_webview_unittests', retcode=1) 258 # Test runner error
259 api.step_data('[collect] android_webview_unittests', retcode=1) +
260 # Test runner infrastructure error
261 api.step_data('[collect] base_unittests', retcode=87) +
262 # Test runner warning
263 api.step_data('[collect] cc_unittests', retcode=88)
257 ) 264 )
258 265
259 yield ( 266 yield (
260 api.test('analyze_no_compilation') + 267 api.test('analyze_no_compilation') +
261 api.properties.generic( 268 api.properties.generic(
262 mastername='tryserver.chromium.linux', 269 mastername='tryserver.chromium.linux',
263 buildername='android_amp_rel_tests_recipe', 270 buildername='android_amp_rel_tests_recipe',
264 slavename='slavename') + 271 slavename='slavename') +
265 api.override_step_data( 272 api.override_step_data(
266 'analyze', api.json.output({'status': 'No compile necessary'}))) 273 'analyze', api.json.output({'status': 'No compile necessary'})))
267 274
268 yield ( 275 yield (
269 api.test('analyze_no_tests') + 276 api.test('analyze_no_tests') +
270 api.properties.generic( 277 api.properties.generic(
271 mastername='tryserver.chromium.linux', 278 mastername='tryserver.chromium.linux',
272 buildername='android_amp_rel_tests_recipe', 279 buildername='android_amp_rel_tests_recipe',
273 slavename='slavename') + 280 slavename='slavename') +
274 api.override_step_data( 281 api.override_step_data(
275 'analyze', 282 'analyze',
276 api.json.output({ 283 api.json.output({
277 'status': 'Found dependency', 284 'status': 'Found dependency',
278 'targets': [], 285 'targets': [],
279 'build_targets': ['base_unittests']}))) 286 'build_targets': ['base_unittests']})))
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698