Skip to content

Commit ce56b2c

Browse files
committed
Remove JobReadyPods feature flag
1 parent b2817dc commit ce56b2c

File tree

2 files changed

+1
-13
lines changed

2 files changed

+1
-13
lines changed

pkg/controller/job/job_controller.go

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -827,10 +827,7 @@ func (jm *Controller) syncJob(ctx context.Context, key string) (rErr error) {
827827
newSucceededPods, newFailedPods := getNewFinishedPods(jobCtx)
828828
jobCtx.succeeded = job.Status.Succeeded + int32(len(newSucceededPods)) + int32(len(jobCtx.uncounted.succeeded))
829829
jobCtx.failed = job.Status.Failed + int32(nonIgnoredFailedPodsCount(jobCtx, newFailedPods)) + int32(len(jobCtx.uncounted.failed))
830-
var ready *int32
831-
if feature.DefaultFeatureGate.Enabled(features.JobReadyPods) {
832-
ready = ptr.To(countReadyPods(jobCtx.activePods))
833-
}
830+
ready := ptr.To(countReadyPods(jobCtx.activePods))
834831

835832
// Job first start. Set StartTime only if the job is not in the suspended state.
836833
if job.Status.StartTime == nil && !jobSuspended(&job) {

pkg/features/kube_features.go

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -345,13 +345,6 @@ const (
345345
// based on the set of succeeded pods.
346346
JobSuccessPolicy featuregate.Feature = "JobSuccessPolicy"
347347

348-
// owner: @alculquicondor
349-
// alpha: v1.23
350-
// beta: v1.24
351-
//
352-
// Track the number of pods with Ready condition in the Job status.
353-
JobReadyPods featuregate.Feature = "JobReadyPods"
354-
355348
// owner: @marquiz
356349
// kep: http://kep.k8s.io/4033
357350
// alpha: v1.28
@@ -1041,8 +1034,6 @@ var defaultKubernetesFeatureGates = map[featuregate.Feature]featuregate.FeatureS
10411034

10421035
JobSuccessPolicy: {Default: false, PreRelease: featuregate.Alpha},
10431036

1044-
JobReadyPods: {Default: true, PreRelease: featuregate.GA, LockToDefault: true}, // remove in 1.31
1045-
10461037
KubeletCgroupDriverFromCRI: {Default: false, PreRelease: featuregate.Alpha},
10471038

10481039
KubeletInUserNamespace: {Default: false, PreRelease: featuregate.Alpha},

0 commit comments

Comments
 (0)