Index: build/android/provision_devices.py |
diff --git a/build/android/provision_devices.py b/build/android/provision_devices.py |
index b5981d9a39ffab3eb1b8ac3bba2fc0483e1882c1..54c90c3a6eef8e652f02f5ed34041e5201ce6553 100755 |
--- a/build/android/provision_devices.py |
+++ b/build/android/provision_devices.py |
@@ -140,7 +140,8 @@ def WipeDeviceIfPossible(device): |
try: |
device.EnableRoot() |
jbudorick
2014/08/21 17:15:15
@tonyg: it'll skip the wipe here. if we can't enab
|
WipeDeviceData(device) |
- device.Reboot(True, timeout=180, retries=0) |
+ # TODO(jbudorick): Tune the timeout per OS version. |
+ device.Reboot(True, timeout=600, retries=0) |
except (errors.DeviceUnresponsiveError, device_errors.CommandFailedError): |
pass |
@@ -149,7 +150,10 @@ def ProvisionDevice(device, options, is_perf): |
try: |
if not options.skip_wipe: |
WipeDeviceIfPossible(device) |
- device.EnableRoot() |
+ try: |
+ device.EnableRoot() |
tonyg
2014/08/21 17:12:58
I think the old code would skip the wipe if we cou
|
+ except device_errors.CommandFailedError as e: |
+ logging.warning(str(e)) |
_ConfigureLocalProperties(device, is_perf) |
device_settings.ConfigureContentSettings( |
device, device_settings.DETERMINISTIC_DEVICE_SETTINGS) |
@@ -187,7 +191,8 @@ def ProvisionDevice(device, options, is_perf): |
time.sleep(60) |
battery_info = device.old_interface.GetBatteryInfo() |
device.RunShellCommand('date -u %f' % time.time(), as_root=True) |
- device.Reboot(True, timeout=180, retries=0) |
+ # TODO(jbudorick): Tune the timeout per OS version. |
+ device.Reboot(True, timeout=600, retries=0) |
props = device.RunShellCommand('getprop') |
for prop in props: |
logging.info(' %s' % prop) |