Skip to content

[exporter/signalfx] Add an option to strip k8s.<resource>.label. prefix from dimension property updates.#47494

Merged
dmitryax merged 1 commit into
open-telemetry:mainfrom
dmitryax:signalfx-strip-k8s-label-prefix
Apr 11, 2026
Merged

[exporter/signalfx] Add an option to strip k8s.<resource>.label. prefix from dimension property updates.#47494
dmitryax merged 1 commit into
open-telemetry:mainfrom
dmitryax:signalfx-strip-k8s-label-prefix

Conversation

@dmitryax
Copy link
Copy Markdown
Member

@dmitryax dmitryax commented Apr 9, 2026

Add dimension_client::strip_k8s_label_prefix option to strip k8s.<resource>.label. prefix from dimension property updates.

Once #47493 is merged, the k8s cluster receiver will emit Kubernetes resource labels in entity events with the k8s.<resource>.label. prefix per OTel semantic conventions (e.g. k8s.pod.label.app). When strip_k8s_label_prefix: true (the default), the SignalFx exporter strips this prefix when forwarding labels as dimension properties, preserving the existing SignalFx behavior (e.g. app).

@dmitryax dmitryax requested review from a team, ChrsMark, TylerHelmuth and crobert-1 as code owners April 9, 2026 03:58
@github-actions github-actions Bot requested a review from povilasv April 9, 2026 03:58
@dmitryax dmitryax force-pushed the signalfx-strip-k8s-label-prefix branch 3 times, most recently from a2e26eb to b6a5305 Compare April 9, 2026 16:11
Comment thread .chloggen/signalfxexporter-strip-k8s-label-prefix.yaml
@dmitryax dmitryax force-pushed the signalfx-strip-k8s-label-prefix branch from b6a5305 to 2eb8ba9 Compare April 10, 2026 01:12
Comment thread exporter/signalfxexporter/config.go Outdated
@dmitryax dmitryax force-pushed the signalfx-strip-k8s-label-prefix branch from 2eb8ba9 to aef0667 Compare April 10, 2026 22:36
@crobert-1
Copy link
Copy Markdown
Member

Looks good once CI is fixed

Config schemas are out of date, please run "make generate-schemas" and commit the changes in this PR.

Add a configurable `dimension_client::strip_k8s_label_prefix` option
(default: true) that strips the `k8s.<resource>.label.` prefix from
Kubernetes resource label keys before sending them as SignalFx dimension
properties.

This maintains backwards compatibility with existing SignalFx consumers
while supporting the new OTel semconv label key format emitted by
receiver/k8s_cluster.

Assisted-by: Claude Opus 4.6
@dmitryax dmitryax force-pushed the signalfx-strip-k8s-label-prefix branch from aef0667 to a3eaac0 Compare April 10, 2026 23:01
@dmitryax dmitryax merged commit 60c4bbb into open-telemetry:main Apr 11, 2026
191 checks passed
AndrewCharlesHay pushed a commit to AndrewCharlesHay/opentelemetry-collector-contrib that referenced this pull request Apr 23, 2026
…efix from dimension property updates. (open-telemetry#47494)

Add `dimension_client::strip_k8s_label_prefix` option to strip
`k8s.<resource>.label.` prefix from dimension property updates.

Once
open-telemetry#47493
is merged, the k8s cluster receiver will emit Kubernetes resource labels
in entity events with the `k8s.<resource>.label.` prefix per OTel
semantic conventions (e.g. `k8s.pod.label.app`). When
`strip_k8s_label_prefix: true` (the default), the SignalFx exporter
strips this prefix when forwarding labels as dimension properties,
preserving the existing SignalFx behavior (e.g. `app`).
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