Closed Bug 1231929 Opened 7 years ago Closed 7 years ago

Add telemetry to Firefox Sync

Categories

(Firefox OS Graveyard :: Sync, defect, P2)

ARM
Gonk (Firefox OS)
defect

Tracking

(Not tracked)

RESOLVED FIXED
2.6 S6 - 1/29

People

(Reporter: ferjm, Assigned: ferjm, NeedInfo)

References

Details

Attachments

(1 file)

No description provided.
Blocks: fxos-sync
Priority: -- → P2
Target Milestone: --- → 2.6 S3 - 12/18
Target Milestone: 2.6 S3 - 12/18 → 2.6 S4 - 1/1
Assignee: nobody → ferjmoreno
Status: NEW → ASSIGNED
Target Milestone: 2.6 S4 - 1/1 → 2.6 S5 - 1/15
Attachment #8708296 - Flags: review?(thills)
Attachment #8708296 - Flags: review?(mbdejong)
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?
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)
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+
Thank you folks. I'll land only the phone part then.
https://github.com/mozilla-b2g/gaia/commit/7f6680d4269fef26acfb9ff68c59a0b5232a3685
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Attachment #8708296 - Flags: review?(thills)
You need to log in before you can comment on or make changes to this bug.