Closed Bug 1834792 Opened 1 year ago Closed 1 year ago

Make Glean ping throttling limits configurable via Nimbus

Categories

(Toolkit :: Telemetry, task, P2)

task

Tracking

()

RESOLVED FIXED
118 Branch
Tracking Status
firefox118 --- fixed

People

(Reporter: chutten, Assigned: perry.mcmanis)

References

Details

Attachments

(2 files)

Current numbers suggest that PingCentre Replacement with Glean shouldn't blow out our budget for sending frequency:

  • Per docs we attempt upload of 15 pings per 60s interval. With bug 1647630 we will be able to configure both the number of the pings and the length of the interval.
  • Ping volumes for all namespaces from PingCentre: most days most clients send fewer than 15 pings. 99th %ile is at 186/day which would take 13 intervals of 60s to clear with current configuration

So this should be fine... but we don't want to lose data.

This bug is about exposing the new configuration options from bug 1647630 to Nimbus so that, should we find ourselves missing some data (which we will detect on the to-be-build pingcentre replacement system validation dashboard), we can remotely loosen our throttling restrictions.

Assignee: nobody → pmcmanis
Attachment #9341980 - Attachment description: WIP: BUG Bug 1834792 - Make Glean ping throttling limits configurable via Nimbus r?TravisLong → Bug 1834792 - Make Glean ping throttling limits configurable via Nimbus r?TravisLong
Attached file Data Review Request
Data review request:

Travis I have a data review request for you please and thank you!

Flags: needinfo?(tlong)

Comment on attachment 9342437 [details]
Data Review Request

Data Review

  1. Is there or will there be documentation that describes the schema for the ultimate data set in a public, complete, and accurate way?

Yes, through the metrics.yaml file and the Glean Dictionary.

  1. Is there a control mechanism that allows the user to turn the data collection on and off?

Yes, through the telemetry preferences in the application settings.

  1. If the request is for permanent data collection, is there someone who will monitor the data over time?

Permanent collection to be monitored by pmcmanis@mozilla.com with glean-team as a fallback

  1. Using the category system of data types on the Mozilla wiki, what collection type of data do the requested measurements fall under?

Category 1, Technical data

  1. Is the data collection request for default-on or default-off?

Default-on

  1. Does the instrumentation include the addition of any new identifiers (whether anonymous or otherwise; e.g., username, random IDs, etc. See the appendix for more details)?

No

  1. Is the data collection covered by the existing Firefox privacy notice?

Yes

  1. Does the data collection use a third-party collection tool?

No

Result

data-review+

Flags: needinfo?(tlong)
Attachment #9342437 - Flags: data-review+

FYI, when you attach the data-review request, you should select the ? next to the data-review option and then enter a data-steward's email in the field next to it rather than using needinfo.

There is an r+ patch which didn't land and no activity in this bug for 2 weeks.
:perry.mcmanis, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit BugBot documentation.

Flags: needinfo?(tlong)
Flags: needinfo?(pmcmanis)

I think this is waiting on an update to Glean in m-c, which has been held up for other reasons. See also https://bugzilla.mozilla.org/show_bug.cgi?id=1840044

Flags: needinfo?(tlong)
Flags: needinfo?(pmcmanis)
Depends on: 1840044

More precisely it's even waiting on yet unreleased changes.

Pushed by pmcmanis@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/2c4954ceb46a Make Glean ping throttling limits configurable via Nimbus r=TravisLong,supply-chain-reviewers
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 118 Branch

Note that as written the gleanMaxPingsPerMinute can currently not be experimented on properly, due to the lack of isEarlyStartup: true for the feature which is required for the C++ API to work.
This is handled in bug 1848331

You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: