Skip to content

docs: surface the cgroup v2 limitation earlier in runtime metrics#24914

Open
Rohan5commit wants to merge 1 commit intodocker:mainfrom
Rohan5commit:docs/runmetrics-cgroup-v2-guidance-20260428
Open

docs: surface the cgroup v2 limitation earlier in runtime metrics#24914
Rohan5commit wants to merge 1 commit intodocker:mainfrom
Rohan5commit:docs/runmetrics-cgroup-v2-guidance-20260428

Conversation

@Rohan5commit
Copy link
Copy Markdown
Contributor

Description

  • surface the cgroup v2 limitation earlier in the runtime metrics page
  • clarify that the detailed metrics section currently documents the cgroup v1 file layout
  • point cgroup v2 readers to the container-specific controller files they should inspect

Related issues or tickets

Guideline alignment

  • followed docs/CONTRIBUTING.md
  • kept the change to one documentation file and one focused issue

Validation

  • not run (docker buildx bake validate) because this is a docs-only clarification

Reviews

  • Technical review
  • Editorial review
  • Product review

@Rohan5commit Rohan5commit requested a review from dvdksn as a code owner April 28, 2026 02:34
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 28, 2026

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit 7b7098c
🔍 Latest deploy log https://app.netlify.com/projects/docsdocker/deploys/69f01cbcda445000083c9e3f
😎 Deploy Preview https://deploy-preview-24914--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@dvdksn
Copy link
Copy Markdown
Contributor

dvdksn commented Apr 28, 2026

/review

Copy link
Copy Markdown
Contributor

@docker-agent docker-agent Bot left a comment

Choose a reason for hiding this comment

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

Assessment: 🟢 APPROVE

The change is focused and accurate. It surfaces the cgroup v2 limitation earlier on the page by adding a callout near the distribution list, renames the detailed metrics heading to clarify it covers cgroup v1, and rewrites the accompanying NOTE to be more actionable. Technical claims (controller file globs memory.*, cpu.*, io.*) are correct, no Vale violations were found, and line lengths are within limits.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants