1092176 - Add keyed histogram section to about:telemetry

Status: RESOLVED FIXED

(Toolkit :: Telemetry, defect)

Comment 1

[Georg Fritzsche [:gfritzsche]]

Attachment: Add keyed histograms to about:telemetry

The section shows up fine, but layout is a little borked:
http://imgur.com/Ah1t4YZ

Comment 2

[Georg Fritzsche [:gfritzsche]]

(In reply to Georg Fritzsche [:gfritzsche] from comment #1)
> The section shows up fine, but layout is a little borked:
> http://imgur.com/Ah1t4YZ

Vladan, could we temporarily live with this?

Comment 3

[Georg Fritzsche [:gfritzsche]]

Per IRC this is fine for now.

Comment 4

[Nathan Froyd [:froydnj]]

Comment on attachment 8515189 [details] [diff] [review]
Add keyed histograms to about:telemetry

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

If this shows up with data, I'm happy with it.

Comment 5

[Georg Fritzsche [:gfritzsche]]

https://hg.mozilla.org/integration/mozilla-inbound/rev/304be83fb3c6

Comment 6

[Georg Fritzsche [:gfritzsche]]

Comment on attachment 8515189 [details] [diff] [review]
Add keyed histograms to about:telemetry

Approval Request Comment
[Feature/regressing bug #]: Search telemetry.
[User impact if declined]: This goes with bug 1069874. No keyed histograms visible in about:telemetry, which is bad for QA, transparency etc.
[Describe test coverage new/current, TBPL]: Manual check.
[Risks and why]: Low, just a medium change to about:telemetry.
[String/UUID change made/needed]: Only one string addition for about:telemetry, which can go untranslated IMO.

Comment 7

[Lawrence Mandel [:lmandel] (use needinfo)]

ni on flod to provide l10n approval for the landing with a string addition for about:telemetry.

Comment 8

[Francesco Lodolo [:flod]]

We do localize about:telemetry

Besides that, adding a string on Aurora and Beta is going to create unnecessary noise on branches, when we already have way too much confusion because of DevEd.

If you really need this patch to fix a serious issue, I'd suggest to have a patch for Aurora and Beta without strings.

Comment 9

[Francesco Lodolo [:flod]]

(In reply to Georg Fritzsche [:gfritzsche] from comment #6)
> [String/UUID change made/needed]: Only one string addition for
> about:telemetry, which can go untranslated IMO.

Just to add one more thought: I'd be much happier with these two bugs riding the trains and being properly localized.

Summary says "No keyed histograms visible in about:telemetry, which is bad for QA, transparency etc.", but to me it seems that you're adding a new feature to about:telemetry. Am I wrong? If so, what is currently broken, and when did it break?

Comment 10

[Georg Fritzsche [:gfritzsche]]

(In reply to Francesco Lodolo [:flod] from comment #9)
> (In reply to Georg Fritzsche [:gfritzsche] from comment #6)
> > [String/UUID change made/needed]: Only one string addition for
> > about:telemetry, which can go untranslated IMO.
> 
> Just to add one more thought: I'd be much happier with these two bugs riding
> the trains and being properly localized.
> 
> Summary says "No keyed histograms visible in about:telemetry, which is bad
> for QA, transparency etc.", but to me it seems that you're adding a new
> feature to about:telemetry. Am I wrong? If so, what is currently broken, and
> when did it break?

We are adding a feature to telemetry that should get uplift to beta. This particular bug is for displaying the data from that feature in a new section in about:telemetry.
Without this addition to about:telemetry it will be much harder to check the data and we don't transparently show the data as we should.

I know that we localize about:telemetry, but i think this isn't exactly a central user-facing feature?

Comment 11

[Georg Fritzsche [:gfritzsche]]

As this is just one section title (plus a section title and data field title in the other bug), what do you think about beta/aurora patches that just put the title in directly without the l10n file changes?

Comment 12

[Georg Fritzsche [:gfritzsche]]

Ah, per bug 1086252, comment 9 the above suggestion seems ok?
Will prepare patches accordingly.

Comment 13

[Georg Fritzsche [:gfritzsche]]

Attachment: Aurora/Beta patch: Add keyed histograms to about:telemetry

Approval Request Comment:
See comment 6. This version just addresses the above l10n concerns by making the strings untranslatable.

Comment 14

[Francesco Lodolo [:flod]]

(In reply to Georg Fritzsche [:gfritzsche] from comment #11)
> As this is just one section title (plus a section title and data field title
> in the other bug), what do you think about beta/aurora patches that just put
> the title in directly without the l10n file changes?

I'm OK with this approach for both bugs.

Comment 15

[Phil Ringnalda (:philor)]

https://hg.mozilla.org/mozilla-central/rev/304be83fb3c6

Comment 16

[Lawrence Mandel [:lmandel] (use needinfo)]

Comment on attachment 8515599 [details] [diff] [review]
Aurora/Beta patch: Add keyed histograms to about:telemetry

Approved for Beta and Aurora with hardcoded strings. As an alternative to making untranslatable changes to about:telemetry during Beta, we used to have an about:Telemetry add-on. If this add-on is up-to-date, making the changes there as opposed to in product should allow the required people to see the changes that you've made without impacting about:telemetry for all users.

Vladan - Do we still have the about:telemetry add-on and is it up-to-date?

Comment 17

[Georg Fritzsche [:gfritzsche]]

Per vladans blog, the about:telemetry extension was obsoleted by adding the about:telemetry page to the official source.

https://blog.mozilla.org/vdjeric/2012/11/25/new-abouttelemetry-page-in-firefox-19/

Comment 18

[Georg Fritzsche [:gfritzsche]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/cf74345923db

Comment 19

[Lawrence Mandel [:lmandel] (use needinfo)]

(In reply to Georg Fritzsche [:gfritzsche] from comment #17)
> Per vladans blog, the about:telemetry extension was obsoleted by adding the
> about:telemetry page to the official source.
> 
> https://blog.mozilla.org/vdjeric/2012/11/25/new-abouttelemetry-page-in-
> firefox-19/

Thanks for following up on this. Not a viable alternative then.

Comment 20

[Georg Fritzsche [:gfritzsche]]

https://hg.mozilla.org/releases/mozilla-beta/rev/e6db2f014e26

Comment 21

[YF (Yang)]

What is meant by "keyed", either the keyboard pressed or key/value data?

What is the difference between histogram.

Comment 22

[Georg Fritzsche [:gfritzsche]]

I updated the telemetry documentation to include keyed histograms (key/value store, or rather key/histogram):
https://developer.mozilla.org/en-US/docs/Mozilla/Performance/Adding_a_new_Telemetry_probe