1920723 - Migrate Services.telemetry.recordEvent calls in SearchService.sys.mjs to Glean APIs

Status: RESOLVED FIXED

(Toolkit :: Telemetry, task, P1)

Description

[Chris H-C :chutten]

search.engine.* events in SearchService were added side-by-side to both Legacy Telemetry and Glean two years ago in bug 1634555.

Now we intend to deprecate Legacy Telemetry's recordEvent API in bug 1918702. To do that we can

1. Remove the Legacy Telemetry events, leaving the Glean ones.
   • If no one's using the Legacy Telemetry events, this'd be the ideal choice.
2. Collect the Legacy Telemetry events via mirroring through the Glean API
   • This will have two very similarly-structured and -named events in Glean, but will maintain Legacy Telemetry event collection

:Standard8, have the Legacy Telemetry events been used these past two years? Can we remove them and their calls? Or should we preserve their collection.

Comment 1

[Mark Banner (:standard8) (afk until 21st July)]

The dashboards I have use the Glean version of the events, I'm asking around in case there's legacy usages.

Comment 2

[Florian Quèze [:florian]]

Attachment: Bug 1920723 - remove search.engine legacy telemetry event that has been replaced by a Glean event, r=Standard8.

Comment 3

[Mark Banner (:standard8) (afk until 21st July)]

I've not had any mentions that we need to keep this, so I think we can go ahead and remove the legacy events.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/276f9948628e
remove search.engine legacy telemetry event that has been replaced by a Glean event, r=Standard8,settings-reviewers.

Comment 5

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/276f9948628e