Skip to content

KEP-3953: Node Resource Hot Plug #3955

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 22 commits into
base: master
Choose a base branch
from
Open

KEP-3953: Node Resource Hot Plug #3955

wants to merge 22 commits into from
+989 −0

Conversation

Karthik-K-N
Copy link

@Karthik-K-N Karthik-K-N commented Apr 17, 2023 •
edited
Loading

  • One-line PR description: Node Resource Hot Plug
  • Other comments:

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/kep Categorizes KEP tracking issues and PRs modifying the KEP directory sig/node Categorizes an issue or PR as relevant to SIG Node. labels Apr 17, 2023
@k8s-ci-robot k8s-ci-robot added the size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. label Apr 17, 2023
@Karthik-K-N Karthik-K-N mentioned this pull request Apr 17, 2023
Open
4 tasks
@Karthik-K-N Karthik-K-N changed the title Dynamic node resize KEP-3953: Dynamic node resize Apr 17, 2023
@bart0sh
Copy link
Contributor

bart0sh commented Apr 17, 2023

/assign @mrunalp @SergeyKanzhelev @klueska

@kad
Copy link
Member

kad commented Apr 28, 2023

/cc

@k8s-ci-robot k8s-ci-robot requested a review from kad April 28, 2023 16:15
@ffromani
Copy link
Contributor

/cc

@k8s-ci-robot k8s-ci-robot requested a review from ffromani May 18, 2023 07:29
@k8s-ci-robot k8s-ci-robot added cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. and removed cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. labels May 22, 2023
pacoxu
pacoxu reviewed May 23, 2023
View reviewed changes
pacoxu
pacoxu reviewed May 23, 2023
View reviewed changes
@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. and removed cncf-cla: no Indicates the PR's author has not signed the CNCF CLA. labels May 23, 2023
@fmuyassarov
Copy link
Member

/cc

@k8s-ci-robot k8s-ci-robot requested a review from fmuyassarov May 25, 2023 11:59
@44past4 44past4 mentioned this pull request Sep 22, 2023
Closed
sftim
sftim reviewed Sep 23, 2023
View reviewed changes
@Karthik-K-N
Copy link
Author

/remove-lifecycle stale

@k8s-ci-robot k8s-ci-robot removed the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Sep 15, 2025
Karthik-K-N and others added 20 commits September 25, 2025 09:31
@Karthik-K-N
KEP for dynamic node resize
4976729
@kishen-v @Karthik-K-N
Fix KEP doc for node-resize.
ba1e790
@Karthik-K-N
Update to emphasis on scale up of resoures
9e8c4f6
@Karthik-K-N
Rename the KEP to match to the updated scope
6ed4376
@Karthik-K-N
Address reveiw comments
093ae95
@Karthik-K-N
Address review comments
3f7f305
@Karthik-K-N
Address review comments
bdc2e89
@Karthik-K-N
Update user-stories
60701e1
@Karthik-K-N
Address review comments
81544b6
@Karthik-K-N @kishen-v
Address review comments
9523e83 
Co-authored-by: kishen-v <kishen.viswanathan@ibm.com>
@Karthik-K-N @kishen-v
Address review comments
54cdc8a 
Co-authored-by: kishen-v <kishen.viswanathan@ibm.com>
@Karthik-K-N @kishen-v
Address review comments
7c96122 
Co-authored-by: kishen-v <kishen.viswanathan@ibm.com>
@Karthik-K-N @kishen-v
Update HotUnplug Scenario
e559ac0 
Co-authored-by: kishen-v <kishen.viswanathan@ibm.com>
@Karthik-K-N
Address review comments
c3877f7 
Co-authored-by: kishen-v <kishen.viswanathan@ibm.com>
@Karthik-K-N
Address review comments
443724d
@Karthik-K-N
Address review comments
5eade5c
@Karthik-K-N
Add CA compatability section
b1d8a41
@Karthik-K-N
Update OOMScoreAdj formula
92f3d8f
@Karthik-K-N
Address reveiw comments
d95a3be 
Co-authored-by: kishen-v <kishen.viswanathan@ibm.com>
@Karthik-K-N
Update for 1.35
a65903d
@Karthik-K-N Karthik-K-N mentioned this pull request Sep 30, 2025
Open
kannon92
kannon92 reviewed Oct 3, 2025
View reviewed changes
kannon92
kannon92 reviewed Oct 3, 2025
View reviewed changes
kannon92
kannon92 reviewed Oct 3, 2025
View reviewed changes
Copy link
Contributor

@kannon92 kannon92 left a comment

Choose a reason for hiding this comment

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

PRR shadow:

The PRR looks good for alpha.
Thank you for answering more than you needed.

We still need sig approval but the PRR is looking good.

@Karthik-K-N
Copy link
Author

PRR shadow:

The PRR looks good for alpha. Thank you for answering more than you needed.

We still need sig approval but the PRR is looking good.

Thank you for the review, I have updated the KEP address your review comments and Yes looking for SIG-Node review as well.

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

@adrianmoisey adrianmoisey adrianmoisey left review comments

@pacoxu pacoxu pacoxu left review comments

@kannon92 kannon92 kannon92 left review comments

@thockin thockin thockin left review comments

@deads2k deads2k deads2k left review comments

@haircommander haircommander haircommander requested changes

@SergeyKanzhelev SergeyKanzhelev SergeyKanzhelev requested changes

@sanposhiho sanposhiho sanposhiho left review comments

@dom4ha dom4ha dom4ha left review comments

@dchen1107 dchen1107 Awaiting requested review from dchen1107

@derekwaynecarr derekwaynecarr Awaiting requested review from derekwaynecarr

@kad kad Awaiting requested review from kad

@fmuyassarov fmuyassarov Awaiting requested review from fmuyassarov

@wangzhen127 wangzhen127 Awaiting requested review from wangzhen127

@johnbelamaric johnbelamaric Awaiting requested review from johnbelamaric

+16 more reviewers

@raywainman raywainman raywainman left review comments

@haosdent haosdent haosdent left review comments

@marquiz marquiz marquiz left review comments

@pbetkier pbetkier pbetkier left review comments

@voelzmo voelzmo voelzmo left review comments

@ndixita ndixita ndixita left review comments

@yujuhong yujuhong yujuhong left review comments

@mkumatag mkumatag mkumatag left review comments

@bart0sh bart0sh bart0sh left review comments

@swatisehgal swatisehgal swatisehgal left review comments

@sftim sftim sftim left review comments

@ffromani ffromani ffromani left review comments

@HirazawaUi HirazawaUi HirazawaUi left review comments

@maxcao13 maxcao13 maxcao13 left review comments

@iholder101 iholder101 iholder101 left review comments

@esotsal esotsal esotsal left review comments

Reviewers whose approvals may not affect merge requirements
Assignees

@mrunalp mrunalp

@klueska klueska

@SergeyKanzhelev SergeyKanzhelev

Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/kep Categorizes KEP tracking issues and PRs modifying the KEP directory sig/node Categorizes an issue or PR as relevant to SIG Node. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.