Closed
Bug 1467856
Opened 6 years ago
Closed 6 years ago
Add a HasDeviceTouchScreen=0/1 field to crash reports
Categories
(Socorro :: General, task, P2)
Socorro
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ehsan.akhgari, Assigned: willkg)
References
Details
Attachments
(1 file)
It would be nice if crash reports had a bit set on them to tell us whether the device has a touch screen or not. Then the correlation detection scripts could have told us for example that bug 1462742 corresponds 100% to HasDeviceTouchScreen=1.
Assignee | ||
Comment 1•6 years ago
|
||
Ehsan: Where is that information in the crash report?
Flags: needinfo?(ehsan)
Reporter | ||
Comment 2•6 years ago
|
||
It doesn't exist right now, it needs to be added, perhaps as an annotation or however it's easiest to consume on the Socorro side... Sorry I wasn't specific about that part, I do expect us to need to file a dependency bug about that on the Gecko side to add the information to crash reports. It would be nice if you or someone familiar with the Socorro side of things could add some information on what format would be most suitable for this data, and note any specific requirements we should keep in mind when adding the information to the crash reports.
Flags: needinfo?(ehsan)
Assignee | ||
Comment 3•6 years ago
|
||
No problem! If it's in the crash report, that's easy to pull out and index. 0 and 1 are easy values to do something with. We'd probably turn them into True and False. We get a lot of values this way like If you want to do stuff with this value in the telemetry data, then we can extract it from the TelemetryEnvironment. Does that help?
Reporter | ||
Comment 4•6 years ago
|
||
Yes, thank you!
Reporter | ||
Comment 5•6 years ago
|
||
I just landed bug 1468612 on inbound, hopefully that's sufficient from the Gecko side.
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → willkg
Status: NEW → ASSIGNED
Assignee | ||
Comment 7•6 years ago
|
||
Here's an example crash with the item in the raw crash: https://crash-stats.mozilla.com/report/index/9e8dfd94-774b-4bb0-b2cc-a12950180912#tab-metadata
Assignee | ||
Comment 8•6 years ago
|
||
Comment 9•6 years ago
|
||
Commits pushed to master at https://github.com/mozilla-services/socorro https://github.com/mozilla-services/socorro/commit/da3fc74838564df4a06490dde83422357ee797df fix bug 1467856: add has_device_touch_screen to super search https://github.com/mozilla-services/socorro/commit/39964ca7144f28c177928a80b07e6f798f137229 Merge pull request #4591 from willkg/1490662-memoryerrorcorrection fix bug 1490662, 1420797, 1467856: add fields to super search
Updated•6 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 10•6 years ago
|
||
Thank you!
You need to log in
before you can comment on or make changes to this bug.
Description
•