storytold/terraform-aws-observability-accelerator
2
0
Fork 0
mirror of https://github.com/storytold/terraform-aws-observability-accelerator.git synced 2026-10-09 00:09:43 +00:00
Code Issues Packages Projects Releases Wiki Activity

Support implicit new AMP creation

Browse Source
This commit is contained in:
Rodrigue Koffi
2022-09-01 15:34:27 +02:00
parent 422a2914e4
commit 9437a97285
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
examples/existing-cluster-with-base-and-infra/main.tf
+5 -3
View File
@@ -28,7 +28,7 @@ terraform {
required_providers { required_providers {
grafana = { grafana = {
source = "grafana/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_endpoint = data.aws_eks_cluster.this.endpoint
eks_cluster_version = data.aws_eks_cluster.this.version eks_cluster_version = data.aws_eks_cluster.this.version
create_new_workspace = var.managed_prometheus_workspace_id == "" ? true : false
tags = { tags = {
Source = "github.com/aws-observability/terraform-aws-observability-accelerator" Source = "github.com/aws-observability/terraform-aws-observability-accelerator"
} }
@@ -61,9 +63,9 @@ module "eks_observability_accelerator" {
enable_cert_manager = true enable_cert_manager = true
# creates a new AMP workspace, defaults to 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_id = var.managed_prometheus_workspace_id
managed_prometheus_workspace_region = null # defaults to the current region, useful for cross region scenarios (same account) managed_prometheus_workspace_region = null # defaults to the current region, useful for cross region scenarios (same account)