-
Notifications
You must be signed in to change notification settings - Fork 6
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
upgrade decco to k8s v1.16.x APIs #18
Conversation
Testing on staging cluster reveals that CRD registration is failing and needs more work:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wondering, don't the manifests (and manifest templates) also need to updated?
The only manifest that I see that would need updating is |
Ah ok, I thought we were also using Related to the validation schemas, it is something that kubebuilder generates for us in the Decco V2 work. So, if we merge #13, I can prepare the next PR that adds the validation schema to the CRDs. |
* upgrade decco to k8s v1.16.x APIs * revert CRD changes because not ready to provide required validation schema Co-authored-by: Bich Le <leb@platform9.com>
The following resources have moved to newer API sets:
CRD(reverted because v1 requires validation schema, while v1beta1 is still supported until 1.19)See https://github.com/kubernetes/kubernetes/blob/master/CHANGELOG/CHANGELOG-1.16.md#deprecations-and-removals for details