Search Telemetry reporting should match the codes that were used before the conversion to WebExtensions

RESOLVED FIXED in Firefox 68

Status

()

defect
P1
normal
RESOLVED FIXED
Last month
20 days ago

People

(Reporter: standard8, Assigned: daleharvey)

Tracking

(Regression, {regression})

unspecified
Firefox 69
Points:
---

Firefox Tracking Flags

(firefox-esr60 unaffected, firefox66 unaffected, firefox67 unaffected, firefox68+ fixed, firefox69 fixed)

Details

Attachments

(1 attachment)

Reporter

Description

Last month

We've identified that our telemetry reporting for search engines has changed the reports since the conversion to WebExtensions.

The values appear that they used to be based on the values in list.json, however they are now just the extension short names. In some cases this looses us slightly more detailed information as to exactly which versions of the engines are in use.

[Tracking Requested - why for this release]: This affects are monitoring and reporting.

Assignee

Updated

27 days ago
Assignee: nobody → dharvey

Comment 2

23 days ago
Pushed by dharvey@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/37b6e7e47cce
Restore engine shortName to pre webextensions value. r=Standard8

Comment 4

23 days ago
Backout by shindli@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/181ef326f7a8
Backed out changeset 37b6e7e47cce for causing Mn failures in browser\components\search\test\marionette\test_engines_on_restart.py CLOSED TREE

Comment 5

23 days ago
Pushed by dharvey@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b404f7a62f2f
Restore engine shortName to pre webextensions value. r=Standard8

Comment 6

22 days ago
bugherder
Status: NEW → RESOLVED
Closed: 22 days ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 69
Assignee

Updated

21 days ago
Flags: needinfo?(dharvey)
Assignee

Comment 7

21 days ago

Comment on attachment 9066208 [details]
Bug 1552193 - Restore engine shortName to pre webextensions value. r?Standard8

Beta/Release Uplift Approval Request

  • User impact if declined: Not user facing, little user impact
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Nightly?: No
  • Needs manual test from QE?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): Changes the name reported via telemetry so existing querying can be maintained, the functional impact is minimal and covered by tests
  • String changes made/needed:
Attachment #9066208 - Flags: approval-mozilla-beta?

Comment on attachment 9066208 [details]
Bug 1552193 - Restore engine shortName to pre webextensions value. r?Standard8

search telemetry fix, approved for 68.0b5

Attachment #9066208 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.