Closed Bug 1283565 Opened 4 years ago Closed 4 years ago

Add telemetry to record when the automigrate "undo" functionality is used

Categories

(Firefox :: Migration, defect, P2)

defect

Tracking

()

RESOLVED FIXED
Firefox 50
Tracking Status
firefox49 --- fixed
firefox50 --- fixed
firefox51 --- fixed

People

(Reporter: Dolske, Assigned: Gijs)

References

Details

(Whiteboard: [migration-needs-uplift])

Attachments

(1 file)

We'd like to understand how often the automigrate "undo" functionality is invoked by users. This is a clear user signal that we didn't automagically do the right thing, and will be a useful signal to look at as we fine-tune how we decide what browser to import from.
Comment on attachment 8766866 [details]
Bug 1283565 - add telemetry to determine whether undo works and how often it's unavailable due to sync

https://reviewboard.mozilla.org/r/61586/#review58452
Attachment #8766866 - Flags: review?(dolske) → review+
Comment on attachment 8766866 [details]
Bug 1283565 - add telemetry to determine whether undo works and how often it's unavailable due to sync

https://reviewboard.mozilla.org/r/61586/#review58472

data-review=me with some better description.

::: toolkit/components/telemetry/Histograms.json:4334
(Diff revision 1)
> +    "alert_emails": ["gijs@mozilla.com"],
> +    "expires_in_version": "53",
> +    "kind": "enumerated",
> +    "n_values": 31,
> +    "releaseChannelCollection": "opt-out",
> +    "description": "Where undo of the automatic migration was attempted, indicates to what degree we succeeded to undo."

It would be really nice to document what the 31 enumerated values actually mean.
Attachment #8766866 - Flags: review?(benjamin) → review+
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Pushed by gijskruitbosch@gmail.com:
https://hg.mozilla.org/integration/fx-team/rev/adfe23aae513
add telemetry to determine whether undo works and how often it's unavailable due to sync, r=dolske,data-r=bsmedberg
https://hg.mozilla.org/mozilla-central/rev/adfe23aae513
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 50
Whiteboard: [migration-needs-uplift]
Comment on attachment 8766866 [details]
Bug 1283565 - add telemetry to determine whether undo works and how often it's unavailable due to sync

Approval Request Comment
[Feature/regressing bug #]: automigration, used in funnelcake builds 88 (bug 1295873)
[User impact if declined]: no data about the efficacy of the 'undo' functionality of the data migration in the funnelcake
[Describe test coverage new/current, TreeHerder]: nope, telemetry only.
[Risks and why]: low risk. All the code here is behind the pref (so won't run on general release). This is basic data gathering of how well the undo functionality works. This way we'll know pretty quickly if there are major issues with it and how it plays out with sync. Based on the QE verification of the earlier funnelcake based on 48, I'm confident that this change doesn't affect the workings of the funnelcake otherwise (and as said, it won't run at all on general release).
[String/UUID change made/needed]: no.
Attachment #8766866 - Flags: approval-mozilla-beta?
Comment on attachment 8766866 [details]
Bug 1283565 - add telemetry to determine whether undo works and how often it's unavailable due to sync

More telemetry that should only happen on release for funnelcake users.
Attachment #8766866 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.