-
Notifications
You must be signed in to change notification settings - Fork 1
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
Support latest EKS versions #30
Conversation
cluster/data.tf
Outdated
"1.31" : { | ||
"cluster_version" = "1.31" | ||
"cluster_addons" = { | ||
"coredns" = "v1.10.1-eksbuild.10" | ||
"kube_proxy" = "v1.28.1-eksbuild.1" | ||
"vpc_cni" = "v1.16.4-eksbuild.2" | ||
"ebs_csi_driver" = "v1.26.1-eksbuild.1" | ||
"coredns" = "v1.11.3-eksbuild.2" | ||
"kube_proxy" = "v1.30.6-eksbuild.3" | ||
"vpc_cni" = "v1.19.0-eksbuild.1" | ||
"ebs_csi_driver" = "v1.37.0-eksbuild.1" | ||
} | ||
}, | ||
"1.27" : { | ||
"cluster_version" = "1.27" | ||
"1.29" : { | ||
"cluster_version" = "1.31" | ||
"cluster_addons" = { | ||
"coredns" = "v1.10.1-eksbuild.7" | ||
"kube_proxy" = "v1.27.10-eksbuild.2" | ||
"vpc_cni" = "v1.15.3-eksbuild.1" | ||
"ebs_csi_driver" = "v1.25.0-eksbuild.1" | ||
"coredns" = "v1.11.3-eksbuild.2" | ||
"kube_proxy" = "v1.29.10-eksbuild.3" | ||
"vpc_cni" = "v1.19.0-eksbuild.1" | ||
"ebs_csi_driver" = "v1.37.0-eksbuild.1" | ||
} | ||
}, | ||
"1.26" : { | ||
"cluster_version" = "1.26" | ||
"1.28" : { | ||
"cluster_version" = "1.31" |
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.
I can see the mismatch between keys and cluster_versions in the EKS version dict. Are they typos or intentional?
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.
They're fixed now.
@@ -122,6 +122,8 @@ Created resources (if all enabled): | |||
|
|||
Make sure to install [terraform-docs](https://github.com/terraform-docs/terraform-docs) and run `make prepare` before making a commit to make sure the documentation is up to date and the code is valid. | |||
|
|||
In order to support new version of EKS you need to make sure the addons that we use are compatible with the new target version. You can get a list of addons and their EKS compatiblity with running `aws eks describe-addons-versions` and then searching for the addons that are defined in `cluster/data.tf`. | |||
|
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.
👍
1bd8146
to
4514258
Compare
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.
LGTM
This PR adds support for latest version of EKS and its addons. I did a test install for v1.31 and installation was successful. I assume v1.30 will work as wel.