enhancements icon indicating copy to clipboard operation
enhancements copied to clipboard

Respect PodTopologySpread after rolling upgrades

Open denkensk opened this issue 3 years ago • 67 comments
trafficstars

Enhancement Description

  • One-line enhancement description (can be used as a release note): Respect PodTopologySpread after rolling upgrades
  • Kubernetes Enhancement Proposal: https://github.com/kubernetes/enhancements/blob/master/keps/sig-scheduling/3243-respect-pod-topology-spread-after-rolling-upgrades/README.md
  • Discussion Link: https://docs.google.com/document/d/12F1YXsvy4SmumPiL3LHMLnuMHvyMqSzDqF-L9J37ZfA/edit?usp=sharing
  • Primary contact (assignee): @denkensk
  • Responsible SIGs: /sig scheduling
  • Enhancement target (which target equals to which milestone):
    • Alpha release target (x.y): 1.25
    • Beta release target (x.y): 1.27
    • Stable release target (x.y): 1.29
  • [x] Alpha
    • [x] KEP (k/enhancements) update PR(s): https://github.com/kubernetes/enhancements/pull/3244
    • [x] Code (k/k) update PR(s): https://github.com/kubernetes/kubernetes/pull/111441
    • [x] Docs (k/website) update PR(s): https://github.com/kubernetes/website/pull/35165
  • [x] Beta
    • [x] KEP (k/enhancements) update PR(s): https://github.com/kubernetes/enhancements/pull/3740
    • [x] Code (k/k) update PR(s):
      • https://github.com/kubernetes/kubernetes/pull/116291
      • https://github.com/kubernetes/kubernetes/pull/129874
    • [x] Docs (k/website) update(s): https://github.com/kubernetes/website/pull/39826

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

denkensk avatar Mar 18 '22 08:03 denkensk

/sig scheduling

denkensk avatar Mar 18 '22 08:03 denkensk

/assign

denkensk avatar Jun 09 '22 09:06 denkensk

/milestone v1.25

ahg-g avatar Jun 15 '22 14:06 ahg-g

Hello @denkensk 👋, 1.25 Enhancements team here.

Just checking in as we approach enhancements freeze on 18:00 PST on Thursday June 23, 2022.

For note, This enhancement is targeting for stage alpha for 1.25 (correct me, if otherwise)

Here's where this enhancement currently stands:

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

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

Priyankasaggu11929 avatar Jun 15 '22 15:06 Priyankasaggu11929

Hi @denkensk Enhancements team here again 👋

Checking in as we approach Code Freeze at 01:00 UTC on Wednesday, 3rd August 2022.

Please ensure that the following items are completed before the code-freeze:

  • [ ] 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.

Currently, the status of the enhancement is marked as at-risk

Thanks :)

Atharva-Shinde avatar Jul 25 '22 15:07 Atharva-Shinde

Hey @denkensk as https://github.com/kubernetes/kubernetes/pull/111441 PR is now merged and included in the issue description, I have marked this enhancement as tracked :)

Atharva-Shinde avatar Aug 01 '22 15:08 Atharva-Shinde

@denkensk can you please open a docs PR?

ahg-g avatar Aug 05 '22 17:08 ahg-g

Sorry. I created it before, forgot to link it with this issue. https://github.com/kubernetes/website/pull/35165

denkensk avatar Aug 06 '22 00:08 denkensk

Hope to see this feature promoting to beta in v1.27, we delayed our blog about the improvements of PodTopologySpread in recent several releases, and we hope to post one in v1.27, including minDomains, nodeInclusionPolicy.

If you're out of bandwidth, I can help with you @denkensk

kerthcet avatar Dec 06 '22 04:12 kerthcet

Thanks. I will promote it to beta in v1.27.

denkensk avatar Dec 09 '22 08:12 denkensk

@denkensk note that the Enhancements freeze has been set for February 10th.

alculquicondor avatar Jan 09 '23 17:01 alculquicondor

@denkensk are you planning to move this to beta? @kerthcet would you like to pick it up if @denkensk can't?

ahg-g avatar Jan 11 '23 21:01 ahg-g

@denkensk are you planning to move this to beta? @kerthcet would you like to pick it up if @denkensk can't?

I can help, but up to @denkensk

kerthcet avatar Jan 12 '23 02:01 kerthcet

@denkensk note that the Enhancements freeze has been set for February 10th.

OK Thanks. I am working on this and will submit PR this week.

denkensk avatar Jan 12 '23 02:01 denkensk

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

ahg-g avatar Jan 12 '23 06:01 ahg-g

/stage beta

ahg-g avatar Jan 23 '23 15:01 ahg-g

Hello @denkensk 👋, v1.27 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 1.27 (please 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.
  • [ ] KEP status is marked as implementable for latest-milestone: 1.27
  • [X] KEP readme has a updated detailed test plan section filled out
  • [X] 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 need to update the following:

  • Update kep.yaml to reflect current stage information.
  • Complete the missing PRR sections

#3740 might address this, once it's merged.

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!

fsmunoz avatar Feb 01 '23 12:02 fsmunoz

Update kep.yaml to reflect current stage information.

Done @fsmunoz

denkensk avatar Feb 02 '23 03:02 denkensk

Thank you @denkensk ! This is almost there, there is a new question in the PRR Scalability that you might want to address:

Can enabling / using this feature result in resource exhaustion of some node resources (PIDs, sockets, inodes, etc.)?

The Scalability section of the PRR is mandatory for beta, hence why I am mentioning it.

fsmunoz avatar Feb 05 '23 17:02 fsmunoz

This is almost there, there is a new question in the PRR Scalability that you might want to address:

Done @fsmunoz

denkensk avatar Feb 08 '23 10:02 denkensk

Thank you @denkensk !

  • [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.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.

This enhancement is ready to be traced for graduation to beta in v1.27. /label tracked/yes /remove-label tracked/no

fsmunoz avatar Feb 08 '23 16:02 fsmunoz

Hi @denkensk 👋,

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.
  • https://github.com/kubernetes/kubernetes/pull/116291

For this enhancement, it looks like all linked PRs are merged.

Please let me know what other PRs in k/k I should be tracking for this KEP.

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

fsmunoz avatar Mar 13 '23 10:03 fsmunoz

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

This bot triages un-triaged issues according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue as fresh with /remove-lifecycle stale
  • Close this issue with /close
  • Offer to help out with Issue Triage

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

/lifecycle stale

k8s-triage-robot avatar Jan 20 '24 07:01 k8s-triage-robot

/remove-lifecycle stale

alculquicondor avatar Jan 22 '24 13:01 alculquicondor

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

This bot triages un-triaged issues according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue as fresh with /remove-lifecycle stale
  • Close this issue with /close
  • Offer to help out with Issue Triage

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

/lifecycle stale

k8s-triage-robot avatar Apr 21 '24 14:04 k8s-triage-robot