-
Notifications
You must be signed in to change notification settings - Fork 6
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
Otel #355
Otel #355
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #355 +/- ##
==========================================
+ Coverage 87.17% 87.65% +0.47%
==========================================
Files 42 44 +2
Lines 1645 1725 +80
==========================================
+ Hits 1434 1512 +78
- Misses 211 213 +2 ☔ View full report in Codecov by Sentry. |
what stops this from merging? @keithralphs ? |
@stan-dot I believe this PR is a demo/illustration, it was never intended to be merged. The intention was to make a PR for production if this approach was generally favoured. |
@keithralphs since the observability utils have been merged, can this PR be closed now? |
Yes, this can be closed as a duplicate of #669 |
An Illustration of various ways to instrument BlueAPI to add distributed tracing
!!!!!!! DO NOT MERGE !!!!!!!!