Closed Bug 1033975 Opened 6 years ago Closed 6 years ago
Crash recovery in gaiatest is broken
We're getting many crashes right now and our crash-handling code is not dealing with it. I'm going to have a look and see if I can resolve it.
So this is failing because the exception that Marionette raises has changed from InvalidResponseException to IOError, thus gaiatest is catching the wrong thing Changing that runs into the next problem which is that get_device_manager relies on a session, which is not guaranteed after a crash. This was raised in bug 1034044 but may need some MarionetteTestCase changes. I'm not sure how to (nicely) resolve this without bug 1034044 just yet but I'll try.
Summary: Look at crash handling , does not seem to be working → Crash handling in gaiatest is broken
I don't think this blocks bug 962700. I see this as fixing the crash recovery in gaiatest (allowing subsequent tests to run after a crash has occurred), whereas bug 962700 is about proper detection and reporting of the crashes themselves.
No longer blocks: 962700
Summary: Crash handling in gaiatest is broken → Crash recovery in gaiatest is broken
We have been suffering crashes on device a lot veyr badly lately so I pushed through a variant of the fix we discussed. https://github.com/mozilla-b2g/gaia/commit/f740793dc3f7e59df800c6392209ab0a2486d4c9 I'll keep the bug open so we can follow up with Dave's more refined version and unit test.
I backed it out because it needs marionette client 0.8.11 https://github.com/mozilla-b2g/gaia/commit/bc16638fff952754efbdfd6f84b8e068c12b4f6f Although it doesn't make the crash handling any worse!
Relanded now that client 0.8.1 is being used: https://github.com/mozilla-b2g/gaia/commit/b62a3826120a673a38884d5bc0bc1bfb7026ce12
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.