Skip to content

[exporter/elasticsearch] Update ECS mode encoder to add conversions for telemetry.sdk.language and telemetry.sdk.version#46690

Merged
andrzej-stencel merged 4 commits into
open-telemetry:mainfrom
isaacaflores2:es-exporter-ecs-mode-map-service-lang
Mar 10, 2026
Merged

[exporter/elasticsearch] Update ECS mode encoder to add conversions for telemetry.sdk.language and telemetry.sdk.version#46690
andrzej-stencel merged 4 commits into
open-telemetry:mainfrom
isaacaflores2:es-exporter-ecs-mode-map-service-lang

Conversation

@isaacaflores2

Copy link
Copy Markdown
Contributor

Description

This PR adds semconv conversion to Conversions map semconv attributes telemetry.sdk.language/telemetry.sdk.version to service.language.name/service.language.version'.

This is needed since the connector/elasticapm emit aggregated metrics with the above semconv attributes and without these conversions these fields are dropped from the metrics. This change will allow ECS aggregated metrics to match existing amp-data/apm-aggregation behavior

Link to tracking issue

Fixes

Testing

Updated unit test cases

Documentation

Updated README.md

@isaacaflores2 isaacaflores2 requested a review from a team as a code owner March 5, 2026 17:28
Comment thread .chloggen/es-exporter-ecs-mode-map-service-lang.yaml Outdated
Co-authored-by: Carson Ip <carsonip@users.noreply.github.com>
@andrzej-stencel andrzej-stencel merged commit b5e36c7 into open-telemetry:main Mar 10, 2026
191 checks passed
@otelbot

otelbot Bot commented Mar 10, 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.

6 participants