Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Permit non-alphabetic characters as the 1st character in metric names #4371

Open
heyams opened this issue Jan 16, 2025 · 3 comments
Open

Permit non-alphabetic characters as the 1st character in metric names #4371

heyams opened this issue Jan 16, 2025 · 3 comments
Labels
spec:metrics Related to the specification/metrics directory triage:deciding:community-feedback Open to community discussion. If the community can provide sufficient reasoning, it may be accepted

Comments

@heyams
Copy link

heyams commented Jan 16, 2025

https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/metrics/api.md#instrument-name-syntax

Image

We are looking to start using a dotnet OpenTelemetry distribution. Unfortunately, our metric names begin with a '-' character, which is not allowed as the first character.

Changing the metric names is not a feasible option for us due to the high cost and the potential invalidation of numerous queries accumulated across various tools, dashboards, and users. Therefore, we hope that a feature request could be considered to allow non-alphabetic characters as the first character of metric names. This would enable us to fully utilize the features and benefits provided by OpenTelemetry metrics.

Thanks,
Helen

@heyams heyams added the spec:metrics Related to the specification/metrics directory label Jan 16, 2025
@heyams heyams changed the title Permit non-alphabetic characters as the initial character in metric names Permit non-alphabetic characters as the 1st character in metric names Jan 16, 2025
@regexrowboat
Copy link

👍

@svrnm svrnm added the triage:deciding:community-feedback Open to community discussion. If the community can provide sufficient reasoning, it may be accepted label Jan 20, 2025
@svrnm
Copy link
Member

svrnm commented Jan 20, 2025

hey thanks for your proposal, this needs a broader discussion, I recommend that you attend the SIG meeting to get some feedback: https://github.com/open-telemetry/community/?tab=readme-ov-file#specification-sigs

@cijothomas
Copy link
Member

Previous relevant Issue/PR that removed some restriction on metric name : #3422

This keeps coming up. "/" is now allowed, but there are asks to support longer-than-63 length, ":", " " (space). And this issue which is asking to relax 1st character restriction.

Will join the next Spec SIG meeting to discuss how to unblock users.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spec:metrics Related to the specification/metrics directory triage:deciding:community-feedback Open to community discussion. If the community can provide sufficient reasoning, it may be accepted
Projects
None yet
Development

No branches or pull requests

4 participants