Skip to content

Add optimizing Kubernetes applications and services with NRI resource policies - #51

Open
askervin wants to merge 3 commits into
intel:mainfrom
askervin:5id-add-kubernetes-nri
Open

Add optimizing Kubernetes applications and services with NRI resource policies#51
askervin wants to merge 3 commits into
intel:mainfrom
askervin:5id-add-kubernetes-nri

Conversation

@askervin

Copy link
Copy Markdown

No description provided.

@adgubrud
adgubrud requested review from adgubrud and a lite review from CopilotAugust 14, 2026 16:57

CopilotAI 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 expands the Optimization Zone documentation to cover Kubernetes-focused tuning by introducing a new Kubernetes section centered on NRI (Node Resource Interface) resource-policy plugins, with supporting pages for policy selection and hands-on usage.

Changes:

  • Adds a Kubernetes section to the root README with navigation to new NRI resource-policy documentation.
  • Introduces Kubernetes Optimization docs and NRI resource-policy pages (Topology-aware, Balloons, choosing guides, and a Balloons quickstart).
  • Adds cross-references from existing hardware guides (NUMA, Priority Core Turbo) to the relevant Kubernetes/NRI policy docs.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 3 comments.

Show a summary per file
FileDescription
README.mdAdds top-level navigation entries for the new Kubernetes optimization documentation.
kubernetes/README.mdIntroduces the Kubernetes Optimization landing page and maps existing Optimization Zone topics to NRI policy features.
kubernetes/nri-resource-policies/README.mdAdds an NRI resource-policy overview plus install/configuration guidance.
kubernetes/nri-resource-policies/topology-aware.mdAdds a Topology-aware policy summary and links to upstream reference docs.
kubernetes/nri-resource-policies/balloons.mdAdds a Balloons policy summary and links to upstream reference docs/recipes.
kubernetes/nri-resource-policies/choosing-a-policy.mdAdds guidance for choosing between Topology-aware and Balloons.
kubernetes/nri-resource-policies/choosing-a-policy-with-kubernetes-managers.mdExplains how NRI policies relate to Kubernetes built-in managers and when to use each approach.
kubernetes/nri-resource-policies/quickstart.mdAdds a step-by-step Balloons quickstart demonstrating latency isolation with Redis and a noisy neighbor.
hardware/priority_core_turbo/README.mdAdds a Kubernetes callout pointing readers to using Balloons for PCT-related pod placement/tuning.
hardware/NUMA/README.mdAdds a Kubernetes callout pointing readers to using Topology-aware for NUMA-aligned pod placement.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment threadhardware/priority_core_turbo/README.md Outdated
Comment threadhardware/NUMA/README.md Outdated
Comment threadkubernetes/nri-resource-policies/README.md

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

Thank you for your submission! I like that this article references other existing articles and enhances them with tie-ins to Kubernetes. Most of what I found was just providing expanded forms of lesser known abbreviations. My only other concern is about the Redis performance examples in the quickstart guide. That data might have to go through PDT or be removed.

Comment threadkubernetes/nri-resource-policies/topology-aware.md Outdated
Comment threadkubernetes/nri-resource-policies/balloons.md Outdated
Comment threadkubernetes/nri-resource-policies/choosing-a-policy.md Outdated
Comment threadkubernetes/README.md Outdated
Comment threadkubernetes/nri-resource-policies/quickstart.md
@askervin
askervinforce-pushed the 5id-add-kubernetes-nri branch 2 times, most recently from 4256714 to 61edab3CompareAugust 17, 2026 11:50
Signed-off-by: Antti Kervinen <antti.kervinen@intel.com>
Signed-off-by: Antti Kervinen <antti.kervinen@intel.com>
Signed-off-by: Antti Kervinen <antti.kervinen@intel.com>
@askervin
askervinforce-pushed the 5id-add-kubernetes-nri branch from 61edab3 to 7eaa50cCompareAugust 17, 2026 11:57
@askervin
askervin requested a review from adgubrudAugust 17, 2026 11:58

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

Thank you for addressing the items in my previous round of review and correcting my own typo for PCT :)

Just a couple more items around placing this article and making sure it has an entry in the main repo's TOC

Comment threadREADME.md

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.

Can you please put the "kubernetes" directory within the "software" directory? And also please add an entry for your article table of contents in the main repo README.

Sign up for freeto 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.

3 participants

@askervin@adgubrud