-
Notifications
You must be signed in to change notification settings - Fork 849
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
Increase metric name maximum length from 63 to 255 characters #5697
Increase metric name maximum length from 63 to 255 characters #5697
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #5697 +/- ##
============================================
- Coverage 91.28% 91.26% -0.02%
- Complexity 5016 5032 +16
============================================
Files 554 556 +2
Lines 14860 14861 +1
Branches 1387 1389 +2
============================================
- Hits 13565 13563 -2
- Misses 893 896 +3
Partials 402 402
☔ View full report in Codecov by Sentry. |
Should hold off on this for 1.29.0 release since this didn't make it into the 1.24.0 spec release. |
makes sense based on open-telemetry/opentelemetry-specification#2728 (comment) |
I'm really hoping this makes it into 1.30.0 🤞🤞 |
Pending spec PR open-telemetry/opentelemetry-specification#3648Spec update is merged!