Open Bug 2083946 Opened 25 days ago Updated 3 days ago

Add PerfMetrics to glean probes

Categories

(Core :: Disability Access APIs, task)

task

Tracking

()

ASSIGNED

People

(Reporter: morgan, Assigned: morgan)

References

(Depends on 1 open bug, Blocks 3 open bugs)

Details

Attachments

(1 obsolete file)

Tracking our perfmetrics in glean probes will allow us to understand the variety of performance experiences users of the a11y engine are facing.

This data will help us in a few ways:

  1. It'll help us prioritise existing performance work. We will prioritise bugs that are indicative of especially poor performance over those that fall close to or within "average" performance profiles for users.
  2. It'll help us write tests that better approximate "poor performance".

In the convo I had with the perf folks at the recent engineering summit, I was told we should target the 95th percentile of in-the-wild performance as our "poor performance" benchmark.

Blocks: a11yperf
Summary: Add PerfMetrics to telemetry probes → Add PerfMetrics to glean probes

This patch adds Glean metrics for all of our existing PerfStats metrics, and passes them into the
existing PerfStats auto recorder constructors.
With D337305, the durations recorded by the PerfStats metrics are mirrored to Glean.

There's a small discrepancy in the units we use for recording: the Glean metrics use nanoseconds
while the PerfStats metrics use miliseconds. This is because Glean records into an integer while
PerfStats records into a double. Based on my local runs and try runs on the reference hardware,
we aren't overflowing these fields, but we do have a wide range (1ns for some probes, 10+ms for others).

Assignee: nobody → mreschenberg
Attachment #9669173 - Attachment description: WIP: Bug 2083946 - Mirror accessibility PerfStats metrics to Glean. r?#accessibility-platform-reviewers → Bug 2083946 - Mirror accessibility PerfStats metrics to Glean. r?#accessibility-platform-reviewers
Status: NEW → ASSIGNED
Depends on: 1898810
Attachment #9669173 - Attachment is obsolete: true
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: