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

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

Issue 2485763002: Use devil's adb with each bisect iteration, also start/stop daemons. (Closed)
Patch Set: Whitespace cleanup. Created 4 years, 1 month 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 'build/adb', 9 'build/adb',
10 'build/chromium', 10 'build/chromium',
(...skipping 171 matching lines...) Expand 10 before | Expand all | Expand 10 after
182 required_apks = builder.get('required_apks', []) 182 required_apks = builder.get('required_apks', [])
183 for apk in required_apks: 183 for apk in required_apks:
184 api.chromium_android.adb_install_apk(apk) 184 api.chromium_android.adb_install_apk(apk)
185 185
186 api.chromium_android.host_info( 186 api.chromium_android.host_info(
187 args=api.chromium_tests.get_common_args_for_scripts()) 187 args=api.chromium_tests.get_common_args_for_scripts())
188 188
189 test_runner = api.chromium_tests.create_test_runner( 189 test_runner = api.chromium_tests.create_test_runner(
190 api, builder.get('tests', [])) 190 api, builder.get('tests', []))
191 191
192 api.chromium_android.logcat_gs_bucket = 'chromium-android'
jbudorick 2016/11/08 00:18:31 Same question.
ghost stip (do not use) 2016/11/08 20:56:33 sure, was just re-implementing the same logic. I c
193
192 try: 194 try:
193 failures = [] 195 failures = []
194 if test_runner: 196 if test_runner:
195 try: 197 try:
196 test_runner() 198 test_runner()
197 except api.step.StepFailure as f: 199 except api.step.StepFailure as f:
198 failures.append(f) 200 failures.append(f)
199 201
200 dynamic_perf_tests = api.chromium_tests.steps.DynamicPerfTests( 202 dynamic_perf_tests = api.chromium_tests.steps.DynamicPerfTests(
201 builder['perf_id'], 'android', None, 203 builder['perf_id'], 'android', None,
202 max_battery_temp=builder.get('max_battery_temp'), 204 max_battery_temp=builder.get('max_battery_temp'),
203 num_device_shards=builder['num_device_shards'], 205 num_device_shards=builder['num_device_shards'],
204 num_host_shards=builder.get('num_host_shards', 1), 206 num_host_shards=builder.get('num_host_shards', 1),
205 shard_index=builder.get('shard_index', 0), 207 shard_index=builder.get('shard_index', 0),
206 override_browser_name=builder.get('browser_name'), 208 override_browser_name=builder.get('browser_name'),
207 enable_platform_mode=builder.get('enable_platform_mode')) 209 enable_platform_mode=builder.get('enable_platform_mode'))
208 dynamic_perf_tests.run(api, None) 210 dynamic_perf_tests.run(api, None)
209 211
210 if failures: 212 if failures:
211 raise api.step.StepFailure('src-side perf tests failed %s' % failures) 213 raise api.step.StepFailure('src-side perf tests failed %s' % failures)
212 finally: 214 finally:
213 api.chromium_android.common_tests_final_steps( 215 api.chromium_android.common_tests_final_steps()
214 logcat_gs_bucket='chromium-android')
215 216
216 217
217 def _sanitize_nonalpha(text): 218 def _sanitize_nonalpha(text):
218 return ''.join(c if c.isalnum() else '_' for c in text) 219 return ''.join(c if c.isalnum() else '_' for c in text)
219 220
220 221
221 def GenTests(api): 222 def GenTests(api):
222 for mastername, builders in BUILDERS.iteritems(): 223 for mastername, builders in BUILDERS.iteritems():
223 for buildername in builders: 224 for buildername in builders:
224 yield ( 225 yield (
(...skipping 141 matching lines...) Expand 10 before | Expand all | Expand 10 after
366 "adb_status": "device", 367 "adb_status": "device",
367 "imei_slice": "", 368 "imei_slice": "",
368 "ro.build.product": "bullhead", 369 "ro.build.product": "bullhead",
369 "ro.build.id": "MDB08Q", 370 "ro.build.id": "MDB08Q",
370 "serial": "00d0d567893340f4", 371 "serial": "00d0d567893340f4",
371 "wifi_ip": "" 372 "wifi_ip": ""
372 }] 373 }]
373 }}), 374 }}),
374 retcode=87)) 375 retcode=87))
375 376
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698