From 9437a97285d26d7422c94c0c393ab43204af4269 Mon Sep 17 00:00:00 2001 From: Rodrigue Koffi Date: Thu, 1 Sep 2022 15:34:27 +0200 Subject: [PATCH] Support implicit new AMP creation --- examples/existing-cluster-with-base-and-infra/main.tf | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/examples/existing-cluster-with-base-and-infra/main.tf b/examples/existing-cluster-with-base-and-infra/main.tf index 741dad2..81232a1 100644 --- a/examples/existing-cluster-with-base-and-infra/main.tf +++ b/examples/existing-cluster-with-base-and-infra/main.tf @@ -28,7 +28,7 @@ terraform { required_providers { grafana = { source = "grafana/grafana" - version = "1.25.0" + version = ">= 1.25.0" } } } @@ -41,6 +41,8 @@ locals { eks_cluster_endpoint = data.aws_eks_cluster.this.endpoint eks_cluster_version = data.aws_eks_cluster.this.version + create_new_workspace = var.managed_prometheus_workspace_id == "" ? true : false + tags = { Source = "github.com/aws-observability/terraform-aws-observability-accelerator" } @@ -61,9 +63,9 @@ module "eks_observability_accelerator" { enable_cert_manager = true # creates a new AMP workspace, defaults to true - enable_managed_prometheus = false + enable_managed_prometheus = local.create_new_workspace - # reusing existing AMP -- needs data source for alerting rules + # reusing existing AMP if specified managed_prometheus_workspace_id = var.managed_prometheus_workspace_id managed_prometheus_workspace_region = null # defaults to the current region, useful for cross region scenarios (same account)