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
win: fix cpu count to calculate cpu_maximum #114231
Conversation
Please note that we're already in Test Freeze for the Fast forwards are scheduled to happen every 6 hours, whereas the most recent run was: Thu Dec 1 09:40:53 UTC 2022. |
Welcome @mweibel! |
Hi @mweibel. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/ok-to-test |
/retest log looks like it didn't fail because of the change, I'm not 100% sure but I hope it's ok if I just run a retest. |
/retest |
/triage accepted |
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
/retest-required |
👍 thanks. Tests passed! |
lgtm |
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
@mweibel - Can you update the release notes to something like `Fixing issue on Windows when calculating cpu limits on nodes with more than 64 logical processors' ?
@marosset thanks! I updated the PR to include the release note. |
/uncc |
folks, what can I do to get this approved? |
looks like it is on the sig-node triage board marked ready for approval. Will need someone from the kubelet approvers file to give the approval. /assign @derekwaynecarr @mrunalp |
The PR just touches windows related code and nothing in kubelet and has a bunch of LGTM(s) from sig-windows folks. Happy to land this with my sig-node reviewer hat on. /approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dims, marosset, mweibel The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@mweibel: The following tests failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
/retest |
Suggested changelog tweak: Fixed an issue when calculating CPU limits on Windows nodes with more than 64 logical processors |
@sftim thanks - applied it on the first comment. I assume when generating the changelog it'll take this into account? |
That's what I understand. Thanks for the tweak. |
/sig windows
/hold
What type of PR is this?
/kind bug
What this PR does / why we need it:
Take all processor groups into account when calculating cpu maximum for windows.
Signed-off-by: Michael Weibel michael@helio.exchange
Which issue(s) this PR fixes:
Fixes #114211
Special notes for your reviewer:
I'd appreciate help with writing tests for this.
Does this PR introduce a user-facing change?
Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.: