Closed
Bug 1399592
Opened 7 years ago
Closed 7 years ago
Mochitest and reftest runner are using a too short timeout for starting a Marionette session
Categories
(Testing :: Mochitest, enhancement)
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.
|
Assignee | |
Updated•7 years ago
|
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
| Comment hidden (mozreview-request) |
|
|
Assignee | |
Updated•7 years ago
|
Attachment #8908132 -
Flags: review?(gbrown)
|
||
Comment 2•7 years ago
|
||
| mozreview-review | ||
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
|
|
||
Comment 4•7 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox57:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
|
||
Updated•7 years ago
|
Assignee: nobody → hskupin
You need to log in
before you can comment on or make changes to this bug.
Description
•