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

Can't propagate trace headers with actors #1151

Closed
alicejgibbons opened this issue Oct 21, 2024 · 2 comments · May be fixed by #1154
Closed

Can't propagate trace headers with actors #1151

alicejgibbons opened this issue Oct 21, 2024 · 2 comments · May be fixed by #1154
Assignees
Labels
kind/bug Something isn't working

Comments

@alicejgibbons
Copy link

Expected Behavior

I should be able to use opentelemetry SDK to propagate trace headers from the actor client to server.

Actual Behavior

Today, providing a traceparent header in the actor client doesn't propagate to the actor server while using the latest version of Dapr (1.14.4) and the Java SDK. Additionally following the example here still causes the actor client and actor server traces to show up with two separate trace IDs.

Steps to Reproduce the Problem

Run actor client and actor server apps, and use the open telemetry SDK to propagate the trace header through from client -> server. Print the headers out on each side to see that they are different.

Release Note

RELEASE NOTE: FIX Propagate traces in actors

@alicejgibbons alicejgibbons added the kind/bug Something isn't working label Oct 21, 2024
@artursouza artursouza self-assigned this Oct 21, 2024
@artursouza
Copy link
Member

I got it to work without any SDK change, see this new test version: https://github.com/dapr/java-sdk/pull/1154/files#r1839036940

@alicejgibbons
Copy link
Author

Closing this since there is a workaround as linked

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants