1244293 - Don't upload telemetry if user opts out

Status: RESOLVED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

Looks like I forgot to include this functionality in the initial implementation.

Comment 1

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

Attachment: MozReview Request: Bug 1244293 - Don't upload telemetry if user opts out. r=mfinkle

I'm slightly concerned we're providing too much configuration information in
the debugging statements.

Review commit: https://reviewboard.mozilla.org/r/32841/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/32841/

Comment 2

[Mark Finkle (:mfinkle) (use needinfo?)]

Comment on attachment 8713876 [details]
MozReview Request: Bug 1244293 - Don't upload telemetry if user opts out. r=mfinkle

https://reviewboard.mozilla.org/r/32841/#review29741

::: mobile/android/services/src/main/java/org/mozilla/gecko/background/BackgroundService.java:44
(Diff revision 1)
> -  protected boolean backgroundDataIsEnabled() {
> +  protected static boolean backgroundDataIsEnabled(final Context context) {

Looks like backgroundDataIsEnabled has never been used anywhere else. How do we know this code works as intended?

Comment 3

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

https://reviewboard.mozilla.org/r/32841/#review29741

> Looks like backgroundDataIsEnabled has never been used anywhere else. How do we know this code works as intended?

It was used by the `HealthReportUploadService` in a simila manner to this patch.

Comment 4

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

https://hg.mozilla.org/integration/fx-team/rev/6c9e43e567ef824dfbabd4eb468d17374162420f
Bug 1244293 - Don't upload telemetry if user opts out. r=mfinkle

Comment 5

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/6c9e43e567ef

Comment 6

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

NI self – we should verify in tomorrow's Nightly that this does not affect the numbers we're receiving in an unexpected way.

Comment 7

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

Let's land with the initial core ping implementation (bug 1205835).

Comment 8

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

(In reply to Michael Comella (:mcomella) from comment #6)
> NI self – we should verify in tomorrow's Nightly that this does not affect
> the numbers we're receiving in an unexpected way.

This should be in the 2/5 build, which I can see on the telemetry s3 server at the following url:
 <s3-server>/telemetry-2/20160205/telemetry/1/core/Fennec/nightly/47.0a1/

Using `ls | wc -l`, there are 89% of pings from 2/5 builds on 2/5 as there were for 2/3 for builds from 2/3. The day's not over yet so this drop seems reasonable but it's worth noting that our Nightly numbers of users on a build built that day may not be statistically significant enough to really do this analysis. That being said, we're still getting pings and I'm satisfied. :)

Comment 9

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

Comment on attachment 8713876 [details]
MozReview Request: Bug 1244293 - Don't upload telemetry if user opts out. r=mfinkle

This must be uplifted with bug 1205835.

Approval Request Comment
[Feature/regressing bug #]: Landing of bug 1205835
[User impact if declined]: Users can't opt out of telemetry, which probably breaks our privacy policy or something

[Describe test coverage new/current, TreeHerder]: Tested locally, verified on Nightly that uploads are not broken (i.e. data is still coming in)
[Risks and why]: Low – we move some existing checks around and add a check for whether or not the preference is enabled.

[String/UUID change made/needed]: None

Comment 10

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

This (and probably a bunch of the related uplift requests) needs to be rebased to work around the lack of bug 1107811 on the release branches.

Comment 11

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

https://hg.mozilla.org/releases/mozilla-aurora/rev/92e846782a65

Comment 12

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

https://hg.mozilla.org/releases/mozilla-beta/rev/4ae5b1d83228

Comment 13

[Sylvestre Ledru [:Sylvestre]]

Comment on attachment 8713876 [details]
MozReview Request: Bug 1244293 - Don't upload telemetry if user opts out. r=mfinkle

michael had my approval. Updating the uplift flags for posterity.
Should be in 45 beta 4