-
Notifications
You must be signed in to change notification settings - Fork 1.8k
Upgrade OTLP to 1.8.0 #13758
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
Upgrade OTLP to 1.8.0 #13758
Conversation
Codecov Report❌ Patch coverage is ❌ Your patch status has failed because the patch coverage (93.21%) is below the target coverage (95.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #13758 +/- ##
==========================================
- Coverage 91.44% 91.39% -0.06%
==========================================
Files 640 644 +4
Lines 42665 42553 -112
==========================================
- Hits 39016 38890 -126
- Misses 2831 2841 +10
- Partials 818 822 +4 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
florianl
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I focused on pprofiles and its related changes.
Upgrade OTLP to 1.8.0. This is the counterpart of open-telemetry/opentelemetry-collector#13758 --------- Co-authored-by: Edmo Vamerlatti Costa <[email protected]>
#### Description Fix `protoID`s that were changed with breaking changes of the Profiling protocol and were missed in the initial change #13758. --------- Signed-off-by: Florian Lehner <[email protected]>
This PR upgrades OTLP to 1.8.0.
See https://github.com/open-telemetry/opentelemetry-proto/releases/tag/v1.8.0
-contrib counterpart coming soon (I will move this out of draft once both PRs are ready).