Skip to content
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

Zipkin (and OpenTracing) tracing providers do not provide context propagation #8848

Closed
tjquinno opened this issue Jun 5, 2024 · 0 comments · Fixed by #9119
Closed

Zipkin (and OpenTracing) tracing providers do not provide context propagation #8848

tjquinno opened this issue Jun 5, 2024 · 0 comments · Fixed by #9119
Assignees
Labels
4.x Version 4.x bug Something isn't working Monday P3 tracing

Comments

@tjquinno
Copy link
Member

tjquinno commented Jun 5, 2024

Environment Details

  • Helidon Version: 4.x
  • Helidon SE or Helidon MP
  • JDK version:
  • OS:
  • Docker version (if applicable):

Problem Description

The Jaeger and OpenTelemetry tracing providers both provide context propagation.

But neither the OpenTracing provider nor the Zipkin provider does so.

Addressing #8846 would allow the Zipkin provider to do propagation very similarly to how our OTel provider does. That change plus this one would put Zipkin on an equal footing with OTel and Jaeger as providers.

Given that OpenTracing is essentially retired we might not choose to invest the time to add propagation for it. But Zipkin probably should do propagation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4.x Version 4.x bug Something isn't working Monday P3 tracing
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants