Fix dependency issue

examples/existing-cluster-with-base-and-infra/main.tf

@@ -28,7 +28,7 @@ terraform {
   required_providers {
     grafana = {
       source  = "grafana/grafana"
-      version = "1.24.0"
+      version = "1.25.0"
     }
   }
 }
@@ -78,6 +78,17 @@ module "eks_observability_accelerator" {
   tags = local.tags
 }
 
+# https://www.terraform.io/language/modules/develop/providers
+# A module intended to be called by one or more other modules must not contain
+# any provider blocks.
+# This allows forcing depedency between base and workloads module
+provider "grafana" {
+  url = module.eks_observability_accelerator.managed_grafana_workspace_endpoint
+
+  # TODO: manage with Secrets manager
+  auth = var.grafana_api_key
+}
+
 module "workloads_infra" {
   source = "../../workloads/infra"
   # source = "aws-ia/terrarom-aws-observability-accelerator/workloads/infra"
@@ -91,16 +102,10 @@ module "workloads_infra" {
   managed_prometheus_workspace_endpoint = module.eks_observability_accelerator.managed_prometheus_workspace_endpoint
   managed_prometheus_workspace_region   = module.eks_observability_accelerator.managed_prometheus_workspace_region
 
-
-  managed_grafana_workspace_endpoint = module.eks_observability_accelerator.managed_grafana_workspace_endpoint
-
-  # TODO: manage with Secrets manager
-  grafana_api_key = var.grafana_api_key
-
   # module custom configuration, check module documentation
   # config               = {}
 
-  # depends_on = [
-  #   module.eks_observability_accelerator
-  # ]
+  depends_on = [
+    module.eks_observability_accelerator
+  ]
 }

providers.tf

@@ -11,7 +11,7 @@ terraform {
     }
     grafana = {
       source  = "grafana/grafana"
-      version = "1.24.0"
+      version = "1.25.0"
     }
   }
 }

workloads/infra/main.tf

@@ -3,16 +3,11 @@ terraform {
   required_providers {
     grafana = {
       source  = "grafana/grafana"
-      version = "1.24.0"
+      version = "1.25.0"
     }
   }
 }
 
-provider "grafana" {
-  url  = var.managed_grafana_workspace_endpoint
-  auth = var.grafana_api_key
-}
-
 resource "helm_release" "kube_state_metrics" {
   count            = var.config.enable_kube_state_metrics ? 1 : 0
   chart            = var.config.ksm_helm_chart_name

workloads/infra/variables.tf

@@ -3,7 +3,6 @@ variable "eks_cluster_id" {
   type        = string
 }
 
-
 variable "helm_config" {
   description = "Helm Config for Prometheus"
   type        = any
@@ -39,16 +38,6 @@ variable "managed_prometheus_workspace_region" {
   default     = null
 }
 
-variable "managed_grafana_workspace_endpoint" {
-  type    = string
-  default = ""
-}
-
-variable "grafana_api_key" {
-  type    = string
-  default = null
-}
-
 variable "dashboards_folder_id" {
   type = string
 }

workloads/java/main.tf

@@ -1,4 +1,3 @@
-
 locals {
   name      = "adot-collector-java"
   namespace = try(var.helm_config.namespace, local.name)
@@ -8,7 +7,7 @@ terraform {
   required_providers {
     grafana = {
       source  = "grafana/grafana"
-      version = "1.24.0"
+      version = "1.25.0"
     }
   }
 }