open-telemetry / opentelemetry-js-contrib

OpenTelemetry instrumentation for JavaScript modules
https://opentelemetry.io
Apache License 2.0
677 stars 497 forks source link

Container detector - cgroupv1 and v2 - prefix support & fix #2339

Open abhee11 opened 2 months ago

abhee11 commented 2 months ago

Is your feature request related to a problem? Please describe

There is an issue where sometimes using lightweight k8 cgroup v1 or v2 - prefix may be there in front of the containerId - current implementation does not account for it.

Describe the solution you'd like to see

Will need to add a check ( regex to filter crio, crio-containerd and containers prefix before the containerId extraction

Describe alternatives you've considered

Additional context

abhee11 commented 2 months ago

Please assign it to me.