-
Notifications
You must be signed in to change notification settings - Fork 268
feat: migrate otelgrpc to stats handlers and update dependencies #4895
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
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## v2 #4895 +/- ##
==========================================
- Coverage 60.29% 60.27% -0.03%
==========================================
Files 136 136
Lines 13375 13373 -2
==========================================
- Hits 8065 8061 -4
- Misses 4629 4630 +1
- Partials 681 682 +1
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
80848db to
d6df69e
Compare
3e60ce8 to
2667d91
Compare
|
thank you! id like to wait for this to get merged into the upstream instead of depending on a fork if possible |
2667d91 to
35f86b5
Compare
e4da2f2 to
d9d0986
Compare
Modernize OpenTelemetry gRPC instrumentation by migrating from deprecated interceptors to the newer stats handler API supported by grpchan. Signed-off-by: Roman Dmytrenko <[email protected]>
Signed-off-by: Roman Dmytrenko <[email protected]>
Signed-off-by: Roman Dmytrenko <[email protected]>
Signed-off-by: Roman Dmytrenko <[email protected]>
d9d0986 to
9dc2109
Compare
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.
awesome! thank you for taking this on
Modernize OpenTelemetry gRPC instrumentation by migrating from deprecated
interceptors to the newer stats handler API supported by grpchan.