kubernetes / enhancements

Enhancements tracking repo for Kubernetes
Apache License 2.0
3.43k stars 1.48k forks source link

Node memory swap support #2400

Open ehashman opened 3 years ago

ehashman commented 3 years ago

Enhancement Description

Please keep this description up to date. This will help the Enhancement Team to track the evolution of the enhancement efficiently.

/sig node

karan commented 3 years ago

@cookieisaac

ehashman commented 3 years ago

/stage alpha /milestone v1.22

jrsapi commented 3 years ago

Greetings @ehashman! Enhancement shadow checking in with a few reminders. 1.22 Enhancements Freeze starts at 23:59:59 pst on Thursday, May 13. A few items needing review for this KEP:

Thanks!

ehashman commented 3 years ago

All of this is covered in https://github.com/kubernetes/enhancements/pull/2602

jrsapi commented 3 years ago

Greetings @ehashman! Thanks for the follow-up. After reviewing the KEP and PRR everything looks on target. The enhancement is marked at risk but once merged we can move its status to tracked. A reminder that tomorrow 5/13 the Enhancement freeze starts at 23:59:59 PST.

Thanks!

ehashman commented 3 years ago

Work breakdown for 1.22

External to k8s but still need to happen:

Once the above CRI updates happen, ensure that CI environment is using latest container runtimes with updated CRI.

jrsapi commented 3 years ago

Greetings @ehashman , Enhancement shadow checking with a reminder that we are 2 weeks away from code freeze (July 8, 2021). Can you like the k/k PR/s that are needed to implement this enhancement for the 1.22 milestone?

jrsapi commented 3 years ago

Greetings @ehashman, A friendly reminder that code freeze is this Thursday, July 8th and we're tracking the following k/k PR:

Thanks!

ehashman commented 3 years ago

PR just merged. Docs placeholder is https://github.com/kubernetes/website/pull/28838

jrsapi commented 3 years ago

Awesome! Thanks for the update. Moving this to "Tracked".

ehashman commented 3 years ago

We are good to go for 1.22! Docs complete.

ehashman commented 3 years ago

/milestone v1.23

ehashman commented 3 years ago

/stage beta

kevindelgado commented 3 years ago

Hi @ehashman 1.23 Enhancements team here. Just checking in as we approach enhancements freeze on Thursday 09/09. Here's where this enhancement currently stands:

~Looks like for this one we would just need the KEP/PRR to merge (and the "tentative" label on graduation criteria removed) by enhancements freeze :)~

Good to go now!

Thanks!

kevindelgado commented 3 years ago

Hi @ehashman

Ping! As a reminder your PR (#2858) needs to merge by EOD PST tomorrow September 9th to be included in the 1.23 Release. After that time you will need to request an exception.

Lmk if you need anything, Kevin

ehashman commented 3 years ago

Hello @kevindelgado,

PRR is now approved and the "tentative" bit has been moved to GA. Awaiting final node approval.

jlbutler commented 3 years ago

Hi @ehasman :wave: 1.23 Docs lead here.

This enhancement is marked as 'Needs Docs' for the 1.23 release.

Please open a PR against the dev-1.23 branch in the k/website repo. This PR can be just a placeholder at this time and must be created before Thu November 18, 11:59 PM PDT.

Thanks!

SergeyKanzhelev commented 2 years ago

As discussed at SIG Node meeting on October 26, 2021 we are removing this enhancement from 1.23 milestone. @kubernetes/release-managers can you please remove the milestone label?

endocrimes commented 2 years ago

Swap support has now landed in containerd: https://github.com/containerd/containerd/pull/6320 - Have a branch prepped for when a release drops to include in test-infra.

ehashman commented 2 years ago

/milestone v1.24

yangjunmyfm192085 commented 2 years ago

/cc

gracenng commented 2 years ago

Hi @ehashman ! 1.24 Enhancements team here. Just checking in as we approach enhancements freeze on 18:00pm PT on Thursday Feb 3rd. This enhancements is targeting beta for 1.24.

The status of this enhancement is track as tracked , no action is required for Enhancements Freeze. Thanks!

nate-double-u commented 2 years ago

Hi @ehashman :wave: 1.24 Docs lead here.

This enhancement is marked as Needs Docs for the 1.24 release.

Please follow the steps detailed in the documentation to open a PR against the dev-1.24 branch in the k/website repo. This PR can be just a placeholder at this time and must be created before Thursday, March 31st, 2022 @ 18:00 PDT.

Also, if needed take a look at Documenting for a release to familiarize yourself with the docs requirement for the release.

Thanks!

ehashman commented 2 years ago

Work for the 1.24 release was organized on a project board: https://github.com/orgs/kubernetes/projects/69

We also have a slack channel: #sig-node-swap

I am doubtful this will make the soft node freeze on Mar. 4 (this Friday) given how much work is left to complete.

ehashman commented 2 years ago

/milestone clear per https://docs.google.com/document/d/1Ne57gvidMEWXR70OxxnRkYquAoMpt56o75oZtg-OeBg/edit#bookmark=id.w24ewvhebv4u

gracenng commented 2 years ago

Removing this enhancements from the release

k8s-triage-robot commented 2 years ago

The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs.

This bot triages issues and PRs according to the following rules:

You can:

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

pacoxu commented 2 years ago

/remove-lifecycle stale

marosset commented 2 years ago

/milestone v1.26 /label lead-opted-in (I'm doing this on behalf of @ruiwen-zhao / SIG-node)

rhockenbury commented 2 years ago

Hello @ehashman πŸ‘‹, 1.26 Enhancements team here.

Just checking in as we approach enhancements freeze on 18:00 PDT on Thursday 6th October 2022.

This enhancement is targeting for stage alpha for 1.26 (from this comment, we'll track it as a major change for alpha, rather than beta graduation)

Here's where this enhancement currently stands:

For this KEP, we would just need to update the following:

The status of this enhancement is marked as at risk. Please keep the issue description up-to-date with appropriate stages as well. Thank you!

rhockenbury commented 2 years ago

/stage alpha /label tracked/yes

derekwaynecarr commented 2 years ago

/remove-label lead-opted-in

no volunteer has come forward yet this release to push this work further, it will remain unchanged in 1.26.

SergeyKanzhelev commented 2 years ago

/remove-label lead-opted-in

no volunteer has come forward yet this release to push this work further, it will remain unchanged in 1.26.

@derekwaynecarr I plan to move this forward, not to beta, but to more clarity. See https://github.com/kubernetes/enhancements/pull/3571.

rhockenbury commented 2 years ago

/label lead-opted-in

rhockenbury commented 2 years ago

It looks like we would still need to update the KEP readme to use the latest template to meet all requirements for enhancements freeze.

rhockenbury commented 2 years ago

Hello πŸ‘‹, 1.26 Enhancements Lead here.

Unfortunately, this enhancement did not meet requirements for enhancements freeze.

If you still wish to progress this enhancement in v1.26, please file an exception request. Thanks!

/milestone clear /label tracked/no /remove-label tracked/yes /remove-label lead-opted-in

SergeyKanzhelev commented 2 years ago

/assign

@rhockenbury I changed the formatting of TestPlan and added some links to tests we introduced in Alpha stage. Can this be returned back to milestone without the exception process since the content was not modified?

SergeyKanzhelev commented 2 years ago

/unassign @ehashman

(@ehashman please feel free to re-assign back to yourself)

rhockenbury commented 2 years ago

Alright, thanks for making those changes. I have it marked as tracked now but can you please also update the KEP yaml to set stage: alpha? - https://github.com/kubernetes/enhancements/blob/master/keps/sig-node/2400-node-swap/kep.yaml#L21

/milestone v1.26 /label tracked/yes /label lead-opted-in /remove-label tracked/no

ehashman commented 2 years ago

@SergeyKanzhelev I am not planning on working on this at this time, please feel free to continue driving it forward and let me know if you need any feedback/reviews.

Atharva-Shinde commented 2 years ago

Hey @ehashman @SergeyKanzhelev πŸ‘‹,

Checking in as we approach 1.26 code freeze at 17:00 PDT on Tuesday 8th November 2022.

Please ensure the following items are completed:

As always, we are here to help should questions come up. Thanks :)

cathchu commented 2 years ago

Hello @SergeyKanzhelev πŸ‘‹ 1.26 Release Docs shadow here!

This enhancement is marked as β€˜Needs Docs’ for 1.26 release. Please follow the steps detailed in the documentation to open a PR against dev-1.26 branch in the k/website repo. This PR can be just a placeholder at this time, and must be created by November 9. Also, take a look at Documenting for a release to familiarize yourself with the docs requirement for the release.

Thank you!

Atharva-Shinde commented 1 year ago

Hey @ehashman @SergeyKanzhelev πŸ‘‹, just a quick check-in again before 1.26 code freeze at 17:00 PDT Tuesday 8th November 2022 i.e tomorrow. Looks like we would at least need to get the code PR/s: https://github.com/kubernetes/kubernetes/issues/105019 https://github.com/kubernetes/kubernetes/issues/105020 https://github.com/kubernetes/kubernetes/issues/105021 https://github.com/kubernetes/kubernetes/issues/105022 https://github.com/kubernetes/kubernetes/issues/105023 https://github.com/kubernetes/kubernetes/issues/105025 https://github.com/kubernetes/kubernetes/issues/105978 (any other PRs?) merged before the code-freeze :)

rhockenbury commented 1 year ago

Hello πŸ‘‹, 1.26 Enhancements Lead here.

Unfortunately, this enhancement did not meet requirements for code freeze.

If you still wish to progress this enhancement in v1.26, please file an exception request. Thanks!

/milestone clear /label tracked/no /remove-label tracked/yes /remove-label lead-opted-in

fabiand commented 1 year ago

Hey. We've valiated that swap works nice with KubeVirt workloads. What kind of data is needed in order to lift this feature to beta?

drvnoo commented 1 year ago

I think this is the most important missing feature in K8s currently. Thanks everyone for all the amazing work being done here. This is a game changer!

dchen1107 commented 1 year ago

/label lead-opted-in /milestone v1.27

k8s-ci-robot commented 1 year ago

@dchen1107: Can not set label lead-opted-in: Must be member in one of these teams: [release-team-enhancements release-team-leads sig-api-machinery-leads sig-apps-leads sig-architecture-leads sig-auth-leads sig-autoscaling-leads sig-cli-leads sig-cloud-provider-leads sig-cluster-lifecycle-leads sig-contributor-experience-leads sig-docs-leads sig-instrumentation-leads sig-k8s-infra-leads sig-multicluster-leads sig-network-leads sig-node-leads sig-release-leads sig-scalability-leads sig-scheduling-leads sig-security-leads sig-storage-leads sig-testing-leads sig-windows-leads]

In response to [this](https://github.com/kubernetes/enhancements/issues/2400#issuecomment-1410962086): >/label lead-opted-in >/milestone v1.27 > Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
logicalhan commented 1 year ago

/label lead-opted-in

SergeyKanzhelev commented 1 year ago

@marosset this KEP is not moving to the different stage, but we will be working on it for "Alpha2" milestone. Is there anything needed for this KEP to be tracked for 1.27?