Closed Bug 1282545 Opened 5 years ago Closed 5 years ago

Update description for E10S_STATUS histogram

Categories

(Toolkit :: Telemetry, defect)

48 Branch
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla50
Tracking Status
firefox48 --- fixed
firefox49 --- fixed
firefox50 --- fixed

People

(Reporter: Felipe, Assigned: Felipe)

Details

Attachments

(1 file)

The E10S_STATUS histogram is an enumerated histogram and when created we had reserved a number of extra buckets before using them, in order to not need to change the probe name. These new values are in use now but the description wasn't updated, which makes it a bit worse to understand the data when looking at it at telemetry.mozilla.org
Attachment #8765573 - Flags: review?(chutten) → review+
Comment on attachment 8765573 [details]
Bug 1282545 - Update description for E10S_STATUS histogram.

https://reviewboard.mozilla.org/r/60866/#review57716

LGTM
Pushed by felipc@gmail.com:
https://hg.mozilla.org/integration/fx-team/rev/b27893d49f11
Update description for E10S_STATUS histogram. r=chutten
https://hg.mozilla.org/mozilla-central/rev/b27893d49f11
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
Comment on attachment 8765573 [details]
Bug 1282545 - Update description for E10S_STATUS histogram.

Approval Request Comment
[Feature/regressing bug #]: description not updated with newest possible values used for this data
[User impact if declined]: understanding this probe's data on telemetry.mozilla.org becomes a bit more cumbersome due to the outdated description
[Describe test coverage new/current, TreeHerder]: not part of the build
[Risks and why]: this string is not included in the build, just used by telemetry.mozilla.org
[String/UUID change made/needed]: n/a for builds
Attachment #8765573 - Flags: approval-mozilla-beta?
Attachment #8765573 - Flags: approval-mozilla-aurora?
Comment on attachment 8765573 [details]
Bug 1282545 - Update description for E10S_STATUS histogram.

For telemetry.mozilla.org usage, take it in 48 beta 6 and aurora.
Attachment #8765573 - Flags: approval-mozilla-beta?
Attachment #8765573 - Flags: approval-mozilla-beta+
Attachment #8765573 - Flags: approval-mozilla-aurora?
Attachment #8765573 - Flags: approval-mozilla-aurora+
You need to log in before you can comment on or make changes to this bug.