1885150 - [wpt-sync] Sync PR 45084 - Retry emulator.wait_for_start() if it fails

Status: RESOLVED FIXED

(Testing :: web-platform-tests, task, P4)

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Sync web-platform-tests PR 45084 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/45084
Details from upstream follow.

James Graham <[e-mail]> wrote:

Retry emulator.wait_for_start() if it fails

This is a bit of a wallpaper patch, but I think what's going on is that there's a race condition between calling emulator.start() and emulator.wait_for_start() during which adb commands can fail. This should probably be fixed in mozrunner or mozdevice, but for now this workaround seems to be enough; in particular https://community-tc.services.mozilla.com/tasks/IUqaruRyTCKDSqfyBov3lA/runs/0/logs/live/public/logs/live.log is an example where the initial wait_for_start() call failed, but the second one succeeded.

Comment 1

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=04df130d8dc0dfb3dc987b995956ab438655bdb8

Comment 2

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

The PR was not expected to affect any tests, but the try push wasn't a success. Check the try results for infrastructure issues

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/c8b28ddbb3f4
[wpt PR 45084] - Retry emulator.wait_for_start() if it fails, a=testonly

Comment 4

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Test result changes from PR not available.

Comment 5

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/c8b28ddbb3f4