Skip to content

[exporter/elasticsearch] Remove host.os.type encoding in ECS mode#46900

Merged
andrzej-stencel merged 3 commits into
open-telemetry:mainfrom
isaacaflores2:es-exporter-remove-host-os-type-enrichment
Mar 18, 2026
Merged

[exporter/elasticsearch] Remove host.os.type encoding in ECS mode#46900
andrzej-stencel merged 3 commits into
open-telemetry:mainfrom
isaacaflores2:es-exporter-remove-host-os-type-enrichment

Conversation

@isaacaflores2

Copy link
Copy Markdown
Contributor

Description

The host.os.type enrichment has been moved the processor/elasticapmprocessor for two reasons:

  1. The host.os.type is only needed for events that are ingested by the OTLP receiver (not APM events ingested by the receiver/elasticapmintakereciver)
  2. This type of enrichment should belong to the processor/elasticapmprocessor

Related to elastic/opentelemetry-collector-components#1069 and elastic/opentelemetry-collector-components#1087

Link to tracking issue

Fixes

Testing

  1. Updated test cases
  2. Removed unnecessary test

Documentation

Update README.md

@lahsivjar lahsivjar left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mostly LGTM! Just a concern over the change type.

Comment thread .chloggen/es-exporter-remove-host-os-type-enrichment.yaml Outdated
@andrzej-stencel andrzej-stencel merged commit f0bac73 into open-telemetry:main Mar 18, 2026
191 checks passed
@otelbot

otelbot Bot commented Mar 18, 2026

Copy link
Copy Markdown
Contributor

Thank you for your contribution @isaacaflores2! 🎉 We would like to hear from you about your experience contributing to OpenTelemetry by taking a few minutes to fill out this survey. If you are getting started contributing, you can also join the CNCF Slack channel #opentelemetry-new-contributors to ask for guidance and get help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants