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

[feature] support k8s.io/api/admission/v1 #20

Open
drnic opened this issue Jan 4, 2020 · 2 comments
Open

[feature] support k8s.io/api/admission/v1 #20

drnic opened this issue Jan 4, 2020 · 2 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@drnic
Copy link

drnic commented Jan 4, 2020

There are two versions APIs for webhooks, and two corresponding APIs versions for the webhook request + response which is handled by this library

https://kubernetes.io/docs/reference/access-authn-authz/extensible-admission-controllers/#webhook-request-and-response

  • admissionregistration.k8s.io/v1beta1 - handled by this library, but deprecated in kube v1.16
  • admissionregistration.k8s.io/v1

Do you have some ideas how this library might migrate to support the new API? Support both in parallel; drop v1beta1 and only support v1; or not support v1 for some X months/Y years and move only when v1beta1 starts being removed?

@drnic drnic added the enhancement New feature or request label Jan 4, 2020
drnic added a commit to drnic/jeopardy-nodeselector that referenced this issue Jan 4, 2020
@elithrar
Copy link
Owner

elithrar commented Jan 4, 2020 via email

@drnic
Copy link
Author

drnic commented Jan 4, 2020

I understand the business of "which kube APIs to support" isn't trivial -- GKE default kubernetes version is 1.14 for example.

@elithrar elithrar self-assigned this Mar 8, 2020
@elithrar elithrar added this to the v1 🚧 milestone Mar 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants