enhancements icon indicating copy to clipboard operation
enhancements copied to clipboard

Minimizing iptables-restore input size

Open danwinship opened this issue 3 years ago • 16 comments

Enhancement Description

  • One-line enhancement description (can be used as a release note): Minimizing iptables-restore input size
  • Kubernetes Enhancement Proposal: https://github.com/kubernetes/enhancements/pull/3454
  • Discussion Link: https://github.com/kubernetes/kubernetes/pull/110268#pullrequestreview-1057848432
  • Primary contact (assignee): @danwinship
  • Responsible SIGs: sig-network
  • Enhancement target (which target equals to which milestone):
    • Alpha release target (x.y): 1.26
    • Beta release target (x.y): 1.27
    • Stable release target (x.y):
  • [x] Alpha
    • [X] KEP (k/enhancements) update PR(s): #3454
    • [x] Code (k/k) update PR(s): https://github.com/kubernetes/kubernetes/pull/110268, https://github.com/kubernetes/kubernetes/pull/114181
    • [x] Testing update PR(s): https://github.com/kubernetes/test-infra/pull/27927, https://github.com/kubernetes/perf-tests/pull/2182, https://github.com/kubernetes/kubernetes/pull/114229
    • [x] Docs (k/website) update PR(s): https://github.com/kubernetes/website/pull/37649, https://github.com/kubernetes/website/pull/38435
  • [ ] Beta
    • [x] KEP (k/enhancements) update PR(s): https://github.com/kubernetes/enhancements/pull/3577
    • [ ] Code (k/k) update PR(s): https://github.com/kubernetes/kubernetes/pull/115138
    • [ ] Docs (k/website) update(s):

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

danwinship avatar Aug 02 '22 15:08 danwinship

/sig network

danwinship avatar Aug 02 '22 15:08 danwinship

/label lead-opted-in

aojea avatar Sep 26 '22 20:09 aojea

sudo @thockin /label lead-opted-in

aojea avatar Sep 26 '22 20:09 aojea

@aojea: Can not set label lead-opted-in: Must be member in one of these teams: [release-team-enhancements 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:

/label lead-opted-in

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.

k8s-ci-robot avatar Sep 26 '22 20:09 k8s-ci-robot

/label lead-opted-in /milestone v1.26 /label tracked/yes /stage alpha /kind kep

rhockenbury avatar Sep 27 '22 15:09 rhockenbury

Hello @danwinship 👋, 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 (correct me, if otherwise)

Here's where this enhancement currently stands:

  • [ ] KEP readme using the latest template has been merged into the k/enhancements repo.
  • [ ] KEP status is marked as implementable for latest-milestone: 1.26
  • [ ] KEP readme has a updated detailed test plan section filled out
  • [ ] KEP readme has up to date graduation criteria
  • [ ] KEP has a production readiness review that has been completed and merged into k/enhancements.

For this KEP, we would just need to merge https://github.com/kubernetes/enhancements/pull/3454, which will take care of above requirements.

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!

ruheenaansari34 avatar Sep 28 '22 15:09 ruheenaansari34

Candidate for Alpha in 1.26

thockin avatar Sep 29 '22 17:09 thockin

@ruheenaansari34 #3454 is merged. Can you confirm that we are OK for Alpha?

danwinship avatar Oct 04 '22 14:10 danwinship

Here’s where this enhancement currently stands:

  • [X] KEP readme using the latest template has been merged into the k/enhancements repo.
  • [X] KEP status is marked as implementable for latest-milestone: 1.26
  • [X] KEP readme has a updated detailed test plan section filled out
  • [X] KEP readme has up-to-date graduation criteria.
  • [X] KEP has a production readiness review that has been completed and merged into k/enhancements.

With all the KEP requirements in place and merged into k/enhancements, this enhancement is all good for the upcoming enhancements freeze. 🚀

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

ruheenaansari34 avatar Oct 04 '22 15:10 ruheenaansari34

Hi @danwinship 👋,

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

Please ensure the following items are completed:

  • [ ] All PRs to the Kubernetes repo that are related to your enhancement are linked in the above issue description (for tracking purposes).
  • [ ] All PRs are fully merged by the code freeze deadline.

For this enhancement, open PR. Please plan to get PRs out for all k/k code so it can be merged up by code freeze. If you do have k/k PRs open, please link them to this issue. Let me know if there aren't any further PRs that need to be created or merged for this enhancements, so that I can mark it as tracked for code freeze.

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

parul5sahoo avatar Nov 01 '22 14:11 parul5sahoo

Hi there @danwinship 👋 1.26 Docs Shadow here.

This enhancement is marked as ‘Needs Docs’ for 1.26 release. I see you've opened https://github.com/kubernetes/website/pull/37649. I'd like to confirm that all Docs related changes will be included in this PR.

If not, 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!

cathchu avatar Nov 02 '22 03:11 cathchu

Hey @danwinship ,

As the Code freeze is just a day away, just wanted to confirm that there are no open PRs in the K/K repo or any repo in general for this enhancement other than the ones outlined in the issues description? So that the enhancement can be marked tracked.

parul5sahoo avatar Nov 07 '22 05:11 parul5sahoo

@parul5sahoo There is nothing more to be merged to k/k, but the KEP graduation criteria said we'd enable the feature in some of the periodic scale tests, which has not yet happened. But I guess that would be due by next week's Test Freeze, not tomorrow's Code Freeze, right?

danwinship avatar Nov 07 '22 14:11 danwinship

Here another relevant doc-PR https://github.com/kubernetes/website/pull/36675 related this KEP

krol3 avatar Nov 28 '22 00:11 krol3

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

marosset avatar Dec 14 '22 19:12 marosset

This is a fairly high-risk change, which is why we did a KEP. Beta may be pointless?

thockin avatar Jan 05 '23 17:01 thockin

@danwinship still shooting to advance in 27?

thockin avatar Jan 28 '23 00:01 thockin

yes. updated the checklist; all that's left for 1.27 is docs

danwinship avatar Feb 01 '23 15:02 danwinship

/label lead-opted-in

thockin avatar Feb 02 '23 23:02 thockin

Hello {enhancement owner} 👋, Enhancements team here.

Just checking in as we approach enhancements freeze on 18:00 PDT Thursday 9th February 2023.

This enhancement is targeting for stage beta for v1.27 (correct me, if otherwise)

Here’s where this enhancement currently stands:

  • [X] KEP readme using the latest template has been merged into the k/enhancements repo.
  • [X] KEP status is marked as implementable for latest-milestone: v1.27
  • [X] KEP readme has a updated detailed test plan section filled out
  • [X] KEP readme has up to date graduation criteria.
  • [X] KEP has a production readiness review that has been completed and merged into k/enhancements.

With all the KEP requirements in place and merged into k/enhancements, this enhancement is all good for the upcoming enhancements freeze. 🚀

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

/stage beta /label tracked/yes /remove-label tracked/no

marosset avatar Feb 03 '23 21:02 marosset

Hi @danwinship :wave:,

Checking in as we approach 1.27 code freeze at 17:00 PDT on Tuesday 14th March 2023.

Please ensure the following items are completed:

  • [x] All PRs to the Kubernetes repo that are related to your enhancement are linked in the above issue description (for tracking purposes).
  • [x] All PRs are fully merged by the code freeze deadline.

Please let me know if there are any other PRs in k/k I should be tracking for this KEP.

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

marosset avatar Mar 08 '23 20:03 marosset

Hi @danwinship 👋 , I’m reaching out from the 1.27 Release Docs team. This enhancement is marked as ‘Needs Docs’ for the 1.27 release. Please follow the steps detailed in the documentation to open a PR against dev-1.27 branch in the k/website repo. This PR can be just a placeholder at this time, and must be created by March 16. For more information, please take a look at Documenting for a release to familiarize yourself with the documentation requirements for the release. Please feel free to reach out with any questions. Thanks!

taniaduggal avatar Mar 10 '23 14:03 taniaduggal

This enhancement is marked as ‘Needs Docs’ for the 1.27 release.

There is already a docs PR filed (https://github.com/kubernetes/website/pull/39188). It was waiting for dev-1.27 to be updated for a while but it's rebased now and can be merged once it's approved.

danwinship avatar Mar 13 '23 12:03 danwinship

NEXT: Aiming for GA in 1.28

thockin avatar Apr 27 '23 16:04 thockin

Hello @danwinship 👋, Enhancements team here.

Just checking in as we approach enhancements freeze on Thursday, 16th June 2023.

Looks like this enhancement is targeting for stage stable for v1.28 (correct me if otherwise)

Here's where this enhancement currently stands:

  • [X] KEP readme using the latest template has been merged into the k/enhancements repo.
  • [X] KEP status is marked as implementable for latest-milestone:v1.28
  • [X] KEP readme has a updated detailed test plan section filled out
  • [X] KEP readme has up to date graduation criteria
  • [X] KEP has a production readiness review that has been completed and merged into k/enhancements.

With all the KEP requirements in place and merged into k/enhancements, this enhancement is all good for the upcoming enhancements freeze. 🚀

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

Atharva-Shinde avatar Jun 09 '23 20:06 Atharva-Shinde

/stage stable

Atharva-Shinde avatar Jun 09 '23 20:06 Atharva-Shinde

Hey again @danwinship :wave:

Just checking in as we approach Code freeze at 01:00 UTC Friday, 19th July 2023 .

Here’s the enhancement’s state for the upcoming code freeze:

  • [X] All the PRs that are related to your enhancement are linked in the above issue description (for tracking purposes). This includes code, tests, and documentation related PR/s.
  • [X] All code related PR/s are merged or are in merge-ready state ( i.e they have approved and lgtm labels applied) by the code freeze deadline. This includes any tests related PR/s too.

These are the code related PR/s that I found on this KEP issue:

  • https://github.com/kubernetes/kubernetes/pull/117787
  • https://github.com/kubernetes/kubernetes/pull/117872

Please keep the issue description up-to-date with all the PR/s that are associated with this KEP and let me know if there are other PR/s in k/k we should be tracking for this KEP.

As always, we are here to help if any questions come up. Thanks!

Atharva-Shinde avatar Jul 15 '23 18:07 Atharva-Shinde

Hey @danwinship 👋 Enhancements Lead here, With https://github.com/kubernetes/kubernetes/pull/117787 and https://github.com/kubernetes/kubernetes/pull/117872 merged as per the issue description, this enhancement is now tracked for v1.28 Code Freeze. Thanks!

Atharva-Shinde avatar Jul 19 '23 04:07 Atharva-Shinde

Hey @danwinship 👋 1.29 Enhancements Lead here, is there any more work planned for this KEP? Also is the milestone accurate (are you targeting v1.30 instead of v1.29?)

npolshakova avatar Sep 07 '23 14:09 npolshakova

/remove-label lead-opted-in

npolshakova avatar Sep 08 '23 15:09 npolshakova