945507 - Add better crash integration so we can fail tests when gecko crashes

Status: RESOLVED FIXED

(Testing Graveyard :: JSMarionette, defect)

Description

[James Lal [:lightsofapollo]]

Right now its entirely possible we just hang when the b2g process crashes... At minimum we should fail the test and dump a crashlog.

Comment 2

[Gareth Aye [:gaye] (back from PTO)]

https://github.com/mozilla-b2g/marionette-js-runner/commit/3ce8c414a5bc324f17ecd0109fb01d02c4dd86f5 thing landed on master but still needs to be downstreamed to gaia

Comment 3

[GitHub Autolander Bot]

Attachment: [gaia-node-modules] gaye:marionette-js-runner-1.0.2 > mozilla-b2g:master

Comment 4

[Gareth Aye [:gaye] (back from PTO)]

Attachment: Link to Github pull-request: https://github.com/mozilla-b2g/gaia-node-modules/pull/97

gaia-node-modules bit

Comment 5

[Gareth Aye [:gaye] (back from PTO)]

Landed on gaia-node-modules master after testing in gaia https://github.com/mozilla-b2g/gaia-node-modules/commit/4164b1d6f3d74ab0eb901550857ab52e24752a5c

Comment 6

[Gareth Aye [:gaye] (back from PTO)]

Attachment: Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/28273

This is the downstream to gaia part.

Comment 7

[Gareth Aye [:gaye] (back from PTO)]

(And stuff now works with make test-perf)

Comment 8

[James Lal [:lightsofapollo]]

Comment on attachment 8570111 [details] [review]
Link to Github pull-request: https://github.com/mozilla-b2g/gaia/pull/28273

The code looks fine here but seems like we have some issues:

 - completely dies on bb
 - pretty red on TC https://treeherder.mozilla.org/#/jobs?repo=gaia&revision=11ae4d37cc3520bb774dab43e47d47de3cb2ca42

Looks like on bb the problem is the insanely old python version (This is in particular very strange) and on TC it looks like we stall out on specific test suites.

Comment 9

[Gareth Aye [:gaye] (back from PTO)]

Landed on gaia master https://github.com/mozilla-b2g/gaia/commit/679aae25d631a96050a9fadd4ff1fb8325913b4f