1241294 - Update E10S_AUTOSTART_STATUS telemetry probe (and rename it to E10S_STATUS)

Status: RESOLVED FIXED

(Firefox :: General, defect)

Description

[:Felipe Gomes (needinfo for replies!)]

In order to have a proper bug number to add to the histogram info, I'm splitting this up from bug 1234673.

We are adding some more criteria on disabling e10s (add-ons and certain locales), and the current telemetry probe has run out of buckets. We can't just increase the number of buckets so we need to create a new one.

I'll leave some spare buckets to avoid this in the future.

I'm also adding this as an "opt-out" for release because I think this is probably valuable information to get when e10s is released.



[Tracking Requested - why for this release]:
Necessary for the e10s beta45 experiment

Comment 1

[:Felipe Gomes (needinfo for replies!)]

Attachment: update histogram

Comment 2

[Vladan Djeric (:vladan)]

Comment on attachment 8710145 [details] [diff] [review]
update histogram

Review of attachment 8710145 [details] [diff] [review]:
-----------------------------------------------------------------

::: toolkit/components/telemetry/Histograms.json
@@ +8307,5 @@
> +    "alert_emails": ["[e-mail]"],
> +    "expires_in_version": "never",
> +    "kind": "enumerated",
> +    "n_values": 12,
> +    "releaseChannelCollection": "opt-out",

good call on making this opt-out

Comment 3

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/493d6820745e

Comment 4

[:Felipe Gomes (needinfo for replies!)]

Comment on attachment 8710145 [details] [diff] [review]
update histogram

Approval Request Comment
[Feature/regressing bug #]: the e10s switching code that we plan to test on 45 has two new conditions to disable e10s (for some locales, bug 1234673, and users with add-ons, bug 1234675). They set values on this telemetry probe that needs to be updated to receive this new value.
[User impact if declined]: We won't be able to tell the % of users who are getting e10s disabled for these new reasons
[Describe test coverage new/current, TreeHerder]: landed on inbound
[Risks and why]: small, just a telemetry probe name change and increase in the number of options it accepts.
[String/UUID change made/needed]: the name of the probe is changed but that is not a localization string

Comment 5

[:Felipe Gomes (needinfo for replies!)]

Attachment: rebased for aurora, r=vladan

patch rebased for aurora. Approval request on comment 4.

It would be nice to get this in today, since bug 1234673 was approved and uplifted today, and that bug reports to this updated telemetry probe.

Comment 6

[:Felipe Gomes (needinfo for replies!)]

Comment on attachment 8710145 [details] [diff] [review]
update histogram

(clearing request from m-c patch)

Comment 7

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/493d6820745e

Comment 8

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/829ed713fa9a

Comment 9

[Mayur Patil]

[bugday-20160323]

Status: RESOLVED,FIXED -> UNVERIFIED

Comments:
STR: Not clear.
Developer specific testing

Component: 
Name			Firefox
Version			46.0b9
Build ID		20160322075646
Update Channel          beta
User Agent		Mozilla/5.0 (Windows NT 6.1; WOW64; rv:46.0) Gecko/20100101 Firefox/46.0
OS			Windows 7 SP1 x86_64

Expected Results: 
Developer specific testing

Actual Results: 
As expected