eventmesh icon indicating copy to clipboard operation
eventmesh copied to clipboard

[Feature] Support Opentelemetry

Open qqeasonchen opened this issue 1 year ago • 6 comments

Search before asking

  • [X] I had searched in the issues and found no similar issues.

Feature Request

Integrate opentelemetry SDK for tracing collecting.

Are you willing to submit PR?

  • [ ] Yes I am willing to submit a PR!

Code of Conduct

qqeasonchen avatar Apr 22 '24 08:04 qqeasonchen

Are there any specific implementation requirements?

okareyou avatar Apr 22 '24 15:04 okareyou

@okareyou Welcome. You may refer to https://eventmesh.apache.org/docs/design-document/observability/metrics-export/ chapter.

Pil0tXia avatar Apr 22 '24 16:04 Pil0tXia

https://opentelemetry.io/docs/languages/java/ @okareyou

qqeasonchen avatar Apr 23 '24 08:04 qqeasonchen

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.

https://opentelemetry.io/docs/languages/java/ @okareyou

@okareyou Welcome. You may refer to https://eventmesh.apache.org/docs/design-document/observability/metrics-export/ chapter.

okareyou avatar May 02 '24 08:05 okareyou

@okareyou

I'm not familier with trace-plugin yet. opentelemetry-sdk has been introduced in trace-plugin api:

https://github.com/apache/eventmesh/blob/e349def6a4eba61b94a8e5b8aeb4ddf9745cfbe7/eventmesh-trace-plugin/eventmesh-trace-api/build.gradle#L23-L24

May be we need some enhancement for trace-plugin? @qqeasonchen

Pil0tXia avatar May 02 '24 10:05 Pil0tXia

@okareyou You may want to look at the trace-plugin to see if it has implemented the tracing collecting feature, and if this feature supports OpenTelemetry.

Pil0tXia avatar Jun 12 '24 07:06 Pil0tXia