Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pkg/webhook: k8s.io/api/admission/v1beta1 is deprecated in v1.19 #1123

Closed
estroz opened this issue Aug 8, 2020 · 9 comments
Closed

pkg/webhook: k8s.io/api/admission/v1beta1 is deprecated in v1.19 #1123

estroz opened this issue Aug 8, 2020 · 9 comments
Assignees

Comments

@estroz
Copy link
Contributor

estroz commented Aug 8, 2020

See kubernetes/kubernetes#90300. Bumping to v1 is a breaking change.

/cc @vincepri @DirectXMan12

@estroz
Copy link
Contributor Author

estroz commented Aug 8, 2020

v1beta1 {Mutating,Validating}WebhookConfigurations have been deprecated for awhile now (v1.16), see kubernetes/kubernetes#90301. Perhaps it's worth removing support for them as well.

@alvaroaleman
Copy link
Member

I don't think we can remove support for v1beta1 webhooksconfigurations because the major cloudproviders lack so severly behind in versions. GKE does support 1.16, but its not the default. Not sure about AKS and EKS.

@estroz
Copy link
Contributor Author

estroz commented Aug 10, 2020

Right. I think v1 and v1beta1 can coexist, with v1beta1 types/methods having a Legacy suffix. This would still be a breaking change, but probably should be to notify users that webhook defaults are changing and that they should upgrade if possible. Thoughts?

@alvaroaleman
Copy link
Member

sgtm. Maybe also add a comment telling users the minimum version needed (I believe its v1.16 for v1)

@vincepri
Copy link
Member

SGTM, let's do it :)

@jiachengxu
Copy link
Contributor

/assign

@adrianludwin
Copy link

Was this fixed by #1284 ?

@estroz
Copy link
Contributor Author

estroz commented Dec 14, 2020

@adrianludwin yes!

/close

@estroz estroz closed this as completed Dec 14, 2020
@adrianludwin
Copy link

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants