Search Telemetry reporting should match the codes that were used before the conversion to WebExtensions
Categories
(Firefox :: Search, defect, P1)
Tracking
()
Tracking | Status | |
---|---|---|
firefox-esr60 | --- | unaffected |
firefox66 | --- | unaffected |
firefox67 | --- | unaffected |
firefox68 | + | fixed |
firefox69 | --- | fixed |
People
(Reporter: standard8, Assigned: daleharvey)
References
(Regression)
Details
(Keywords: regression)
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
jcristau
:
approval-mozilla-beta+
|
Details |
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.
Updated•5 years ago
|
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 1•5 years ago
|
||
Comment 3•5 years ago
|
||
Backed out changeset 37b6e7e47cce (Bug 1552193) for causing Mn failures in browser\components\search\test\marionette\test_engines_on_restart.py CLOSED TREE
https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&classifiedState=unclassified&fromchange=c347f878342197b00ce58f30d86d1b436abfd6ac&selectedJob=248236351
https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=248236351&repo=autoland&lineNumber=70228
Comment 6•5 years ago
|
||
bugherder |
Assignee | ||
Updated•5 years ago
|
Assignee | ||
Comment 7•5 years 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:
Comment 8•5 years ago
|
||
Comment on attachment 9066208 [details]
Bug 1552193 - Restore engine shortName to pre webextensions value. r?Standard8
search telemetry fix, approved for 68.0b5
Comment 9•5 years ago
|
||
bugherder uplift |
Assignee | ||
Updated•5 years ago
|
Updated•3 years ago
|
Description
•