Closed Bug 1938223 Opened 9 months ago Closed 9 months ago

Extend (quietly) GIFFT to support labeled_counter -> keyed count and -> categorical histogram mirroring

Categories

(Toolkit :: Telemetry, enhancement, P2)

enhancement

Tracking

()

RESOLVED FIXED
135 Branch
Tracking Status
firefox135 --- fixed

People

(Reporter: chutten, Assigned: chutten)

References

Details

Attachments

(1 file)

There are 16 (2 of which are TELEMETRY_TEST_*) kind: count, keyed: true histograms according to $ cat toolkit/components/telemetry/Histograms.json | jq 'to_entries | .[] | {key: .key, kind: .value.kind, keyed: .value.keyed} | select(.kind == "count") | select(.keyed) | .key'

(please do not judge my neophyte jq use too harshly. I'm learning)

The direct way to collect this shape of information in Glean would be by labeled_counter. Right now we support keyed scalar and hgram of kind: boolean. We need to add in support for hgrams of kind: count and keyed: true.

Turns out the mapping to histograms with kind: categorical is straightforward to add while I'm in the neighbourhood.

Summary: Extend (quietly) GIFFT to support labeled_counter -> keyed count histogram mirroring → Extend (quietly) GIFFT to support labeled_counter -> keyed count and -> categorical histogram mirroring

These are secret menu items. To use them:

  • Metric of type: labeled_counter
  • telemetry_mirror must have h# prefix before the histogram
  • For categorical:
    • Samples must always be 1, or we'll assert
    • Labels must be specified in both probes with the same spelling, case, and
      order

Tests have been added to t/c/t/tests/python/test_mirrors.py to ensure these
restrictions are adhered to.

Pushed by chutten@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/ed191e0ecbd0 Support mirroring specific labeled_counter metrics to histograms of kind: categorical and keyed histograms of kind: count r=TravisLong
Status: ASSIGNED → RESOLVED
Closed: 9 months ago
Resolution: --- → FIXED
Target Milestone: --- → 135 Branch
Depends on: 1941763
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: