-
Notifications
You must be signed in to change notification settings - Fork 14.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Docs: Windows CPU and Memory Affinity #48469
base: dev-1.32
Are you sure you want to change the base?
Conversation
👷 Deploy Preview for kubernetes-io-vnext-staging processing.
|
✅ Pull request preview available for checkingBuilt without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
/sig windows |
Hi @jsturtevant 👋 Just a reminder to take a look at Documenting for a release - PR Ready for Review to get your PR ready for review ahead of the "Ready for Review" deadline on Tuesday November 19th 2024 18:00 PST. Thank you! |
Hi @jsturtevant 👋 , A quick reminder to get your PR ready for review before the deadline, Tuesday November 19th at 18:00 PDT. For additional information, refer to: Documenting for a release - PR Ready for Review. Thank you! |
Signed-off-by: James Sturtevant <[email protected]>
54b8708
to
e2cccab
Compare
...docs/reference/command-line-tools-reference/feature-gates/windows-cpu-and-memory-affinity.md
Outdated
Show resolved
Hide resolved
...docs/reference/command-line-tools-reference/feature-gates/windows-cpu-and-memory-affinity.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/administer-cluster/cpu-management-policies.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/administer-cluster/cpu-management-policies.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/administer-cluster/cpu-management-policies.md
Outdated
Show resolved
Hide resolved
...docs/reference/command-line-tools-reference/feature-gates/windows-cpu-and-memory-affinity.md
Show resolved
Hide resolved
content/en/docs/tasks/administer-cluster/cpu-management-policies.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Mark Rossetti <[email protected]> Co-authored-by: Qiming Teng <[email protected]> Co-authored-by: Aravindh Puthiyaparambil <[email protected]>
342bba2
to
ba31359
Compare
Signed-off-by: James Sturtevant <[email protected]>
ba31359
to
bdc9bef
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM from sig-node, the docs technical content looks correct and reflecting what we did.
…es.md Co-authored-by: Qiming Teng <[email protected]>
/assign @tengqm |
/assign @marosset @aravindhp |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM other than some nits.
content/en/docs/tasks/administer-cluster/cpu-management-policies.md
Outdated
Show resolved
Hide resolved
…es.md Co-authored-by: Qiming Teng <[email protected]>
Co-authored-by: Qiming Teng <[email protected]>
Co-authored-by: Qiming Teng <[email protected]>
Technical LGTM provided above /lgtm |
LGTM label has been added. Git tree hash: 638adb68cc2ba994b76e4a31f1dea65c6ae3f970
|
@tengqm could we get your final review/approval on this one? thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm from sig-windows
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: marosset The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Description
Place holder for kubernetes/enhancements#4885
kubernetes/kubernetes#125296
kubernetes/kubernetes#124285
opencontainers/runtime-spec#1258
Issue
KEP: kubernetes/enhancements#4885