elastic / opentelemetry

Get started with our Elastic Distros of OpenTelemetry
11 stars 8 forks source link

Adding fix for host.name field update based on k8s.node.name #37

Closed gizas closed 1 month ago

gizas commented 1 month ago

Adds the needed resource/hostname processor that will update the host.name field based on k8s.node.name

Before: 379182423-1dfa46cd-a26d-45c4-b70d-6199e64ffa48

After: Screenshot 2024-10-24 at 5 12 51 PM

Relates:

Closes: https://github.com/elastic/opentelemetry-dev/issues/516 Found in: https://github.com/elastic/opentelemetry-dev/issues/559

rogercoll commented 1 month ago

@lahsivjar @AlexanderWert We first thought that this would be fixed with the elastictrace processor + the corresponding enrichment: https://github.com/elastic/opentelemetry-lib/pull/108

The issue is that the processor only enriches traces, meaning that we cannot use the processor in metrics/logs OTLP pipeline. The configuration changes used in this PR uses the resource processor to do the mapping for any signal type.