Skip to content

[k8s] Promote a selection of container and k8s memory metrics to release_candidate#3883

Open
ChrsMark wants to merge 1 commit into
open-telemetry:mainfrom
ChrsMark:rc_mem_metrics
Open

[k8s] Promote a selection of container and k8s memory metrics to release_candidate#3883
ChrsMark wants to merge 1 commit into
open-telemetry:mainfrom
ChrsMark:rc_mem_metrics

Conversation

@ChrsMark

@ChrsMark ChrsMark commented Jul 13, 2026

Copy link
Copy Markdown
Member

Fixes #3882

After the evaluation described at #3882 (comment) I suggest we promote these metrics to release_candidate.

These have been in use from the https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/receiver/kubeletstatsreceiver/documentation.md for some time now without issues.

Changes

Please provide a brief description of the changes here.

Important

Pull request acceptance is subject to the triage process as described in Issue and PR Triage Management.
PRs that do not follow the guidance above may be automatically rejected and closed.

Merge requirement checklist

  • CONTRIBUTING.md guidelines followed.
  • Links to prototypes or existing instrumentations (when adding or changing conventions)
  • Disclose AI usage, see OTel GenAI policy:
    • no AI used
    • AI-assisted
    • bulk AI-generated
  • I have the experience and knowledge necessary to understand, review, and validate all content in this PR.1

Footnotes

  1. Yes, I can answer maintainer questions about the content of this PR without using AI.

@TylerHelmuth TylerHelmuth 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.

🎉

@dashpole dashpole left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks like some of these are the wrong instrument type.

Comment thread docs/system/k8s-metrics.md Outdated
Comment thread docs/system/k8s-metrics.md Outdated
Comment thread docs/system/container-metrics.md Outdated
@github-project-automation github-project-automation Bot moved this from Untriaged to Blocked in Semantic Conventions Triage Jul 13, 2026
Copilot AI review requested due to automatic review settings July 20, 2026 07:14
@ChrsMark
ChrsMark requested a review from dashpole July 20, 2026 07:15

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR promotes a specific set of Kubernetes and container memory metrics from development to release_candidate, reflecting their maturity and established usage in existing instrumentation (per issue #3882).

Changes:

  • Promotes *.memory.{usage,available,rss,working_set} metrics for container, k8s.pod, and k8s.node to release_candidate.
  • Expands/clarifies metric notes to describe semantics and derivation from Kubelet stats API (and cAdvisor for container metrics).
  • Adds a changelog entry documenting the promotion.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
model/k8s/metrics.yaml Promotes selected k8s pod/node memory metrics to RC and clarifies *.memory.usage derivation/semantics.
model/container/metrics.yaml Promotes selected container memory metrics to RC and expands notes (including cAdvisor + Kubelet derivation).
docs/system/k8s-metrics.md Regenerates published k8s metrics docs to reflect RC status and updated notes.
docs/system/container-metrics.md Regenerates published container metrics docs to reflect RC status and updated notes.
.chloggen/rc_mem_metrics.yaml Adds release note entry for the RC promotion.

Comment thread model/container/metrics.yaml
Comment thread docs/system/container-metrics.md Outdated
…ase_candidate

Signed-off-by: ChrsMark <chrismarkou92@gmail.com>
@lmolkova lmolkova moved this from Blocked to Needs More Approval in Semantic Conventions Triage Jul 20, 2026
@opentelemetry-pr-dashboard

opentelemetry-pr-dashboard Bot commented Jul 21, 2026

Copy link
Copy Markdown

Pull request dashboard status

Status last refreshed: 2026-07-25 18:54:46 UTC.

  • Waiting on: Maintainers
  • Next step: Merge when ready.

This automated status or its linked feedback items may be incorrect. If something looks wrong, please report it with the result you expected.

@lmolkova lmolkova moved this from Needs More Approval to Ready to be Merged in Semantic Conventions Triage Jul 23, 2026
@github-project-automation github-project-automation Bot moved this from Ready to be Merged to Needs More Approval in Semantic Conventions Triage Jul 23, 2026
@lmolkova lmolkova moved this from Needs More Approval to Ready to be Merged in Semantic Conventions Triage Jul 23, 2026
@github-project-automation github-project-automation Bot moved this from Ready to be Merged to Needs More Approval in Semantic Conventions Triage Jul 24, 2026
@lmolkova lmolkova moved this from Needs More Approval to Ready to be Merged in Semantic Conventions Triage Jul 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:k8s enhancement New feature or request

Projects

Status: No status
Status: Ready to be Merged

Development

Successfully merging this pull request may close these issues.

[k8s] Promote k8s and container memory metrics to RC

7 participants