-
Notifications
You must be signed in to change notification settings - Fork 114
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
could not apply CRDs #600
Comments
Also running the MCM gives errors (unsurprisingly):
|
Hi @deitch , This is an issue @AxiomSamarth had already looked into. He had proposed a fix here - #592. I am unsure what the status is, he can probably give more details on this. |
Darn! And I thought I had found a new one. 😄 Thanks |
In the meantime, how do I test |
Does removing this part of the CRD fix it ? - https://github.com/gardener/machine-controller-manager/pull/592/files#diff-b96da678ae66c7304b1f784d2d9e3d405a5ddcb1ba251f7c47e04aefdef17a80L62-L64 |
Indeed, it does! I still get the deprecation warning, but it accepts it.
|
Hey @prashanth26, As Hardik has mentioned, maybe the autogeneration of the CRDs, is reverting it. So, even if we change the CRD file in the repo, it may get updated. I am not sure how I can fix this autogeneration. |
We might have to update the k8s version (to the latest) in our vendors to permanently fix this issue - #402. Let's close this PR and pick this issue on priority. |
/close as the issue has been resolved with vendoring k8s v1.20.6 |
What happened:
I tried to apply the CRDs, in order to test individual providers. I got a warning and an error:
What you expected to happen:
it would apply them without error
How to reproduce it (as minimally and precisely as possible):
kubectl apply -f kubernetes/crds/
Anything else we need to know:
The docs here are wrong, in that they say to apply
kubernetes/crds.yaml
, butkubernetes/crds/
now is a directoryEnvironment:
Running k3d on Docker Desktop for Mac.
The text was updated successfully, but these errors were encountered: