Commit f401cd4b authored by Sergiy Byelozyorov's avatar Sergiy Byelozyorov Committed by Commit Bot

[tools] Re-land: Add retries when trying to discover the device

This is an experimental change that may help mitigate the issue.

TBR=machenbach@chromium.org

No-Try: true
No-Tree-Checks: true
Bug: chromium:893593
Change-Id: Idf15a63006c2c7ba2c31482e5103b2a0b1d64510
Reviewed-on: https://chromium-review.googlesource.com/c/1339401
Commit-Queue: Sergiy Byelozyorov <sergiyb@chromium.org>
Reviewed-by: 's avatarSergiy Byelozyorov <sergiyb@chromium.org>
Cr-Commit-Position: refs/heads/master@{#57558}
parent 02f1529b
...@@ -9,6 +9,7 @@ Wrapper around the Android device abstraction from src/build/android. ...@@ -9,6 +9,7 @@ Wrapper around the Android device abstraction from src/build/android.
import logging import logging
import os import os
import sys import sys
import time
BASE_DIR = os.path.normpath( BASE_DIR = os.path.normpath(
...@@ -50,8 +51,14 @@ class _Driver(object): ...@@ -50,8 +51,14 @@ class _Driver(object):
devil_chromium.Initialize() devil_chromium.Initialize()
if not device: if not device:
# Detect attached device if not specified. # Detect attached device if not specified. Retry 3 times with 60 second
devices = adb_wrapper.AdbWrapper.Devices() # sleep in-between.
for _ in range(3):
devices = adb_wrapper.AdbWrapper.Devices()
if devices:
break
logging.warning('Failed to detect device. Will retry in 60 secs...')
time.sleep(60)
assert devices, 'No devices detected' assert devices, 'No devices detected'
assert len(devices) == 1, 'Multiple devices detected.' assert len(devices) == 1, 'Multiple devices detected.'
device = str(devices[0]) device = str(devices[0])
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment