Skip to content

[exporter/elasticsearch] Profiling: Fix event attributes#40967

Merged
andrzej-stencel merged 5 commits into
open-telemetry:mainfrom
rockdaboot:esex-fix-event-attributes
Jul 7, 2025
Merged

[exporter/elasticsearch] Profiling: Fix event attributes#40967
andrzej-stencel merged 5 commits into
open-telemetry:mainfrom
rockdaboot:esex-fix-event-attributes

Conversation

@rockdaboot
Copy link
Copy Markdown
Contributor

@rockdaboot rockdaboot commented Jun 27, 2025

Description

The HostID, ContainerID, K8SPodName and K8sNamespaceName attributes are no longer stored at the Sample level, instead they are stored at the Resource level.

Also added ServiceName as a Sample level attribute.

See also: open-telemetry/opentelemetry-ebpf-profiler#562

@rockdaboot rockdaboot force-pushed the esex-fix-event-attributes branch from 0409138 to 3965594 Compare June 27, 2025 15:51
@rockdaboot rockdaboot marked this pull request as ready for review June 27, 2025 15:51
@rockdaboot rockdaboot requested a review from a team as a code owner June 27, 2025 15:51
@rockdaboot rockdaboot requested a review from mx-psi June 27, 2025 15:51
Comment thread .chloggen/elasticsearchexporter-fix-event-attributes.yaml Outdated
Copy link
Copy Markdown
Member

@andrzej-stencel andrzej-stencel left a comment

Choose a reason for hiding this comment

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

The service name addition should ideally be done in a separate pull request. Please take it into account for future PRs.

@andrzej-stencel andrzej-stencel merged commit 3ca7e72 into open-telemetry:main Jul 7, 2025
177 checks passed
@github-actions github-actions Bot added this to the next release milestone Jul 7, 2025
Dylan-M pushed a commit to Dylan-M/opentelemetry-collector-contrib that referenced this pull request Aug 5, 2025
…try#40967)

#### Description
The HostID, ContainerID, K8SPodName and K8sNamespaceName attributes are
no longer stored at the `Sample` level, instead they are stored at the
`Resource` level.

Also added ServiceName as a `Sample` level attribute.

See also:
open-telemetry/opentelemetry-ebpf-profiler#562
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