-
Notifications
You must be signed in to change notification settings - Fork 196
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
Switch to OpenTelemetry #392
Comments
I have this (kinda) working on my fork at https://github.com/marcwickenden/eventhorizon/tree/opentelemetry. I haven't submitted a PR because it's not fully tested yet (not sure if propagation is reliable) and I've also implemented it in a way that is not backwards compatible with the existing version. Might be useful though? |
Thanks a lot @marcwickenden ! That will help me getting started 👍 |
Great idea! I would be glad to accept a PR! |
Hey @janiskemper, just curious if you've made any progress on this or would like help? Would be great to see OTEL support in EventHorizon. |
Hey, yes I actually worked on it and did make some progress, but never reached the state that it would be possible to make a PR in this project. I stopped working on it eventually because I realized that my use case was not fully covered by this project. Therefore, I shifted my work a bit away. Currently, I am focusing on very different topics, so I won't be able to continue with this. Sorry! |
Is your feature request related to a problem? Please describe.
OpenTracing is deprecated in favor of OpenTelemetry (opentracing/specification#163)
Describe the solution you'd like
Switch to OpenTelemetry.
The text was updated successfully, but these errors were encountered: