Closed Bug 1572468 Opened 3 months ago Closed 2 months ago

Add telemetryId's to engines.json

Categories

(Firefox :: Search, task, P2)

task
Points:
2

Tracking

()

RESOLVED FIXED
Firefox 71
Iteration:
71.2 - Sept 16 - 29
Tracking Status
firefox71 --- fixed

People

(Reporter: daleharvey, Assigned: daleharvey)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

So I will leave this open until we are all sure but I dont think this is necessary, we can still generate the same shortName using the new configuration format

(In reply to Dale Harvey (:daleharvey) from comment #1)

So I will leave this open until we are all sure but I dont think this is necessary, we can still generate the same shortName using the new configuration format

I think we should add the telemetryId - when we are able to start re-arranging some of the search add-ons (e.g. merge locales or rename them), I think we're going to need it. Adding it at this stage will give us a simple way to verify that we're reporting the id that we expect.

Priority: -- → P2
Assignee: nobody → dharvey
Iteration: --- → 71.2 - Sept 16 - 29
Points: --- → 2

So the above patch matches the modern configuration telemetryIds with the ones list.json will us. Its a simple field we need to add to every configuration we want to distinguish in telemetry

Mike Connor I only matched the telemetryId's for the 6 engines we test, mostly because if we are that bothered about getting it right we should add tests for it and we only have tests for those engines, this means twitter / wikipedia will report as "twitter" and not "twitter-jp" etc, is that ok?

Flags: needinfo?(mconnor)
Pushed by dharvey@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/365f3fcf9ba1
Add telemetryIds to modern config. r=Standard8
Status: NEW → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 71

(In reply to Dale Harvey (:daleharvey) from comment #4)

So the above patch matches the modern configuration telemetryIds with the ones list.json will us. Its a simple field we need to add to every configuration we want to distinguish in telemetry

Mike Connor I only matched the telemetryId's for the 6 engines we test, mostly because if we are that bothered about getting it right we should add tests for it and we only have tests for those engines, this means twitter / wikipedia will report as "twitter" and not "twitter-jp" etc, is that ok?

Mike answered this elsewhere and commented that this was fine.

Flags: needinfo?(mconnor)
You need to log in before you can comment on or make changes to this bug.