odigos-io / opentelemetry-go-instrumentation

OpenTelemetry auto-instrumentation for Go applications
Apache License 2.0
292 stars 44 forks source link

Remove OTEL_TARGET_EXE and auto discover processes #23

Open edeNFed opened 2 years ago

edeNFed commented 2 years ago

This issue is for removing the OTEL_TARGET_EXE env variable and implementing process discovery instead. Implementing this will make integration with OpenTelemetry Operator easier and overall provide a better experience to users (one less argument to specify).

I suggest splitting this into two phases: phase 1: Apply automatic instrumentation to the first Go process found (there is already logic to detect Go processes at analyze.go phase 2: Apply automatic instrumentation to any Go process found (multi-process support)