1268029 - Use jit classifier to change signature for jit crashes

Status: RESOLVED FIXED

(Socorro :: Backend, task)

Description

[(not currently active) Ted Mielczarek]

bug 1124241 added classification of JIT crashes using the output of the `jit-crash-categorize` tool that's in the Socorro tree:
https://github.com/mozilla/socorro/blob/master/minidump-stackwalk/jit-crash-categorize.cc

This is working in production, you can look at the processed crash output and see the entry under "classifications":
https://crash-stats.mozilla.com/api/ProcessedCrash/?crash_id=064b61dd-ef54-434b-b743-344872160421

It never got hooked up to change the resulting crash signature, however, which makes it hard to do anything useful with it.

We should make this generate new signatures, as per bug 1124241 comment 2. In that comment I suggested `[@ jit | <reason> ]`, but I'm not wedded to that proposal, and someone who does more crash analysis, or someone who works on the JIT might have a better suggestion.

Comment 1

[Jan de Mooij [:jandem]]

That sounds reasonable to me, we can see how it turns out and refine if needed. Thanks for filing this!

Comment 2

[[DEACTIVATED] Adrian Gaudebert]

Attachment: Link to Github pull-request: https://github.com/mozilla/socorro/pull/3314

Comment 3

[[github robot]]

Commit pushed to master at https://github.com/mozilla/socorro

https://github.com/mozilla/socorro/commit/fac69f51f8be21668447145b809dca994cad1623
Fixes bug 1268029 - Added a Rule to replace a signature if there is a JIT classification. (#3314)

r=peterbe

Comment 4

[[DEACTIVATED] Adrian Gaudebert]

Here's an example on stage: https://crash-stats.allizom.org/report/index/8df6a572-a88a-41c0-9087-6516f2160504

And a SuperSearch: https://crash-stats.allizom.org/search/?product=Firefox&signature=%5Ejit&_facets=signature