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

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

Issue 2192563004: [Android] Move host_info from test config to recipe step for android perf bots. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: [Android] Move host_info from test config to recipe step for android perf bots. 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 recipe_engine.types import freeze 5 from recipe_engine.types import freeze
6 6
7 7
8 DEPS = [ 8 DEPS = [
9 'adb', 9 'adb',
10 'chromium', 10 'chromium',
(...skipping 148 matching lines...) Expand 10 before | Expand all | Expand 10 after
159 path=builder['path'](api)) 159 path=builder['path'](api))
160 160
161 api.chromium_android.common_tests_setup_steps( 161 api.chromium_android.common_tests_setup_steps(
162 perf_setup=True, 162 perf_setup=True,
163 remove_system_webview=builder.get('remove_system_webview', False)) 163 remove_system_webview=builder.get('remove_system_webview', False))
164 164
165 required_apks = builder.get('required_apks', []) 165 required_apks = builder.get('required_apks', [])
166 for apk in required_apks: 166 for apk in required_apks:
167 api.chromium_android.adb_install_apk(apk) 167 api.chromium_android.adb_install_apk(apk)
168 168
169 api.chromium_android.host_info(
170 args=api.chromium_tests.get_common_args_for_scripts())
171
169 test_runner = api.chromium_tests.create_test_runner( 172 test_runner = api.chromium_tests.create_test_runner(
170 api, builder.get('tests', [])) 173 api, builder.get('tests', []))
171 174
172 try: 175 try:
173 failures = [] 176 failures = []
174 if test_runner: 177 if test_runner:
175 try: 178 try:
176 test_runner() 179 test_runner()
177 except api.step.StepFailure as f: 180 except api.step.StepFailure as f:
178 failures.append(f) 181 failures.append(f)
(...skipping 113 matching lines...) Expand 10 before | Expand all | Expand 10 after
292 mastername='chromium.perf', 295 mastername='chromium.perf',
293 buildername='Android Nexus5 Perf (1)', 296 buildername='Android Nexus5 Perf (1)',
294 parent_buildername='parent_buildername', 297 parent_buildername='parent_buildername',
295 parent_buildnumber='1729', 298 parent_buildnumber='1729',
296 parent_revision='deadbeef', 299 parent_revision='deadbeef',
297 revision='deadbeef', 300 revision='deadbeef',
298 slavename='slavename', 301 slavename='slavename',
299 target='Release') + 302 target='Release') +
300 api.override_step_data( 303 api.override_step_data(
301 'perf_test.foo', retcode=87)) 304 'perf_test.foo', retcode=87))
305 yield (api.test('host_info_failure') +
306 api.properties.generic(
307 path_config='kitchen',
308 repo_name='src',
309 repo_url=REPO_URL,
310 mastername='chromium.perf',
311 buildername='Android Nexus5 Perf (1)',
312 parent_buildername='parent_buildername',
313 parent_buildnumber='1729',
314 parent_revision='deadbeef',
315 revision='deadbeef',
316 slavename='slavename',
317 target='Release')
318 + api.step_data('Host_Info', retcode=87))
319
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698