1621742 - Remove telemetry from chrome.storage.sync

Status: RESOLVED FIXED

(WebExtensions :: Storage, task)

Description

[Ethan Glasser-Camp (:glasserc)]

I received a cool email saying:

The following Firefox probes have either expired or will expire in the next major Firefox nightly release: version 77 [1].


The following probes are expiring in Firefox version 77:
STORAGE_SYNC_GET_OPS_SIZE
STORAGE_SYNC_SET_OPS_SIZE
STORAGE_SYNC_REMOVE_OPS

I don't think these probes have been especially useful and I think now is the time to get rid of them.

Comment 1

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

Hi Ben,
If I recall correctly these probes are not used in the Addons Daily derived dataset, is that right? do you know if we use these probes in any other derived dataset or dashboard?

Also, needinfo-ing Philipp to double-check if it does sounds ok to remove these probes also from his perspective.

Comment 2

[Ben Miroglio [:bmiroglio]]

These aren't used in any datasets/dashboards afaik -- I'd say they are OK to remove.

Comment 3

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

Attachment: Bug 1621742 - Removed STORAGE_SYNC_* histograms. r=mixedpuppy!

Comment 4

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

Attachment: Bug 1621742 - Removed storage.sync telemetry scalars expired in Firefox 64. r=mixedpuppy!

Depends on D67197

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/526758bcd4c2
Removed STORAGE_SYNC_* histograms. r=mixedpuppy,janerik
https://hg.mozilla.org/integration/autoland/rev/50ecaae73f45
Removed storage.sync telemetry scalars expired in Firefox 64. r=mixedpuppy,janerik

Comment 6

[Noemi Erli[:noemi_erli]]

https://hg.mozilla.org/mozilla-central/rev/526758bcd4c2
https://hg.mozilla.org/mozilla-central/rev/50ecaae73f45

Comment 7

[Ryan VanderMeulen [:RyanVM]]

<nvm> Didn't see bug 1625409 before I left this comment.