Closed Bug 1119061 Opened 8 years ago Closed 8 years ago

Add telemetry for Android Sync migration process


(Firefox for Android Graveyard :: Android Sync, defect)

Not set


(firefox35 unaffected, firefox36 unaffected, firefox37 fixed, firefox38 fixed)

Firefox 38
Tracking Status
firefox35 --- unaffected
firefox36 --- unaffected
firefox37 --- fixed
firefox38 --- fixed


(Reporter: nalexander, Assigned: nalexander)




(2 files, 1 obsolete file)

At first blush, perhaps:

* number of syncs against Old Sync account;
* number of GET requests against meta/fxa_credentials;
* number of meta/fxa_credentials fetched;
* number of migrations failed/succeeded;
* number of times "Finish migrating?" notification displayed, or (even better) duration of time that we spend in "MigratedFromSync11" state.
I have manually verified that the TelemetryWrapper works when run from within Fennec, but I have not:

1) verified that the wrapper works when Fennec (Gecko) is not running;
2) verified that the telemetry is actually stored or uploaded.
Assignee: nobody → nalexander
Attachment #8549298 - Flags: review?(rnewman)
Component: Firefox Sync: Cross-client → Android Sync
Product: Mozilla Services → Android Background Services
Comment on attachment 8549298 [details] [review]
Link to Github pull-request:

Looks good. Eagerly awaiting the part 2 that adds the histograms.
Attachment #8549298 - Flags: review?(rnewman) → review+
Attachment #8549909 - Flags: review?(rnewman)
/r/2573 - Bug 1119061 - Part 1: Add Fennec-specific Sync 1.1 -> Sync 1.5 migration telemetry histograms. r=rnewman

Pull down this commit:

hg pull review -r 74fd7f2594714f039a2d8b457775f6b393ec5cf4
Closed: 8 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 38
Comment on attachment 8549909 [details]
MozReview Request: bz://1119061/nalexander

This landed with rnewman's review.
Attachment #8549909 - Flags: review?(rnewman) → review+
Comment on attachment 8549909 [details]
MozReview Request: bz://1119061/nalexander

Approval Request Comment
[Feature/regressing bug #]: feature.  We expect to start migrating Sync users in Firefox 37.  This should be there to collect migration uptake data on pre-release channels.

[User impact if declined]: we'll be flying blind: we won't get any data on the success/fail rate of Android Sync migration uptake.

[Describe test coverage new/current, TreeHerder]: manually tested.

[Risks and why]: very low.  This is new code that is only triggered during Sync migrations.  If we saw crashes, we'd get clean stacktraces for them.

[String/UUID change made/needed]: none.
Attachment #8549909 - Flags: approval-mozilla-aurora?
Comment on attachment 8549909 [details]
MozReview Request: bz://1119061/nalexander

Attachment #8549909 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Attachment #8549909 - Attachment is obsolete: true
Attachment #8619063 - Flags: review+
Product: Android Background Services → Firefox for Android
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.