Closed Bug 1702263 Opened 3 years ago Closed 3 years ago

`glean_parser` forbids labeled metrics having more than sixteen pre-defined labels

Categories

(Data Platform and Tools :: Glean: SDK, defect, P1)

Product:
Data Platform and Tools
Component:
Glean: SDK
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: chutten, Assigned: chutten)

References

Details

(Whiteboard: [telemetry:glean-rs:m?])

Attachments

(2 files)

pr 297: bug 1702263 - Remove limit on statically-defined labels
48 bytes, text/x-github-pull-request
Details | Review
Link to GitHub pull-request: https://github.com/mozilla/glean_parser/pull/297
48 bytes, text/x-github-pull-request
Details | Review

According to the docs we should only limit the number of labels to 16 if they are specified dynamically.

But glean_parser has this line in the schema, which prevents this, giving an error like <list of labels> is not valid under any of the given schemas. <list of labels> is too long <list of labels> is not of type 'null'

Blocks: 1702280

chutten merged PR #297: "bug 1702263 - Raise limit on statically-defined labels to 100" in 8522c49.

Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: