Skip to content

Commit

Permalink
Merge pull request #16 from terraform-modules/fix-providers
Browse files Browse the repository at this point in the history
Fix providers and upgrade docs
  • Loading branch information
badra001 committed Jun 11, 2026
2 parents b747aec + 9df7485 commit ec94da0
Show file tree
Hide file tree
Showing 5 changed files with 55 additions and 42 deletions.
4 changes: 2 additions & 2 deletions examples/full-cluster-tf-upgrade/1.33/charts.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,13 +2,13 @@ cluster-autoscaler:
documentation: "https://artifacthub.io/packages/helm/cluster-autoscaler/cluster-autoscaler"
name: "cluster-autoscaler"
repository: "https://kubernetes.github.io/autoscaler"
version: "9.52.1"
version: "9.47.0"
use_remote: true
cert-manager:
documetation: "https://artifacthub.io/packages/helm/cert-manager/cert-manager"
name: "cert-manager"
repository: "https://charts.jetstack.io"
version: "1.19.1"
version: "1.19.5"
use_remote: true
metrics-server:
# documentation: "https://artifacthub.io/packages/helm/bitnami/metrics-server"
Expand Down
26 changes: 13 additions & 13 deletions examples/full-cluster-tf-upgrade/1.33/images.yml
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ karpenter-controller:
source_tag: null
enabled: false
# tag: "1.0.6"
tag: "1.8.0"
tag: "1.9.0"
cluster-autoscaler:
documentation: "https://github.com/kubernetes/autoscaler/releases"
name: "cluster-autoscaler"
Expand All @@ -18,7 +18,7 @@ cluster-autoscaler:
source_image: "autoscaling/cluster-autoscaler"
source_tag: null
enabled: true
tag: "v1.34.1"
tag: "v1.33.4"
cert-manager-controller:
documentation: "https://cert-manager.io/docs/releases/"
name: "cert-manager-controller"
Expand All @@ -28,7 +28,7 @@ cert-manager-controller:
source_image: "jetstack/cert-manager-controller"
source_tag: null
enabled: true
tag: "v1.19.1"
tag: "v1.19.5"
cert-manager-cainjector:
documentation: "https://cert-manager.io/docs/releases/"
name: "cert-manager-cainjector"
Expand All @@ -38,7 +38,7 @@ cert-manager-cainjector:
source_image: "jetstack/cert-manager-cainjector"
source_tag: null
enabled: true
tag: "v1.19.1"
tag: "v1.19.5"
cert-manager-webhook:
documentation: "https://cert-manager.io/docs/releases/"
name: "cert-manager-webhook"
Expand All @@ -48,7 +48,7 @@ cert-manager-webhook:
source_image: "jetstack/cert-manager-webhook"
source_tag: null
enabled: true
tag: "v1.19.1"
tag: "v1.19.5"
cert-manager-ctl:
documentation: "https://cert-manager.io/docs/releases/"
name: "cert-manager-ctl"
Expand All @@ -59,7 +59,7 @@ cert-manager-ctl:
source_tag: null
enabled: false
# tag: "v1.16.2"
tag: "v1.14.7"
tag: "v1.19.5"
cert-manager-startupapicheck:
documentation: "https://cert-manager.io/docs/releases/"
name: "cert-manager-startupapicheck"
Expand All @@ -69,7 +69,7 @@ cert-manager-startupapicheck:
source_image: "jetstack/cert-manager-startupapicheck"
source_tag: null
enabled: true
tag: "v1.19.1"
tag: "v1.19.5"
metrics-server:
# documentation: "https://hub.docker.com/r/bitnami/metrics-server/tags"
name: "metrics-server"
Expand All @@ -93,7 +93,7 @@ istio-operator:
source_image: "istio/operator"
source_tag: null
enabled: true
tag: "1.23.6"
tag: "1.26.8"
istio-pilot:
documentation: "https://istio.io/latest/docs/releases/supported-releases"
name: "istio/pilot"
Expand All @@ -104,7 +104,7 @@ istio-pilot:
source_tag: null
enabled: true
# tag: "1.25.3"
tag: "1.28.0"
tag: "1.26.8"
istio-proxyv2:
documentation: "https://istio.io/latest/docs/releases/supported-releases"
name: "istio/proxyv2"
Expand All @@ -115,7 +115,7 @@ istio-proxyv2:
source_tag: null
enabled: true
# tag: "1.25.3"
tag: "1.28.0"
tag: "1.26.8"
prometheus:
documentation: "https://hub.docker.com/r/bitnami/prometheus/tags"
name: "prometheus"
Expand All @@ -128,7 +128,7 @@ prometheus:
enabled: true
# tag: "3.0.1"
# tag: "v2.54.0"
tag: "v3.8.0"
tag: "v3.9.1"
prometheus-operator:
# documentation: "https://hub.docker.com/r/bitnami/prometheus-operator/tags"
name: "prometheus-operator"
Expand All @@ -143,7 +143,7 @@ prometheus-operator:
enabled: true
# tag: "0.79.2"
## tag: "v0.74.0"
tag: "v0.87.0"
tag: "v0.89.0"
alertmanager:
# documentation: "https://hub.docker.com/r/bitnami/alertmanager/tags"
name: "alertmanager"
Expand All @@ -155,4 +155,4 @@ alertmanager:
enabled: true
# tag: "0.27.0"
## tag: "v0.28.0"
tag: "v0.29.0"
tag: "v0.31.1"
22 changes: 11 additions & 11 deletions examples/full-cluster-tf-upgrade/1.33/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,7 @@ resource "aws_eks_node_group" "eks-nodegroup" {
node_group_name = format("%v%v-nodegroup", local._prefixes["eks"], var.cluster_name)
node_role_arn = module.role_eks-nodegroup.role_arn
subnet_ids = local.subnets
ami_type = "AL2_x86_64"
ami_type = "AL2023_x86_64_STANDARD"
# instance_types = [var.eks_instance_type]
# disk_size = var.eks_instance_disk_size

Expand Down Expand Up @@ -230,17 +230,17 @@ resource "aws_launch_template" "eks-nodegroup" {
}
}

user_data = base64encode(local.eks-node-private-userdata)
# user_data = base64encode(local.eks-node-private-userdata)
}

#### User data for worker launch

locals {
eks-node-private-userdata = templatefile(
"${path.module}/templates/node-private-userdata.tmpl", {
endpoint = aws_eks_cluster.eks_cluster.endpoint
cluster_ca = aws_eks_cluster.eks_cluster.certificate_authority[0].data
cluster_name = var.cluster_name
}
)
}
# locals {
# eks-node-private-userdata = templatefile(
# "${path.module}/templates/node-private-userdata.tmpl", {
# endpoint = aws_eks_cluster.eks_cluster.endpoint
# cluster_ca = aws_eks_cluster.eks_cluster.certificate_authority[0].data
# cluster_name = var.cluster_name
# }
# )
# }
22 changes: 11 additions & 11 deletions examples/full-cluster-tf-upgrade/1.34/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,7 @@ resource "aws_eks_node_group" "eks-nodegroup" {
node_group_name = format("%v%v-nodegroup", local._prefixes["eks"], var.cluster_name)
node_role_arn = module.role_eks-nodegroup.role_arn
subnet_ids = local.subnets
ami_type = "AL2_x86_64"
ami_type = "AL2023_x86_64_STANDARD"
# instance_types = [var.eks_instance_type]
# disk_size = var.eks_instance_disk_size

Expand Down Expand Up @@ -230,17 +230,17 @@ resource "aws_launch_template" "eks-nodegroup" {
}
}

user_data = base64encode(local.eks-node-private-userdata)
# user_data = base64encode(local.eks-node-private-userdata)
}

#### User data for worker launch

locals {
eks-node-private-userdata = templatefile(
"${path.module}/templates/node-private-userdata.tmpl", {
endpoint = aws_eks_cluster.eks_cluster.endpoint
cluster_ca = aws_eks_cluster.eks_cluster.certificate_authority[0].data
cluster_name = var.cluster_name
}
)
}
# locals {
# eks-node-private-userdata = templatefile(
# "${path.module}/templates/node-private-userdata.tmpl", {
# endpoint = aws_eks_cluster.eks_cluster.endpoint
# cluster_ca = aws_eks_cluster.eks_cluster.certificate_authority[0].data
# cluster_name = var.cluster_name
# }
# )
# }
23 changes: 18 additions & 5 deletions examples/full-cluster-tf-upgrade/README.upgrade-1.32-1.33.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,18 +7,27 @@ update to the AMI (to AL2023) and any charts or image changes.

Copy files from 1.33/{path} to eks-{clustername}/{path}

* versions.tf
* addons/addons.tf
* main.tf
* charts.yml
* images.yml
* addons/addons.yml


## Update Files

Update `cluster_version` from 1.32 to 1.33 in

* settings.auto.tfvars

## Apply changes
Ensure `includes.d/parent_rs.tf` is correct.
If cluster is an upgrade, comment out `ebs-encryption.tf` line 50-58 the resource for `delete_default_sc` as it should already have been applied and will throw an error if not commented out.

Update `main.tf` line 112 -> resource "aws_eks_node_group" "eks-nodegroup" -> change the ami_type to `ami_type = "AL2023_x86_64_STANDARD"`
then on line 251 -> comment out the `user_data` on the resource "aws_launch_template" "eks-nodegroup"

## Apply changes
NOTE: irsa-roles/ does not nav after tf-run, should go back to cluster folder and `tf-run apply 37`
NOTE: common-services/cluster-autoscaler does not nav after tf-run, go back to cluster folder and `tf-run apply 41`
Apply in various directories where changes happened

* (main)
Expand Down Expand Up @@ -46,7 +55,11 @@ There is some approach/process to upgrade the version, find it and put it here.
- addons/addon_cloudwatch.tf and
- common-services/cluster-autoscaler/cluster-autoscaler.tf

- 1.33.0 -- 2026-04-16
- 1.33.0 -- 2026-05-05
- ami updated from `AL2_x86_64` to `AL2023_x86_64_STANDARD`
- user data no longer required
- add files to update
- addons/addons.tf
- main.tf
- charts.yml
- images.yml
- addons/addons.yml

0 comments on commit ec94da0

Please sign in to comment.