fix: Correct alert manager default definition (#4)

Co-authored-by: Bryant Biggs <[email protected]>

Author: Zvikan

.pre-commit-config.yaml

@@ -1,6 +1,6 @@
 repos:
   - repo: https://github.com/antonbabenko/pre-commit-terraform
-    rev: v1.68.1
+    rev: v1.73.0
     hooks:
       - id: terraform_fmt
       - id: terraform_validate
@@ -23,7 +23,7 @@ repos:
           - '--args=--only=terraform_standard_module_structure'
           - '--args=--only=terraform_workspace_remote'
   - repo: https://github.com/pre-commit/pre-commit-hooks
-    rev: v4.2.0
+    rev: v4.3.0
     hooks:
       - id: check-merge-conflict
       - id: end-of-file-fixer

README.md

@@ -81,7 +81,7 @@ No modules.
 
 | Name | Description | Type | Default | Required |
 |------|-------------|------|---------|:--------:|
-| <a name="input_alert_manager_definition"></a> [alert\_manager\_definition](#input\_alert\_manager\_definition) | The alert manager definition that you want to be applied. See more in the [AWS Docs](https://docs.aws.amazon.com/prometheus/latest/userguide/AMP-alert-manager.html) | `string` | `"alertmanager_config: |\nroute:\n  receiver: 'default'\nreceivers:\n  - name: 'default'\n"` | no |
+| <a name="input_alert_manager_definition"></a> [alert\_manager\_definition](#input\_alert\_manager\_definition) | The alert manager definition that you want to be applied. See more in the [AWS Docs](https://docs.aws.amazon.com/prometheus/latest/userguide/AMP-alert-manager.html) | `string` | `"alertmanager_config: |\n  route:\n    receiver: 'default'\n  receivers:\n    - name: 'default'\n"` | no |
 | <a name="input_create"></a> [create](#input\_create) | Determines whether a resources will be created | `bool` | `true` | no |
 | <a name="input_rule_group_namespaces"></a> [rule\_group\_namespaces](#input\_rule\_group\_namespaces) | A map of one or more rule group namespace definitions | `map(any)` | `{}` | no |
 | <a name="input_tags"></a> [tags](#input\_tags) | A map of tags to add to all resources | `map(string)` | `{}` | no |

examples/complete/README.md

@@ -34,6 +34,7 @@ No providers.
 
 | Name | Source | Version |
 |------|--------|---------|
+| <a name="module_default"></a> [default](#module\_default) | ../.. | n/a |
 | <a name="module_disabled"></a> [disabled](#module\_disabled) | ../.. | n/a |
 | <a name="module_prometheus"></a> [prometheus](#module\_prometheus) | ../.. | n/a |
 

examples/complete/main.tf

@@ -11,12 +11,6 @@ locals {
 # Prometheus Module
 ################################################################################
 
-module "disabled" {
-  source = "../.."
-
-  create = false
-}
-
 module "prometheus" {
   source = "../.."
 
@@ -53,3 +47,15 @@ module "prometheus" {
     }
   }
 }
+
+module "disabled" {
+  source = "../.."
+
+  create = false
+}
+
+module "default" {
+  source = "../.."
+
+  workspace_alias = "${local.name}-default"
+}

variables.tf

@@ -27,13 +27,13 @@ variable "workspace_alias" {
 variable "alert_manager_definition" {
   description = "The alert manager definition that you want to be applied. See more in the [AWS Docs](https://docs.aws.amazon.com/prometheus/latest/userguide/AMP-alert-manager.html)"
   type        = string
-  default     = <<-EOF
-	    alertmanager_config: |
-	    route:
-	      receiver: 'default'
-	    receivers:
-	      - name: 'default'
-  EOF
+  default     = <<-EOT
+    alertmanager_config: |
+      route:
+        receiver: 'default'
+      receivers:
+        - name: 'default'
+  EOT
 }
 
 ################################################################################