Skip to content

Commit 335eaa9

Browse files
authored
Merge pull request kubernetes#3449 from soltysh/cronjob_tz_beta
KEP 3140: Promote CronJob's TimeZone support to GA
2 parents 5473613 + 9176a2f commit 335eaa9

File tree

3 files changed

+14
-42
lines changed

3 files changed

+14
-42
lines changed

keps/prod-readiness/sig-apps/3140.yaml

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

keps/sig-apps/3140-TimeZone-support-in-CronJob/README.md

Lines changed: 10 additions & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -90,10 +90,6 @@ fully GA-ed it's time to satisfy the original request.
9090

9191
### Non-Goals
9292

93-
<!--
94-
What is out of scope for this KEP? Listing non-goals helps to focus discussion
95-
and make progress.
96-
-->
9793

9894
## Proposal
9995

@@ -118,6 +114,10 @@ This problem can be mitigated with a fresh build of kube-controller-manager with
118114
updated golang version, but it heavily relies on the go community to keep the
119115
time zone database up-to-date.
120116

117+
- Outdated time zone database on the system where kube-controller-manager is running
118+
119+
Cluster administrator should always ensure their systems are up-to-date.
120+
121121
- Malicious user can create multiple CronJobs with different time zone which can
122122
actually trigger Jobs at the exact same time
123123

@@ -175,17 +175,17 @@ to implement this enhancement.
175175

176176
##### Unit tests
177177

178-
- `k8s.io/kubernetes/pkg/apis/batch/validation`: `2022-06-09` - `94.4%`
179-
- `k8s.io/kubernetes/pkg/controller/cronjob`: `2022-06-09` - `50.8%`
180-
- `k8s.io/kubernetes/pkg/registry/batch/cronjob`: `2022-06-09` - `61.8%`
178+
- `k8s.io/kubernetes/pkg/apis/batch/validation`: `2023-01-18` - `96.0%`
179+
- `k8s.io/kubernetes/pkg/controller/cronjob`: `2023-01-18` - `51.7%`
180+
- `k8s.io/kubernetes/pkg/registry/batch/cronjob`: `2023-01-18` - `56.3%`
181181

182182
##### Integration tests
183183

184184
None.
185185

186186
##### e2e tests
187187

188-
None.
188+
- [CronJob should support timezone](https://github.com/kubernetes/kubernetes/blob/db4c64b1a987675fff7a234e8633f4fd01c69530/test/e2e/apps/cronjob.go#L301): https://storage.googleapis.com/k8s-triage/index.html?sig=apps&test=should%20support%20timezone
189189

190190
### Graduation Criteria
191191

@@ -203,38 +203,7 @@ None.
203203

204204
#### GA
205205

206-
TBD
207-
208-
<!---
209-
#### Beta
210-
211-
- Gather feedback from developers and surveys
212-
- Complete features A, B, C
213-
- Additional tests are in Testgrid and linked in KEP
214-
215-
#### GA
216-
217-
- N examples of real-world usage
218-
- N installs
219-
- More rigorous forms of testing—e.g., downgrade tests and scalability tests
220-
- Allowing time for feedback
221-
222-
**Note:** Generally we also wait at least two releases between beta and
223-
GA/stable, because there's no opportunity for user feedback, or even bug reports,
224-
in back-to-back releases.
225-
226-
**For non-optional features moving to GA, the graduation criteria must include
227-
[conformance tests].**
228-
229-
[conformance tests]: https://git.k8s.io/community/contributors/devel/sig-architecture/conformance-tests.md
230-
231-
#### Deprecation
232-
233-
- Announce deprecation and support policy of the existing flag
234-
- Two versions passed since introducing the functionality that deprecates the flag (to address version skew)
235-
- Address feedback on usage/changed behavior, provided on GitHub issues
236-
- Deprecate the flag
237-
-->
206+
- Add a new condition when a cronjob controller encounters an invalid time zone.
238207

239208
### Upgrade / Downgrade Strategy
240209

@@ -429,6 +398,7 @@ lies.
429398

430399
- *2022-01-14* - Initial KEP draft
431400
- *2022-06-09* - Updated KEP for beta promotion.
401+
- *2023-01-18* - Updated KEP for stable promotion.
432402

433403
## Drawbacks
434404

keps/sig-apps/3140-TimeZone-support-in-CronJob/kep.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,12 +18,12 @@ see-also:
1818
replaces:
1919

2020
# The target maturity stage in the current dev cycle for this KEP.
21-
stage: beta
21+
stage: stable
2222

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

2828
# The milestone at which this feature was, or is targeted to be, at each stage.
2929
milestone:

0 commit comments

Comments
 (0)