fix: added depends on for external secrets and loki, update helm char…

…t version for external secrets
maddevsio · Nov 25, 2024 · 33dadd0 · 33dadd0
1 parent f56eb5a
commit 33dadd0
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/terraform/modules/k8s-addons/eks-external-secrets.tf b/terraform/modules/k8s-addons/eks-external-secrets.tf
@@ -154,4 +154,5 @@ resource "helm_release" "external_secrets" {
     local.external_secrets_values
   ]
 
+  depends_on = [helm_release.aws_loadbalancer_controller]
 }
diff --git a/terraform/modules/k8s-addons/eks-loki-stack.tf b/terraform/modules/k8s-addons/eks-loki-stack.tf
@@ -180,5 +180,8 @@ resource "helm_release" "loki_stack" {
     local.loki_stack_values
   ]
 
-  depends_on = [kubectl_manifest.kube_prometheus_stack_operator_crds]
+  depends_on = [
+    kubectl_manifest.kube_prometheus_stack_operator_crds,
+    helm_release.aws_loadbalancer_controller
+  ]
 }
diff --git a/terraform/modules/k8s-addons/helm-releases.yaml b/terraform/modules/k8s-addons/helm-releases.yaml
@@ -39,7 +39,7 @@ releases:
     enabled: true
     chart: external-secrets
     repository: https://charts.external-secrets.io
-    chart_version: 0.10.0
+    chart_version: 0.10.4
     namespace: external-secrets
   - id: gitlab-runner
     enabled: false
-Original file line number
+Diff line change
@@ Expand Up / @@ -154,4 +154,5 @@ resource "helm_release" "external_secrets" { @@
         local.external_secrets_values
       ]
+      depends_on = [helm_release.aws_loadbalancer_controller]
     }