1153252 - Communicate pref changes for "datareporting.healthreport.service.enabled" and update wiki

Status: RESOLVED FIXED

(Toolkit :: Telemetry, defect, P2)

Description

[Georg Fritzsche [:gfritzsche]]

Currently enableTelemetryRecording() depends on the pref:
https://hg.mozilla.org/mozilla-central/annotate/dd32e3ff3717/toolkit/components/telemetry/TelemetryPing.jsm#l715

Comment 1

[Georg Fritzsche [:gfritzsche]]

This means we can't just disable FHR per pref flip, because it would disable all Telemetry recording.

Comment 2

[Benjamin Smedberg]

You mean we need a separate pref to disable telemetry recording?

Comment 3

[Georg Fritzsche [:gfritzsche]]

This should be covered by bug 1148500, leaving this one open for tracking.

We need to communicate this so that enterprise users etc. know that turning off "datareporting.healthreport.service.enabled" will not be sufficient anymore.

Comment 4

[Georg Fritzsche [:gfritzsche]]

Benjamin, do you think that a post to mozilla-enterprise is enough here?
Or do we need something else?

Comment 5

[Benjamin Smedberg]

That plus searching through MDN for any mentions of the current pref and fixing them.

Comment 6

[Georg Fritzsche [:gfritzsche]]

Sent out the mail to the enterprise list (https://mail.mozilla.org/listinfo/enterprise).

Alessio, can you follow up on checking the wiki after the current important work?

Comment 7

[Alessio Placitelli [:Dexter]]

A search engine query (|"datareporting." site:developer.mozilla.org|) and a search on MDN revealed that only the example in this page seems to need an update: https://developer.mozilla.org/en-US/Firefox/Enterprise_deployment

I'll change this one (and propagate the change to the pages in the other languages) and keep looking for other pages.

Comment 8

[Alessio Placitelli [:Dexter]]

Georg, would you mind checking if that change is ok?

Comment 9

[Georg Fritzsche [:gfritzsche]]

That looks fine, thanks.