Skip to content

Conversation

@odubajDT
Copy link
Contributor

@odubajDT odubajDT commented Dec 3, 2025

Description

Introduction of new observability metric:

otelcol.k8s.pod.association

with status, otelcol.signal and pod_identifier attributes

Link to tracking issue

Related to #44587

@odubajDT odubajDT changed the title [processor/k8sattributes] introduce new selfmon metrics [processor/k8sattributes] introduce new observability metrics Dec 3, 2025
@odubajDT odubajDT marked this pull request as ready for review December 3, 2025 06:28
@github-actions github-actions bot added the processor/k8sattributes k8s Attributes processor label Dec 3, 2025
@odubajDT odubajDT force-pushed the k8sattributesprocessor-stable-observability branch from a96f868 to 1fc115d Compare December 9, 2025 06:12
@odubajDT odubajDT changed the title [processor/k8sattributes] introduce new observability metrics [processor/k8sattributes] introduce new observability metric Dec 9, 2025
@odubajDT odubajDT changed the title [processor/k8sattributes] introduce new observability metric [processor/k8sattributes] introduce new observability metric otelsvc_k8s_pod_association Dec 9, 2025
@odubajDT odubajDT force-pushed the k8sattributesprocessor-stable-observability branch from 1fc115d to b8c5481 Compare December 11, 2025 05:42
@odubajDT odubajDT changed the title [processor/k8sattributes] introduce new observability metric otelsvc_k8s_pod_association [processor/k8sattributes] introduce new observability metric otelcol.k8s.pod.association Dec 11, 2025
@github-actions
Copy link
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions bot added the Stale label Dec 26, 2025
@odubajDT odubajDT removed the Stale label Jan 5, 2026
@ChrsMark
Copy link
Member

I would be fine with adding the new metric following the suggested dots approach. However I think we should not close #44587 until we fixed the components' internal metrics according to open-telemetry/opentelemetry-collector#12909. WDYT?

@ChrsMark
Copy link
Member

We briefly discussed this specific case during the Collector Stability SIG. We can wait to have open-telemetry/opentelemetry-collector#14273 merged and then we can try to migrate the internal metrics from _ to . through a component specific feature gates pair (following the pattern from open-telemetry/opentelemetry-collector#14273).

@mx-psi
Copy link
Member

mx-psi commented Jan 26, 2026

open-telemetry/opentelemetry-collector/pull/14273 is on the merge queue :)

Copy link
Member

@ChrsMark ChrsMark left a comment

Choose a reason for hiding this comment

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

The addition LGTM (needs rebase).

But let's also ensure we do the rename (_ to using .) as part of the related issue: #44718 (comment)

Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
Signed-off-by: odubajDT <ondrej.dubaj@dynatrace.com>
@odubajDT odubajDT force-pushed the k8sattributesprocessor-stable-observability branch from d9ed007 to 794725b Compare January 27, 2026 14:43
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