Closed Bug 1861676 Opened 9 months ago Closed 9 months ago

Extend and update SERP telemetry configuration for mobile

Categories

(Firefox :: Search, task, P2)

task

Tracking

()

RESOLVED FIXED
121 Branch
Tracking Status
firefox121 --- fixed

People

(Reporter: standard8, Assigned: standard8)

References

Details

(Whiteboard: [sng])

Attachments

(2 files)

Splitting this out from the first part of bug 1860531, since that needs to be uplifted.

+++ This bug was initially created as a clone of Bug #1860531 +++

In bug 1834842, the android client is currently adding remote settings support for their SERP telemetry. We want them to use the same configuration as desktop, but there are some differences that we'll need to extend the current SERP telemetry collection with a new field.

The new field should be called queryParamNames and is an array of strings. The values of this will duplicate queryParamName, but be added to so that we can support mobile, which needs different parameter names for Baidu.

Older desktop clients will continue to use queryParamName, however new desktop clients and the mobile clients will use queryParamNames.

We'll also need to make sure that the rest of the current configuration also works for mobile.

Moving across from bug 1860531

The remote settings changes are now approved on stage. Please can QA test this?

For testing:

  • Existing builds should be unaffected by the changes wrt to SERP telemetry for most engines that we record, except for Baidu.
  • Builds with the full set of patches, should also be unchanged for all engines except Baidu.
    • These try builds have the full set of patches. We'll land the other patch once the changes are on production.
    • Baidu will now also record for urls such as https://m.baidu.com/s?word=test (note: change of parameter name).
  • Google has been updated to adjust the ad server regular expressions. For desktop, this should not cause a change in the recording of ads on pages.
Flags: needinfo?(cbaica)

Tested the new changes to Baidu SERP telemetry from staging for both existing builds and the provided try builds. The full testing coverage can be viewed here.

Also per Marks instructions we also ran a small smoke check for Google ad recordings in telemetry. Results for the testing coverage can be viewed here.

Flags: needinfo?(cbaica)

The remote settings changes are on production as of about 11:49 UTC today.

Leaving open to land the additional desktop changes once the production has been synced to the repository.

Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/eb4c4b32c59b
Use the new queryParamNames SERP telemetry parameter by default. r=search-reviewers,jteow

Fixing.

Flags: needinfo?(standard8)
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/51901c4ba87a
Use the new queryParamNames SERP telemetry parameter by default. r=search-reviewers,jteow
Status: NEW → RESOLVED
Closed: 9 months ago
Resolution: --- → FIXED
Target Milestone: --- → 121 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: