1915183 - Add telemetry in preparation to enforcing storage.sync quota on android

Status: RESOLVED FIXED

(WebExtensions :: Storage, task, P1)

Description

[Tomislav Jovanovic :zombie]

We are currently in the planning phase implementing support for syncing storage.sycn on android. Unfortunately, the current backend for storage.sync (which doesn't actually sync) doesn't enforce quotas, so we should do that ahead of switching to the new backend to prevent/minimize data loss.

Comment 1

[Mark Hammond [:markh] [:mhammond]]

There was some discussion that we might end up throwing away local data when enabling syncing of this data on Android, mainly to avoid the risk that the (currently local only) Android data ends up "winning" when merging, and thus that Android data ends up being what's synced to other desktop devices, which user's might find more surprising than the reverse.

While this decision will be owned by the Addons team, it might be worth making that decision before going to this effort?

Comment 2

[Luca Greco [:rpl] [:luca] [:lgreco]]

(In reply to Mark Hammond [:markh] [:mhammond] from comment #1)

There was some discussion that we might end up throwing away local data when enabling syncing of this data on Android, mainly to avoid the risk that the (currently local only) Android data ends up "winning" when merging, and thus that Android data ends up being what's synced to other desktop devices, which user's might find more surprising than the reverse.

While this decision will be owned by the Addons team, it might be worth making that decision before going to this effort?

yes, we slightly reworded this ticket to make it focused on the "gathering details through telemetry" part to inform decisions about it.

Comment 3

[Tomislav Jovanovic :zombie]

Attachment: Data review request

Comment 4

[Tomislav Jovanovic :zombie]

Attachment: Bug 1915183 - Collect telemetry for usage of storage.sync r?rpl

Comment 5

[Tomislav Jovanovic :zombie]

Comment on attachment 9425253 [details]
Data review request

TIL we can do data review for technical category just through Phabricator: https://phabricator.services.mozilla.com/D222442#inline-1235311

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d58013eb4051
Collect telemetry for usage of storage.sync r=rpl

Comment 7

[pstanciu]

https://hg.mozilla.org/mozilla-central/rev/d58013eb4051