Henrik Skupin [:whimboo][⌚️UTC+1] Assignee
	Description • 6 years ago

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()`.

	Andreas Tolfsen ❲:ato❳
	Comment 2 • 6 years ago
mozreview-review

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.

	Pulsebot
	Comment 3 • 6 years ago

Pushed by hskupin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0d99bcacaf32
[marionette] Check for available port each time before starting binary. r=ato

	Andrei Ciure[:aciure]
	Comment 4 • 6 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/0d99bcacaf32