enhancements icon indicating copy to clipboard operation
enhancements copied to clipboard

Introduce `MatchLabelKeys` to Pod Affinity and Pod Anti Affinity

Open sanposhiho opened this issue 3 years ago • 64 comments

Enhancement Description

  • One-line enhancement description (can be used as a release note): Introduce MatchLabelKeys to Pod Affinity and Pod Anti Affinity
  • Kubernetes Enhancement Proposal: https://github.com/kubernetes/enhancements/tree/master/keps/sig-scheduling/3633-matchlabelkeys-to-podaffinity
  • Discussion Link: https://github.com/kubernetes/kubernetes/issues/112947
  • Primary contact (assignee): @sanposhiho
  • Responsible SIGs: sig/scheduling
  • Enhancement target (which target equals to which milestone):
    • Alpha release target (x.y): v1.29
    • Beta release target (x.y): v1.30
    • Stable release target (x.y): v1.32
  • [x] Alpha
    • [x] KEP (k/enhancements) update PR(s):
      • https://github.com/kubernetes/enhancements/pull/3651
      • https://github.com/kubernetes/enhancements/pull/4069
    • [x] Code (k/k) update PR(s): 
      • https://github.com/kubernetes/kubernetes/pull/116065
    • [x] Docs (k/website) update PR(s):
      • https://github.com/kubernetes/website/pull/43812
  • [ ] Beta
    • [x] KEP (k/enhancements) update PR(s): https://github.com/kubernetes/enhancements/pull/4450
    • [ ] Code (k/k) update PR(s): https://github.com/kubernetes/kubernetes/pull/123638
    • [ ] Docs (k/website) update(s): https://github.com/kubernetes/website/pull/45181

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

/sig scheduling /assign

sanposhiho avatar Oct 20 '22 00:10 sanposhiho

@kubernetes/sig-scheduling-leads Do you think we can work on it this release? Or should postpone until the next release?

sanposhiho avatar Jan 11 '23 11:01 sanposhiho

I took a review and I think we can include it in 1.27.

/label lead-opted-in

Huang-Wei avatar Jan 11 '23 21:01 Huang-Wei

@Huang-Wei Thanks! I'll take a look and modify KEP this weekend. Do we need to add the milestone to track this issue as a v1.27 feature? If so, could you? (I don't have permission to add milestone on it.) https://github.com/kubernetes/enhancements/milestone/29

sanposhiho avatar Jan 12 '23 04:01 sanposhiho

/milestone v1.27

sanposhiho avatar Jan 12 '23 04:01 sanposhiho

@sanposhiho: You must be a member of the kubernetes/milestone-maintainers GitHub team to set the milestone. If you believe you should be able to issue the /milestone command, please contact your Milestone Maintainers Team and have them propose you as an additional delegate for this responsibility.

In response to this:

/milestone v1.27

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 Jan 12 '23 04:01 k8s-ci-robot

/milestone v1.27

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

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

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

Hello @sanposhiho 👋, 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 alpha for v1.27 (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: v1.27
  • [ ] 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, it looks like once https://github.com/kubernetes/enhancements/pull/3651 merges all the above requirements will be met.

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!

marosset avatar Jan 31 '23 01:01 marosset

Hi @marosset, https://github.com/kubernetes/enhancements/pull/3651 just got merged.

sanposhiho avatar Jan 31 '23 13:01 sanposhiho

Hi @sanposhiho - With #3651 merged this enhancement is now tracked for now inclusion in the v1.27 release.

Thanks!

/label tracked/yes

marosset avatar Jan 31 '23 21:01 marosset

Hi @sanposhiho :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).
  • [ ] All PRs are fully merged by the code freeze deadline.

For this enhancement, it looks like the following PRs are open and need to be merged before code freeze:

  • https://github.com/kubernetes/kubernetes/pull/116065

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

Unfortunately the implementation PRs associated with this enhancement have not merged by code-freeze so this enhancement is getting removed from the release.

If you would like to file an exception please see https://github.com/kubernetes/sig-release/blob/master/releases/EXCEPTIONS.md

/milestone clear /remove-label tracked/yes /label tracked/no

marosset avatar Mar 15 '23 00:03 marosset

One use case for this: https://github.com/kubernetes-sigs/jobset/issues/27

ahg-g avatar Apr 14 '23 21:04 ahg-g

@ahg-g Could you add /milestone v1.28 on it? I'm planning to continue working on it in v1.28.

sanposhiho avatar Apr 15 '23 03:04 sanposhiho

/stage alpha /milestone v1.28 /label lead-opted-in

ahg-g avatar Apr 16 '23 04:04 ahg-g

@ahg-g By your comment above, just checking again if we are opting-in this enhancement for v1.28 release?

Atharva-Shinde avatar Jun 07 '23 15:06 Atharva-Shinde

@ahg-g By your comment above, just checking again if we are opting-in this enhancement for v1.28 release?

Yes we are

ahg-g avatar Jun 07 '23 16:06 ahg-g

/label lead-opted-in

Atharva-Shinde avatar Jun 07 '23 16:06 Atharva-Shinde

Hi @sanposhiho 👋, Enhancements team here!

Just checking in as we approach enhancements freeze on 01:00 UTC Friday, 16th June 2023.

This enhancement is targeting for stage alpha for 1.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.
  • [ ] KEP status is marked as implementable for latest-milestone: 1.28
  • [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.

It looks like there was one field that missed as part of the 1.27 enhancement review. The KEP status field is currently set to provisional but it should be changed to implementable. In addition to that update you need a PRR approval from PRR team. Let me know if you have any questions!

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!

npolshakova avatar Jun 12 '23 22:06 npolshakova

Hi @sanposhiho 👋, just checking in again before the enhancements freeze on 01:00 UTC Friday, 16th June 2023. The status of this enhancement is currently at risk. It looks like the PRR is approved, but the KEP status field needs to be updated to implementable. Let me know if I missed anything. Thanks!

npolshakova avatar Jun 14 '23 19:06 npolshakova

The status is updated. https://github.com/kubernetes/enhancements/pull/4092

sanposhiho avatar Jun 15 '23 00:06 sanposhiho

Hey @sanposhiho

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 15 '23 12:06 Atharva-Shinde

Hello @sanposhiho 1.28 Docs Shadow here.

Does this enhancement work planned for 1.28 require any new docs or modification to existing docs?

If so, please follows the steps here to open a PR against dev-1.28 branch in the k/website repo. This PR can be just a placeholder at this time and must be created before Thursday 20th July 2023.

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

Thank you!

taniaduggal avatar Jun 27 '23 11:06 taniaduggal

Hey again @sanposhiho :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.
  • [ ] 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.

For this enhancement, it looks like the following code related PR/s are open and they need to be merged or should be in merge-ready state before the code freeze commences :

  • https://github.com/kubernetes/kubernetes/pull/116065

Also please let me know if there are other PRs in k/k we should be tracking for this KEP. As always, we are here to help if any questions come up. Thanks!

npolshakova avatar Jul 11 '23 22:07 npolshakova

Hey @sanposhiho , could you please create a docs PR even if it is a draft PR with no content yet against dev-1.28 branch in the k/website repo. The deadline to create this draft PR is Thursday 20th July 2023.

Rishit-dagli avatar Jul 12 '23 15:07 Rishit-dagli

@Rishit-dagli @taniaduggal Sorry about the delay, I missed the message before. I created the draft PR: https://github.com/kubernetes/website/pull/42011

sanposhiho avatar Jul 13 '23 15:07 sanposhiho

Hello @sanposhiho 👋, 1.28 Enhancements Lead here.

Unfortunately, the implementation (code related) PR associated with this enhancement was not in the merge-ready state by the code-freeze and hence this enhancement is now being removed from the v1.28 milestone.

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

/milestone clear

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

/sig scheduling

@sanposhiho are you pursuing this in 1.29?

alculquicondor avatar Sep 20 '23 18:09 alculquicondor

Yes, still waiting for reviews on https://github.com/kubernetes/kubernetes/pull/116065. I'll keep working on it.

sanposhiho avatar Sep 21 '23 12:09 sanposhiho

For now, can you update the target version in the KEP? IIUC, the design stays the same?

alculquicondor avatar Sep 21 '23 13:09 alculquicondor