chore: Update EKS module version to v20.0
#1871
Conversation
| @@ -58,7 +58,7 @@ jobs: | |||
| - name: Setup Terraform | |||
| uses: hashicorp/setup-terraform@v3 | |||
| with: | |||
| terraform_version: 1.0.0 | |||
| terraform_version: 1.3.10 | |||
There was a problem hiding this comment.
The EKS module now requires MSTV of 1.3 so updating this in our e2e tests as well
| @@ -29,6 +29,7 @@ repos: | |||
| - '--args=--only=terraform_naming_convention' | |||
| - '--args=--only=terraform_required_version' | |||
| - '--args=--only=terraform_required_providers' | |||
| - '--args=--only=terraform_unused_required_providers' | |||
There was a problem hiding this comment.
nifty little lint check to make sure we remove providers when they aren't being utilized
| @@ -127,29 +127,6 @@ module "eks" { | |||
| group_name = aws_placement_group.efa.name | |||
| } | |||
|
|
|||
| pre_bootstrap_user_data = <<-EOT | |||
There was a problem hiding this comment.
this is available by default today through 1.25+ and will soon be available through 1.23+
| @@ -275,7 +252,7 @@ resource "kubernetes_daemonset" "aws_efa_k8s_device_plugin" { | |||
|
|
|||
| container { | |||
| name = "aws-efa-k8s-device-plugin" | |||
| image = "602401143452.dkr.ecr.us-west-2.amazonaws.com/eks/aws-efa-k8s-device-plugin:v0.3.3" | |||
| image = "602401143452.dkr.ecr.us-west-2.amazonaws.com/eks/aws-efa-k8s-device-plugin:v0.4.3" | |||
There was a problem hiding this comment.
this was updated recently for some CVE fixes https://github.com/aws-samples/aws-efa-eks/pull/21/files
| cluster_endpoint_public_access = true | ||
|
|
||
| cluster_addons = { | ||
| coredns = {} | ||
| kube-proxy = {} | ||
| vpc-cni = { | ||
| preserve = true |
There was a problem hiding this comment.
EKS module v20 now sets this by default
| @@ -50,10 +50,10 @@ locals { | |||
|
|
|||
| module "eks" { | |||
| source = "terraform-aws-modules/eks/aws" | |||
| version = "~> 19.16" | |||
| version = "~> 19.21" | |||
There was a problem hiding this comment.
need to update teams module to add native access entry support and then we can remove the aws-auth bits here and update to v20
| @@ -16,10 +16,10 @@ provider "kubernetes" { | |||
|
|
|||
| module "eks" { | |||
| source = "terraform-aws-modules/eks/aws" | |||
| version = "~> 19.16" | |||
| version = "~> 19.21" | |||
There was a problem hiding this comment.
will update in separate PR to resolve #1816
bryantbiggs
force-pushed
the
chore/update-eks-module
branch
from
7577013 to
54f739c
Compare
Description
v20.0Motivation and Context
How was this change tested?
pre-commit run -awith this PRAdditional Notes