This change is 

@kubernetes/sig-api-machinery-pr-reviews ptal as well

Moved this issue into the next milestone @soltysh. This PR did not get lgtm before code freeze and needs rebase and or rework. Feel free to file an exception if you feel it's warranted.

cc: @ethernetdan, @kubernetes/kubernetes-release-managers

@calebamiles I did file an exception, I'm waiting for hearing back.

It looks like the cronjob controller is tightly coupled with batch/v2alpha version and when batch/v2alpha1 and batch/v1beta1 are enabled at once it confuses kubectl on the other hand. So I need to put a lot more work to make it usable. Again multiversioning problems, sigh... 😕

@soltysh would it be easier if we remove batch/v2alpha1 altogether?

@soltysh would it be easier if we remove batch/v2alpha1 altogether?

It would definitely be easier, not sure if desirable, though. @erictune any objections on Janet's suggestion? I'll bring it up during next sig-apps call, to have a broader audience.

It would definitely be easier, not sure if desirable, though. @erictune any objections on Janet's suggestion? I'll bring it up during next sig-apps call, to have a broader audience.

It seems mean to break users when the serialization and behavior is actually compatible. Perhaps a series of simple reproducers will help us push past the problems with help from @kubernetes/sig-cli-bugs

There's a proposal that could in theory subsume cron jobs
https://docs.google.com/document/d/1-H2hnZap7gQivcSU-9j4ZrJ8wE_WwcfOkTeAGjzUyLA/edit
but it will require a lot of work.

@davidopp will you be able to attend sig-apps today and we can have a discussion.

@soltysh Sorry, didn't see this in time.

We'll be discussing the proposal at the sig-scheduling meeting today at 2:30 PM PT https://zoom.us/j/966775765

@davidopp sorry but too late for me :( I'll watch the recording and will catch up with you later this week.

@janetkuo ptal, I've enabled only the API here so that we can move with this forward. I need to address the remaining bits, such us how to deal with CronJobs when both v2alpha1 and v1beta1 is enabled. But I'd prefer to do it as a followup, for now I've turned off this new API and when all the pieces will be in I'll re-enable it permanently.

Forgot to actually comment the v1beta1 in master.go. Fixed, should be good.

One more try, I've rebased hack/update-all.sh-ed and crossing my fingers.

Fixed last bit, the test/integration/etcd/etcd_storage_path_test.go. Should be green now 🤞

/retest

@soltysh: The following tests failed, say /retest to rerun them all:

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

@erictune @janetkuo it's finally green 😄 mind pushing this forward?

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: erictune, smarterclayton, soltysh

Associated issue: 41890

The full list of commands accepted by this bot can be found here.

/test all [submit-queue is verifying that this PR is safe to merge]

Automatic merge from submit-queue