Add telemetry to Firefox Sync

RESOLVED FIXED in 2.6 S6 - 1/29

Status

P2
normal
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: ferjm, Assigned: ferjm, NeedInfo)

Tracking

unspecified
2.6 S6 - 1/29
ARM
Gonk (Firefox OS)

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Updated

3 years ago
Blocks: 824026
Priority: -- → P2
Target Milestone: --- → 2.6 S3 - 12/18
(Assignee)

Updated

3 years ago
Target Milestone: 2.6 S3 - 12/18 → 2.6 S4 - 1/1
(Assignee)

Updated

3 years ago
Assignee: nobody → ferjmoreno
Status: NEW → ASSIGNED
(Assignee)

Updated

3 years ago
Target Milestone: 2.6 S4 - 1/1 → 2.6 S5 - 1/15
Created attachment 8708296 [details] [review]
[gaia] ferjm:bug1231929.sync.telemetry > mozilla-b2g:master
(Assignee)

Updated

3 years ago
Attachment #8708296 - Flags: review?(thills)
Attachment #8708296 - Flags: review?(mbdejong)
(Assignee)

Comment 2

3 years ago
The attached patch adds telemetry to record user actions related to Sync: enable, disable and manual sync. For further information (i.e. new users) we can use Syncto metrics.

Tamara, do we have telemetry enabled for the TV?
(Assignee)

Comment 3

3 years ago
Comment on attachment 8708296 [details] [review]
[gaia] ferjm:bug1231929.sync.telemetry > mozilla-b2g:master

YiFan, could you take a look at the TV parts, please? Thanks!
Attachment #8708296 - Flags: review?(yliao)
(In reply to Fernando Jiménez Moreno [:ferjm] from comment #2)

> Tamara, do we have telemetry enabled for the TV?

Hi Fernando,

I'm going to expose my ignorance of how tv gets built here...  I'm looking at gaia/tv_apps and from what I can gather, it does not look like telemetry is enabled here.  Essentially, a setting called metrics.selectedMetrics.level needs to be set in order for telemetry to be turned on.  In phones, it can get turned on via the build config (e.g. for a foxfooder) or the user can be given a choice to turn it on/off both in the FTU and the settings.  It looks like that has not been done.   

I would think we would need some conversations with the product owners before either pushing it into the FTU or the settings.

Let me know if I can be of further help on this.  

-tamara
Hi Joe,

The intent of this patch is to record user sync action. We only record app usage on tv now, can we add user sync action in telemetry? Could you please help to confirm if there is any legal issue?
Flags: needinfo?(joechengla)
(Assignee)

Updated

3 years ago
Target Milestone: 2.6 S5 - 1/15 → 2.6 S6 - 1/29
Comment on attachment 8708296 [details] [review]
[gaia] ferjm:bug1231929.sync.telemetry > mozilla-b2g:master

Left one comment on github.
Attachment #8708296 - Flags: review?(mbdejong) → review+
Comment on attachment 8708296 [details] [review]
[gaia] ferjm:bug1231929.sync.telemetry > mozilla-b2g:master

Thank you Fernando and Tamara. No the advanced telemetry is not handled in TV system. According to Luke once phone and TV systems merged (if there's no direction change...) it will be enabled and the feature will be evaluated on TV.
Attachment #8708296 - Flags: review?(yliao) → review+
(Assignee)

Comment 8

3 years ago
Thank you folks. I'll land only the phone part then.
(Assignee)

Comment 9

3 years ago
https://github.com/mozilla-b2g/gaia/commit/7f6680d4269fef26acfb9ff68c59a0b5232a3685
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
(Assignee)

Updated

3 years ago
Attachment #8708296 - Flags: review?(thills)
You need to log in before you can comment on or make changes to this bug.