reporter: use semantic convention to report build_id

open-telemetry / opentelemetry-ebpf-profiler

The production-scale datacenter profiler (C/C++, Go, Rust, Python, Java, NodeJS, .NET, PHP, Ruby, Perl, ...)

Apache License 2.0

2.31k stars 244 forks source link

reporter: use semantic convention to report build_id #153

Open florianl opened 5 days ago

florianl commented 5 days ago

https://github.com/open-telemetry/semantic-conventions/pull/1329 got merged, which introduces a semantic convention for build_ids. Use this semantic convention in favor of BuildIdKind, that will be dropped with https://github.com/open-telemetry/opentelemetry-proto/pull/584.

florianl commented 1 day ago

Do we want to also support sending the Go build id as per spec?

As there is a higher demand for process.executable.build_id.gnu and process.executable.build_id.profiling I priorized their implementation with this change. process.executable.build_id.go is planned to follow.