Closed Bug 1449862 Opened 2 years ago Closed 2 years ago
Check for available port each time before starting binary
59 bytes, text/x-review-board-request
Currently we only check if port 2828 is available when initializing Marionette: https://dxr.mozilla.org/mozilla-central/rev/a456475502b80a1264642d9eaee9394a8fad8315/testing/marionette/client/marionette_driver/marionette.py#618-620 This is racy and can lead to side-effects like I have seen on bug 1400819 when using non in_app `restart()` and `quit()` calls, because for the next session this check is no longer performed and accidentally left-over instances of Firefox would still claim that port. Result is that we would continue to run our tests in the other process but not the currently started one. To prevent things like that we should really do this check each time the process gets started via `start_binary()`.
Comment on attachment 8963500 [details] Bug 1449862 - [marionette] Check for available port each time before starting binary. https://reviewboard.mozilla.org/r/232442/#review237854 It’s terrible we are still stuck with this type of port allocation polling.
Attachment #8963500 - Flags: review?(ato) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/0d99bcacaf32 [marionette] Check for available port each time before starting binary. r=ato
You need to log in before you can comment on or make changes to this bug.