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

Side by Side Diff: infra/recipes/android/perf.py

Issue 2392423002: [Android] Don't pass adb path to sharded perf tests. (Closed)
Patch Set: Created 4 years, 2 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
« no previous file with comments | « no previous file | infra/recipes/android/perf.expected/full_chromium_perf_Android_Galaxy_S5_Perf__1_.json » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 170 matching lines...) Expand 10 before | Expand all | Expand 10 after
181 except api.step.StepFailure as f: 181 except api.step.StepFailure as f:
182 failures.append(f) 182 failures.append(f)
183 183
184 dynamic_perf_tests = api.chromium_tests.steps.DynamicPerfTests( 184 dynamic_perf_tests = api.chromium_tests.steps.DynamicPerfTests(
185 builder['perf_id'], 'android', None, 185 builder['perf_id'], 'android', None,
186 max_battery_temp=builder.get('max_battery_temp'), 186 max_battery_temp=builder.get('max_battery_temp'),
187 num_device_shards=builder['num_device_shards'], 187 num_device_shards=builder['num_device_shards'],
188 num_host_shards=builder.get('num_host_shards', 1), 188 num_host_shards=builder.get('num_host_shards', 1),
189 shard_index=builder.get('shard_index', 0), 189 shard_index=builder.get('shard_index', 0),
190 override_browser_name=builder.get('browser_name'), 190 override_browser_name=builder.get('browser_name'),
191 enable_platform_mode=builder.get('enable_platform_mode')) 191 enable_platform_mode=builder.get('enable_platform_mode'),
192 pass_adb_path=False)
192 dynamic_perf_tests.run(api, None) 193 dynamic_perf_tests.run(api, None)
193 194
194 if failures: 195 if failures:
195 raise api.step.StepFailure('src-side perf tests failed %s' % failures) 196 raise api.step.StepFailure('src-side perf tests failed %s' % failures)
196 finally: 197 finally:
197 api.chromium_android.common_tests_final_steps( 198 api.chromium_android.common_tests_final_steps(
198 logcat_gs_bucket='chromium-android') 199 logcat_gs_bucket='chromium-android')
199 200
200 201
201 def _sanitize_nonalpha(text): 202 def _sanitize_nonalpha(text):
(...skipping 148 matching lines...) Expand 10 before | Expand all | Expand 10 after
350 "adb_status": "device", 351 "adb_status": "device",
351 "imei_slice": "", 352 "imei_slice": "",
352 "ro.build.product": "bullhead", 353 "ro.build.product": "bullhead",
353 "ro.build.id": "MDB08Q", 354 "ro.build.id": "MDB08Q",
354 "serial": "00d0d567893340f4", 355 "serial": "00d0d567893340f4",
355 "wifi_ip": "" 356 "wifi_ip": ""
356 }] 357 }]
357 }}), 358 }}),
358 retcode=87)) 359 retcode=87))
359 360
OLDNEW
« no previous file with comments | « no previous file | infra/recipes/android/perf.expected/full_chromium_perf_Android_Galaxy_S5_Perf__1_.json » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698