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

Beyla not sending headers on Java (Spring Boot) but will send them through Go (HTTP1.1 calls) #1338

Open
ddz99 opened this issue Nov 7, 2024 · 2 comments

Comments

@ddz99
Copy link

ddz99 commented Nov 7, 2024

I have an issue/question. Supposedly Beyla should be capable to rewrite headers to send the trace parent even in Java applications, right?
I made a demo to showcase to my company so that we could use Beyla but I cant seem to figure out why it doesnt work.
Go apps do send the traceparent in the request headers, but Spring apps do not. Is there a reason for this?

Java Spring is working on HTTP1.1

Kernel is 5.15 on WSL2 Ubuntu but I've tried this on 4.18 RHEL8 aswell.

Running Beyla 1.90 built from Main branch on this repo.

@ddz99 ddz99 changed the title Beyla not sending headers on Java (Spring Boot) but will send them through Go Beyla not sending headers on Java (Spring Boot) but will send them through Go (HTTP1.1 calls) Nov 7, 2024
@grcevski
Copy link
Contributor

grcevski commented Nov 7, 2024

Hi @ddz99 , we are still working on this and it will happen soon. Two things we need to finish:

  1. The context propagation support. If you'd like you can try this today on the main branch if you set this option BEYLA_BPF_TC_L7_CP=true. It's not documented because we are still working on it. If you have Beyla on both sides of the communication you can also try BEYLA_BPF_TC_CP=true, but don't use both of these options together. The current form of BEYLA_BPF_TC_L7_CP needs kernel 5.17 (which seems newer than what you have), I expect that in 2 weeks this requirement will not be there anymore.
  2. We have issues with certain Java thread pools, so depending on how you use the Spring rest client it may not work. We have plan to fix this in the next month or so.

Please give this a shot and let me know if it works.

@grcevski
Copy link
Contributor

grcevski commented Nov 7, 2024

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants