-
Notifications
You must be signed in to change notification settings - Fork 642
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
[Feature] Support Opentelemetry #4857
Comments
Are there any specific implementation requirements? |
@okareyou Welcome. You may refer to https://eventmesh.apache.org/docs/design-document/observability/metrics-export/ chapter. |
I checked out the EventMesh code and it looks like it's using OpenTelemetry for both metrics and tracing. Does that mean it's already set up with the OpenTelemetry SDK? Could I be missing something? Any advice would be great.
|
I'm not familier with trace-plugin yet. eventmesh/eventmesh-trace-plugin/eventmesh-trace-api/build.gradle Lines 23 to 24 in e349def
May be we need some enhancement for trace-plugin? @qqeasonchen |
@okareyou You may want to look at the trace-plugin to see if it has implemented the |
Search before asking
Feature Request
Integrate opentelemetry SDK for tracing collecting.
Are you willing to submit PR?
Code of Conduct
The text was updated successfully, but these errors were encountered: