`net/http` instrumentation (eBPF probe) is not working

[ocampeau]

`net/http` instrumentation (eBPF probe) is not working

[edeNFed]

Hi @hnhbwlp this fix will be released in the next version (probably next week)

[edeNFed]

Thank you for the detailed explanation!
I think a good solution may be replacing the following line
https://github.com/keyval-dev/opentelemetry-go-instrumentation/blob/c4799f7d5fbedcd7533daf0e4103449c6ed156e5/pkg/instrumentors/bpf/net/http/server/bpf/probe.bpf.c#L84
with

void* httpReq_ptr = find_context_in_map(ctx_iface, &context_to_http_events);

The method find_context_in_map handles this case precisely, it walks up the context tree until it finds a parent context in the provided map. (you can see similar usage in the gRPC server instrumentor).

Please let me know if this fix works, I'll be happy to merge a fix.

[ocampeau]

Thanks for your reply. Sure, I'll look into that and once I get something working I send a PR.

[hnhbwlp]

@edeNFed Is there any new progress on this issue, I have encountered the same problem.

[mariomac]

I tried the approach from the PR #48 and it didn't succeed in my case. How get_argument_by_stack is supposed to work in Go 1.17 if the arguments are passed by register?

[mariomac]

This fix in the homonymous repo worked for me: open-telemetry/opentelemetry-go-instrumentation#34