Skip to content

Commit

Permalink
aws-parameterization-2 (#447)
Browse files Browse the repository at this point in the history
  • Loading branch information
sekka1 authored Sep 22, 2023
1 parent a885fb0 commit df1abfc
Show file tree
Hide file tree
Showing 4 changed files with 2 additions and 14 deletions.
2 changes: 1 addition & 1 deletion terraform-modules/aws/cluster-autoscaler/main.tf
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
module "iam_assumable_role_admin" {
source = "terraform-aws-modules/iam/aws//modules/iam-assumable-role-with-oidc"
version = var.iam_assumable_role_admin_version
version = "3.6.0"
create_role = true
role_name = "cluster-autoscaler-${var.cluster_name}"
provider_url = replace(var.eks_cluster_oidc_issuer_url, "https://", "")
Expand Down
6 changes: 0 additions & 6 deletions terraform-modules/aws/cluster-autoscaler/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -40,9 +40,3 @@ variable "cluster-autoscaler_helm_version" {
default = "9.24.0"
description = "cluster-autoscaler helm chart version. https://artifacthub.io/packages/helm/cluster-autoscaler/cluster-autoscaler"
}

variable "iam_assumable_role_admin_version" {
type = string
default = "3.6.0"
description = "iam_assumable_role_admin module version"
}
2 changes: 1 addition & 1 deletion terraform-modules/aws/helm/kube-prometheus-stack/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ resource "helm_release" "helm_chart" {
module "iam_assumable_role_grafana" {
count = var.enable_iam_assumable_role_grafana ? 1 : 0
source = "terraform-aws-modules/iam/aws//modules/iam-assumable-role-with-oidc"
version = var.iam_assumable_role_grafana_version
version = "3.6.0"
create_role = true
role_name = local.k8s_service_account_name
provider_url = replace(var.eks_cluster_oidc_issuer_url, "https://", "")
Expand Down
6 changes: 0 additions & 6 deletions terraform-modules/aws/helm/kube-prometheus-stack/variables.tf
Original file line number Diff line number Diff line change
Expand Up @@ -87,9 +87,3 @@ variable "aws_policy_grafana" {
EOF
description = "The AWS policy for the Grafana AWS role. The default is a read only role to all Cloudwatch logs."
}

variable "iam_assumable_role_grafana_version" {
type = string
default = "3.6.0"
description = "iam_assumable_role_grafana module version"
}

0 comments on commit df1abfc

Please sign in to comment.