Closed Bug 1399592 Opened 2 years ago Closed 2 years ago

Mochitest and reftest runner are using a too short timeout for starting a Marionette session

Categories

(Testing :: Mochitest, enhancement)

57 Branch
enhancement
Not set

Tracking

(firefox57 fixed)

RESOLVED FIXED
mozilla57
Tracking Status
firefox57 --- fixed

People

(Reporter: whimboo, Assigned: whimboo)

References

Details

Attachments

(1 file)

Via bug 1397675 I noticed that at least Mochitest runner is using a wrong timeout for starting the session. As result Marionette bails out after 60s which is most likely the reason for all the intermittent failures we have seen in the past months.

The line here needs to be changed:

https://dxr.mozilla.org/mozilla-central/rev/f9a5e9ed62103c84e4cde915f4d08f1ce71be83e/testing/mochitest/runtests.py#2163

So whether we use the default value of 120 by removing it, or manually increase it.

I can work on a patch once bug 1397675 is fixed.
Summary: Mochitest runner is using a too short timeout for starting a Marionette session → Mochitest and reftest runner are using a too short timeout for starting a Marionette session
Attachment #8908132 - Flags: review?(gbrown)
Comment on attachment 8908132 [details]
Bug 1399592 - Increase Marionette startup timeout for mochitest/reftest.

https://reviewboard.mozilla.org/r/179808/#review185472

Thanks for all your efforts on Marionette startup. Those intermittents have been frustrating!
Attachment #8908132 - Flags: review?(gbrown) → review+
Pushed by hskupin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/880d772e282e
Increase Marionette startup timeout for mochitest/reftest. r=gbrown
https://hg.mozilla.org/mozilla-central/rev/880d772e282e
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
Assignee: nobody → hskupin
You need to log in before you can comment on or make changes to this bug.