This repository has been archived by the owner on Dec 15, 2022. It is now read-only.
generated from crossplane/provider-template
-
Notifications
You must be signed in to change notification settings - Fork 30
README Addition #163
Comments
Feel free to open a PR with this contribution @natereid72, along with the explanation/details for why that version is needed :) |
I'll close that PR and open a new one tomorrow. Need to switch laptops to sign. |
idk, @jbw976 we may need to sync tomorrow on this. I signed it but DCO still showing issue with not signed, |
Since this is an AWS provider, I assume a lot of people (like me) will be running it on EKS, which only supports up to Kubernetes 1.21. I think a recommendation of 1.23 would be fine, with an explanation of what the issue(s) are with earlier versions. |
2 tasks
ok, replaced previous PR with #167. Git amend didn't seem to fix the sign-off issue in previous PR. Added statement on requirement and brief reason. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Can we add requirement for Kubernetes API server version =>v1.23 to the README?
The text was updated successfully, but these errors were encountered: