Closed Bug 826132 Opened 7 years ago Closed 7 years ago

Don't restart emulator on socket.timeout

Categories

(Testing :: Marionette, defect)

defect
Not set

Tracking

(firefox19 fixed, firefox20 fixed, b2g18 fixed)

RESOLVED FIXED
mozilla20
Tracking Status
firefox19 --- fixed
firefox20 --- fixed
b2g18 --- fixed

People

(Reporter: jgriffin, Assigned: jgriffin)

References

Details

Attachments

(1 file)

We currently have code that tries to restart the emulator if a socket.timeout occurs, in order to automatically recover from some types of failures.  However, this doesn't work, and in fact causes other problems (see stack trace in bug 824079 for example).  We should just remove this code.
Attachment #697291 - Flags: review?(ahalberstadt)
Comment on attachment 697291 [details] [diff] [review]
Don't restart emulator on failure,

Review of attachment 697291 [details] [diff] [review]:
-----------------------------------------------------------------

lgtm!
Attachment #697291 - Flags: review?(ahalberstadt) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/78454c7cecf3
Assignee: nobody → jgriffin
Whiteboard: [automation-needed-in-aurora][automation-needed-in-b2g18]
Target Milestone: --- → mozilla20
https://hg.mozilla.org/mozilla-central/rev/78454c7cecf3
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Duplicate of this bug: 826182
You need to log in before you can comment on or make changes to this bug.