1379665 - TelemetryEnvironment should be presented visually in report index

Status: RESOLVED FIXED

(Socorro :: Webapp, task)

Description

[Benjamin Smedberg]

The telemetry environment is submitted in JSON format in the TelemetryEnvironment annotation field. On individual crash reports this field should be formatted and presented to users: epsecially as we move towards unifying with the telemetry pipeline, this is intended to be the official source of most correlation and other environmental data.

The environment documentation is maintained at https://gecko.readthedocs.io/en/latest/toolkit/components/telemetry/telemetry/data/environment.html

Some possible presentation options can be found in-product by looking at about:telemetry.

This is a followup and longer-term than bug 1379662, which is just about making this field public at all.

Comment 1

[Peter Bengtsson [:peterbe]]

The "Metadata" tab is basically an iteration of all raw and processed crash fields associated with the crash. 

What we *could* do, if you think it's a good idea Benjamin, is add another tab (next to "Metadata" or somesuch) called "Telemetry" (if available!) that displays raw_crash.TelemetryEnvironment in a more visually compelling format. I.e. a tree or nested bullet point list. If we dedicate a tab to it we don't need to do what's done in about:telemetry where you have to click into each node. All nodes can be expanded immediately. 

Happy to work on that if you think it's a good plan. Also, this will sort of side-step https://bugzilla.mozilla.org/show_bug.cgi?id=1379662 since it'd always be visible independent of having PII access or not.

Comment 2

[Benjamin Smedberg]

A new tab is fine.

Although it would probably be good to start out with the dirtiest simple thing to get it public first, since people are using it today to distinguish stylo and non-stylo crashes.

Comment 3

[[DEACTIVATED] Adrian Gaudebert]

As a first step, that field now appears in the Metadata tab of the individual crash page. For example: https://crash-stats.mozilla.com/report/index/d59d9adb-7baf-48eb-9661-6f7c40170711#tab-metadata

Comment 4

[Peter Bengtsson [:peterbe]]

PR https://github.com/mozilla-services/socorro/pull/3847

Comment 5

[[github robot]]

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

https://github.com/mozilla-services/socorro/commit/cdf8d96ee5d72d76f1cc11e77dbb8d1eebf67403
fixes bug 1379665 - Telemetry tab on report index (#3847)

* fixes bug 1379665 - Telemetry tab on report index

* basic test of the TelemetryEnvironment tab in

Comment 6

[Peter Bengtsson [:peterbe]]

PR merged prematurely.

Comment 7

[Peter Bengtsson [:peterbe]]

New PR that uses JSONView
https://github.com/mozilla-services/socorro/pull/3860

Comment 8

[[github robot]]

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

https://github.com/mozilla-services/socorro/commit/9e92479776fe82d10751e6f3de3898de9cfa6f46
fixes bug 1379665 - Use JSON widget to display TelemetryEnvironment (#3860)

* fixes bug 1379665 - Use JSON widget to display TelemetryEnvironment

* nits

Comment 9

[Peter Bengtsson [:peterbe]]

Oops. Typo in bundles so the .less -> .css file became an empty one. 
https://github.com/mozilla-services/socorro/pull/3862

Comment 10

[[github robot]]

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

https://github.com/mozilla-services/socorro/commit/cbe62c9257cbcb89f67df320762778445cf9ad4b
fixes bug 1379665 - build jsonview less file from right source (#3862)