-
Notifications
You must be signed in to change notification settings - Fork 149
feat:(opentelemetry_broadway): add context propagation integration #540
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
base: main
Are you sure you want to change the base?
Conversation
79183e2
to
015bb74
Compare
Nice 👍 I'll review this today. |
instrumentation/opentelemetry_broadway/lib/opentelemetry_broadway.ex
Outdated
Show resolved
Hide resolved
6fe3da2
to
0a73bff
Compare
0a73bff
to
59eb992
Compare
Hey @tsloughter any updates here? |
@whatyouhide could you share some blessing here? 🙏🏻 |
I think this should be enabled by default, https://github.com/open-telemetry/semantic-conventions/blob/main/docs/messaging/messaging-spans.md Another piece, which can wait for a future PR, is it should optionally be able to make the producer the parent and not just a link. |
59eb992
to
eeca580
Compare
@tsloughter I made it enabled by default |
Signed-off-by: Yordis Prieto <[email protected]>
154e46c
to
a02bc86
Compare
@whatyouhide you are owner, it seems you must approve it |
No description provided.