Skip to content

Commit

Permalink
create 1.35 upgrade from 1.34, recreate 1.34 so tags changes are on 1.35
Browse files Browse the repository at this point in the history
  • Loading branch information
morga471 committed Jun 22, 2026
1 parent 8678692 commit 168444f
Show file tree
Hide file tree
Showing 295 changed files with 12,725 additions and 1,135 deletions.
882 changes: 4 additions & 878 deletions examples/full-cluster-tf-upgrade/1.33-1.34.diffs

Large diffs are not rendered by default.

889 changes: 889 additions & 0 deletions examples/full-cluster-tf-upgrade/1.34-1.35.diffs

Large diffs are not rendered by default.

6 changes: 1 addition & 5 deletions examples/full-cluster-tf-upgrade/1.34/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ Here is a sample file:

```hcl
cluster_name = "org-project-env"
cluster_version = "1.34"
cluster_version = "1.33"
region = "us-gov-east-1"
domain = "org-project-env.env.domain.census.gov"
contact_email = "org-project-env-group-mailing-list@census.gov"
Expand Down Expand Up @@ -470,7 +470,3 @@ clusterrolebinding.rbac.authorization.k8s.io/eks-console-dashboard-full-access-b
- 2.7.0 -- 2026-04-16
- update to 1.33
- only addons updated from 1.32. needs more changes to make it work

- 2.8.0 -- 2026-04-16
- update to 1.34
- only addons updated from 1.32. needs more changes to make it work
12 changes: 4 additions & 8 deletions examples/full-cluster-tf-upgrade/1.34/addons/addon_cloudwatch.tf
Original file line number Diff line number Diff line change
Expand Up @@ -43,15 +43,13 @@ module "role_cloudwatch-observability" {
}

tags = merge(
{
"eks:namespace" = local.cloudwatch_observability_namespace
"eks:user" = local.cloudwatch_observability_name
},
local.base_tags,
local.common_tags,
module.tags.finops_roles["iam"],
module.tags.tags,
var.application_tags,
{
"eks:namespace" = local.cloudwatch_observability_namespace
"eks:user" = local.cloudwatch_observability_name
}
)
}

Expand All @@ -63,8 +61,6 @@ resource "aws_cloudwatch_log_group" "cloudwatch-observability" {
tags = merge(
local.base_tags,
local.common_tags,
module.tags.finops_roles["cloudwatch"],
module.tags.tags,
var.application_tags,
)
}
Original file line number Diff line number Diff line change
Expand Up @@ -97,12 +97,10 @@ module "role_ebs-driver" {
attached_policies = [data.aws_iam_policy.ebs-provisioner.arn]

tags = merge(
{ "Name" = format("%v%v-ebs-driver", local._prefixes["eks-role"], var.cluster_name) },
local.base_tags,
local.common_tags,
module.tags.finops_roles["iam"],
module.tags.tags,
var.application_tags,
{ "Name" = format("%v%v-ebs-driver", local._prefixes["eks-role"], var.cluster_name) },
)
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -70,10 +70,8 @@ module "role_vpc-cni" {
tags = merge(
local.base_tags,
local.common_tags,
module.tags.finops_roles["iam"],
module.tags.tags,
tomap({ "Name" = format("%v%v-vpc-cni", local._prefixes["eks-role"], var.cluster_name) }),
var.application_tags,
tomap({ "Name" = format("%v%v-ebs-driver", local._prefixes["eks-role"], var.cluster_name) }),
)
}

Expand Down
2 changes: 0 additions & 2 deletions examples/full-cluster-tf-upgrade/1.34/addons/tf-run.data
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,6 @@ LINK includes.d/kubeconfig.eks-subdirectory.tf
LINK variables.eks.tf
LINK prefixes.tf
LINK providers.tf
LINK tags.tf
LINK tags.yml
LINK variables.addons.tf
LINK versions.tf
LINK version.tf
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,6 @@ module "role_dba_administrator" {
tags = merge(
local.base_tags,
local.common_tags,
module.tags.finops_roles["iam"],
module.tags.tags,
var.application_tags,
)
}
Expand Down Expand Up @@ -104,8 +102,6 @@ module "group_dba_administrator" {
tags = merge(
local.base_tags,
local.common_tags,
module.tags.finops_roles["iam"],
module.tags.tags,
var.application_tags,
)
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -28,8 +28,6 @@ module "service_cicd_deployer" {
tags = merge(
local.base_tags,
local.common_tags,
module.tags.finops_roles["iam"],
module.tags.tags,
var.application_tags,
)
}
Expand All @@ -46,8 +44,6 @@ module "role_cicd_deployer" {
tags = merge(
local.base_tags,
local.common_tags,
module.tags.finops_roles["iam"],
module.tags.tags,
var.application_tags,
)
}
Expand Down Expand Up @@ -156,8 +152,6 @@ module "group_cicd_deployer" {
tags = merge(
local.base_tags,
local.common_tags,
module.tags.finops_roles["iam"],
module.tags.tags,
var.application_tags,
)
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,6 @@ LINKTOP init
LINKTOP provider_configs.d/provider.ldap_new.auto.tfvars
LINKTOP provider_configs.d/provider.ldap_new.tf
LINKTOP provider_configs.d/provider.ldap_new.variables.tf
LINK tags.tf
LINK tags.yml
LINK versions.tf
LINK settings.auto.tfvars
LINK variables.application_tags.auto.tfvars
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,10 +8,8 @@ module "subordinate_ca" {
tags = merge(
local.base_tags,
local.common_tags,
module.tags.finops_roles["acm_pca"],
module.tags.tags,
var.account_tags,
var.application_tags,
var.infrastructure_tags,
var.application_tags,
)
}
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ module "role_cluster-autoscaler" {
source = "terraform-aws-modules/iam/aws//modules/iam-role-for-service-accounts"

description = "EKS IAM Role for ${var.cluster_name} for service account ${var.cluster_autoscaler_namespace}:${var.cluster_autoscaler_name}"
name = format("%v%v-irsa__%v", local._prefixes["eks-role"], var.cluster_name, "as")
name = format("%v%v-irsa__%v", local._prefixes["eks-role"], var.cluster_name, "autoscaler")

attach_cluster_autoscaler_policy = true
cluster_autoscaler_cluster_names = [var.cluster_name]
Expand All @@ -20,8 +20,6 @@ module "role_cluster-autoscaler" {
tags = merge(
local.base_tags,
local.common_tags,
module.tags.finops_roles["iam"],
module.tags.tags,
var.application_tags,
{
"eks:namespace" = var.cluster_autoscaler_namespace
Expand Down Expand Up @@ -51,8 +49,6 @@ module "images_cluster_autoscaler" {
tags = merge(
local.base_tags,
local.common_tags,
module.tags.finops_roles["ecr"],
module.tags.tags,
var.application_tags,
)
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,8 +23,6 @@ LINK includes.d/kubeconfig.eks-subdirectory.tf
LINK variables.eks.tf
LINK prefixes.tf
LINK providers.tf
LINK tags.tf
LINK tags.yml
LINK variables.addons.tf
LINK versions.tf
LINK version.tf
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,8 +22,6 @@ module "images" {
tags = merge(
local.base_tags,
local.common_tags,
module.tags.finops_roles["ecr"],
module.tags.tags,
var.application_tags,
)

Expand Down
28 changes: 6 additions & 22 deletions examples/full-cluster-tf-upgrade/1.34/dns-zone.tf
Original file line number Diff line number Diff line change
Expand Up @@ -57,13 +57,11 @@ resource "aws_route53_zone" "cluster_domain" {
}

tags = merge(
{ "Name" = local.cluster_domain_name },
local.base_tags,
local.common_tags,
module.tags.finops_roles["route53"],
module.tags.tags,
var.application_tags,
var.tags,
var.application_tags,
{ "Name" = local.cluster_domain_name },
)
}

Expand All @@ -83,10 +81,7 @@ module "route53_cluster_domain_east" {
zone_ids = [aws_route53_zone.cluster_domain.zone_id]

tags = merge(
local.base_tags,
local.common_tags,
module.tags.finops_roles["route53"],
module.tags.tags,
var.application_tags,
)
}
Expand All @@ -104,10 +99,7 @@ module "route53_cluster_domain_west" {
zone_ids = [aws_route53_zone.cluster_domain.zone_id]

tags = merge(
local.base_tags,
local.common_tags,
module.tags.finops_roles["route53"],
module.tags.tags,
var.application_tags,
)
}
Expand Down Expand Up @@ -175,10 +167,8 @@ module "route53_main_east" {
zone_ids = [aws_route53_zone.cluster_domain.zone_id]

tags = merge(
module.tags.finops_roles["route53"],
module.tags.tags,
var.application_tags,
local.common_tags,
var.application_tags,
)
}

Expand All @@ -195,10 +185,8 @@ module "route53_main_west" {
zone_ids = [aws_route53_zone.cluster_domain.zone_id]

tags = merge(
module.tags.finops_roles["route53"],
module.tags.tags,
var.application_tags,
local.common_tags,
var.application_tags,
)
}

Expand Down Expand Up @@ -228,10 +216,8 @@ module "route53_main_legacy_east" {
zone_ids = [aws_route53_zone.cluster_domain.zone_id]

tags = merge(
module.tags.finops_roles["route53"],
module.tags.tags,
var.application_tags,
local.common_tags,
var.application_tags,
)
}

Expand All @@ -248,10 +234,8 @@ module "route53_main_legacy_west" {
zone_ids = [aws_route53_zone.cluster_domain.zone_id]

tags = merge(
module.tags.finops_roles["route53"],
module.tags.tags,
var.application_tags,
local.common_tags,
var.application_tags,
)
}

22 changes: 6 additions & 16 deletions examples/full-cluster-tf-upgrade/1.34/dns-zone.tf.dmz
Original file line number Diff line number Diff line change
Expand Up @@ -57,13 +57,11 @@ resource "aws_route53_zone" "cluster_domain" {
}

tags = merge(
{ "Name" = local.cluster_domain_name },
local.base_tags,
local.common_tags,
module.tags.finops_roles["route53"],
module.tags.tags,
var.application_tags,
var.tags,
var.application_tags,
{ "Name" = local.cluster_domain_name },
)
}

Expand All @@ -83,10 +81,8 @@ module "route53_cluster_domain_east" {
zone_ids = [aws_route53_zone.cluster_domain.zone_id]

tags = merge(
module.tags.tags,
module.tags.finops_roles["route53"],
var.application_tags,
local.common_tags,
var.application_tags,
)
}

Expand All @@ -103,10 +99,8 @@ module "route53_cluster_domain_west" {
zone_ids = [aws_route53_zone.cluster_domain.zone_id]

tags = merge(
module.tags.tags,
module.tags.finops_roles["route53"],
var.application_tags,
local.common_tags,
var.application_tags,
)
}

Expand Down Expand Up @@ -162,10 +156,8 @@ module "route53_main_east" {
zone_ids = [aws_route53_zone.cluster_domain.zone_id]

tags = merge(
module.tags.tags,
module.tags.finops_roles["route53"],
var.application_tags,
local.common_tags,
var.application_tags,
)
}

Expand All @@ -181,9 +173,7 @@ module "route53_main_west" {
zone_ids = [aws_route53_zone.cluster_domain.zone_id]

tags = merge(
module.tags.tags,
module.tags.finops_roles["route53"],
var.application_tags,
local.common_tags,
var.application_tags,
)
}
Loading

0 comments on commit 168444f

Please sign in to comment.