Eden Federman

icon indicating an email [email protected]

icon location CTO @odigos-io Making OpenTelemetry and eBPF accessible

Results 64 comments of Eden Federman

use DockerImage from ENV variable

Hi @ashish0x02 We currently have a flag named --image-prefix which you can use during installation to use private image repos instead of docker hub.

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

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 ```c void* httpReq_ptr = find_context_in_map(ctx_iface, &context_to_http_events); ``` The method `find_context_in_map` handles...

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

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

Adding net/http client instrumentor

So sorry for the huge delay! I promise to finish looking on this by the end of this week

How to add request-id/trace-id to http request header?

Hi @yfractal You are right, appending to map is more complex than appending to slice and is not a logic we implemented yet. Until we implement the map operations the...

How to add request-id/trace-id to http request header?

No worries, this will be added very soon

Generate ID for x-Ray

Sounds good, I will be happy to accept a PR.

Permanent error: invalid temporality and type combination; invalid temporality and type combination

I think this is related: https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/15281

Add verification phase to CLI before installing

> could you please elaborate the `Check kernel versions of running nodes` part @edeNFed ? thank you 🙇 We should check that at least one Kubernetes node has kernelVersion above...

Log receivers should be from source namespace only

Hi @edwardwu Thanks for reporting! this is indeed a known bug in the current implementation. If you would like to open a PR to fix it, we will be happy...