Skip to content

Commit 8e392b3

Browse files
authored
Merge pull request kubernetes#4912 from kerthcet/feat/graduate-to-ga
KEP-3094: Graduate NodeInclusionPolicyInPodTopologySpread to GA
2 parents b7b4568 + 853122b commit 8e392b3

File tree

3 files changed

+24
-17
lines changed

3 files changed

+24
-17
lines changed

keps/prod-readiness/sig-scheduling/3094.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,3 +3,5 @@ alpha:
33
approver: "@wojtek-t"
44
beta:
55
approver: "@wojtek-t"
6+
stable:
7+
approver: "@wojtek-t"

keps/sig-scheduling/3094-pod-topology-spread-considering-taints/README.md

Lines changed: 15 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -134,16 +134,16 @@ Items marked with (R) are required *prior to targeting to a milestone / release*
134134
- [x] (R) KEP approvers have approved the KEP status as `implementable`
135135
- [x] (R) Design details are appropriately documented
136136
- [x] (R) Test plan is in place, giving consideration to SIG Architecture and SIG Testing input (including test refactors)
137-
- [ ] e2e Tests for all Beta API Operations (endpoints)
137+
- [x] e2e Tests for all Beta API Operations (endpoints)
138138
- [ ] (R) Ensure GA e2e tests for meet requirements for [Conformance Tests](https://github.com/kubernetes/community/blob/master/contributors/devel/sig-architecture/conformance-tests.md)
139-
- [ ] (R) Minimum Two Week Window for GA e2e tests to prove flake free
139+
- [x] (R) Minimum Two Week Window for GA e2e tests to prove flake free
140140
- [x] (R) Graduation criteria is in place
141-
- [ ] (R) [all GA Endpoints](https://github.com/kubernetes/community/pull/1806) must be hit by [Conformance Tests](https://github.com/kubernetes/community/blob/master/contributors/devel/sig-architecture/conformance-tests.md)
142-
- [ ] (R) Production readiness review completed
141+
- [x] (R) [all GA Endpoints](https://github.com/kubernetes/community/pull/1806) must be hit by [Conformance Tests](https://github.com/kubernetes/community/blob/master/contributors/devel/sig-architecture/conformance-tests.md)
142+
- [x] (R) Production readiness review completed
143143
- [ ] (R) Production readiness review approved
144144
- [x] "Implementation History" section is up-to-date for milestone
145-
- [ ] User-facing documentation has been created in [kubernetes/website], for publication to [kubernetes.io]
146-
- [ ] Supporting documentation—e.g., additional design documents, links to mailing list discussions/SIG meetings, relevant PRs/issues, release notes
145+
- [x] User-facing documentation has been created in [kubernetes/website], for publication to [kubernetes.io]
146+
- [x] Supporting documentation—e.g., additional design documents, links to mailing list discussions/SIG meetings, relevant PRs/issues, release notes
147147

148148
<!--
149149
**Note:** This checklist is iterative and should be reviewed and updated every time this enhancement is being considered for a milestone.
@@ -364,11 +364,11 @@ This can inform certain test coverage improvements that we want to do before
364364
extending the production code to implement this enhancement.
365365
-->
366366

367-
- `pkg/api/pod`: `2022-06-17` - `66.7%`
368-
- `pkg/apis/core/validation`: `2022-06-17` - `82.1%`
369-
- `pkg/scheduler`: `2022-06-17` - `75%`
370-
- `pkg/scheduler/framework/plugins/defaultpreemption`: `2022-06-17` - `85.2%`
371-
- `pkg/scheduler/framework/plugins/podtopologyspread`: `2022-06-17` - `86%`
367+
- `pkg/api/pod`: `2025-02-10` - `79.1%`
368+
- `pkg/apis/core/validation`: `2025-02-10` - `84.4%`
369+
- `pkg/scheduler`: `2025-02-10` - `80.7%`
370+
- `pkg/scheduler/framework/plugins/defaultpreemption`: `2025-02-10` - `80.4%`
371+
- `pkg/scheduler/framework/plugins/podtopologyspread`: `2025-02-10` - `87.5%`
372372

373373
##### Integration tests
374374

@@ -566,7 +566,8 @@ feature, can it break the existing applications?).
566566
567567
NOTE: Also set `disable-supported` to `true` or `false` in `kep.yaml`.
568568
-->
569-
Yes, we can just disable the feature gate.
569+
The feature can be disabled in Alpha and Beta stage, but once GA, there's no way to disable it.
570+
But you can leave it opt-out by unset the two fields, and it will fall back to the default behavior.
570571

571572
###### What happens if we reenable the feature if it was previously rolled back?
572573
The policies are respected again.
@@ -792,8 +793,7 @@ Describe the metrics themselves and the reasons why they weren't added (e.g., co
792793
implementation difficulties, etc.).
793794
-->
794795

795-
Yes, we have a plan to improve observability via metrics [here](https://github.com/kubernetes/kubernetes/issues/110643),
796-
but still on the way.
796+
No.
797797

798798
### Dependencies
799799

@@ -954,6 +954,7 @@ Major milestones might include:
954954
- 2021.01.12: KEP proposed for review, including motivation, proposal, risks,
955955
test plan and graduation criteria.
956956
- 2022.09.22: Graduate to Beta in v1.26.
957+
- 2024.10.10: Graduate to GA in v1.33.
957958

958959
## Drawbacks
959960

keps/sig-scheduling/3094-pod-topology-spread-considering-taints/kep.yaml

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -17,18 +17,18 @@ see-also:
1717
- "/keps/sig-scheduling/1258-default-pod-topology-spread"
1818

1919
# The target maturity stage in the current dev cycle for this KEP.
20-
stage: beta
20+
stage: stable
2121

2222
# The most recent milestone for which work toward delivery of this KEP has been
2323
# done. This can be the current (upcoming) milestone, if it is being actively
2424
# worked on.
25-
latest-milestone: "v1.25"
25+
latest-milestone: "v1.33"
2626

2727
# The milestone at which this feature was, or is targeted to be, at each stage.
2828
milestone:
2929
alpha: "v1.25"
3030
beta: "v1.26"
31-
stable: ""
31+
stable: "1.33"
3232

3333
# The following PRR answers are required at alpha release
3434
# List the feature gate name and the components for which it must be enabled
@@ -39,3 +39,7 @@ feature-gates:
3939
- kube-scheduler
4040

4141
disable-supported: false
42+
43+
metrics:
44+
- plugin_execution_duration_seconds{plugin="PodTopologySpread"}
45+
- schedule_attempts_total{result="error|unschedulable"}

0 commit comments

Comments
 (0)