Make alerts a list and use a for_each.

This avoids the need to sprinkle `var.alert != ""` everywhere.

Signed-off-by: Matt Moore <[email protected]>

dashboard/sections/alerts/main.tf

@@ -6,7 +6,7 @@ module "width" { source = "../width" }
 
 module "alert" {
   source     = "../../widgets/alert"
-  title      = "Alert"
+  title      = var.title
   alert_name = var.alert
 }
 
@@ -25,8 +25,8 @@ module "collapsible" {
 
   // If no alert is defined, this is an empty collapsed section.
   title     = var.title
-  tiles     = var.alert == "" ? [] : local.tiles
-  collapsed = var.collapsed || var.alert == ""
+  tiles     = local.tiles
+  collapsed = var.collapsed
 }
 
 output "section" {

dashboard/service/README.md

@@ -69,7 +69,7 @@ No requirements.
 
 | Name | Description | Type | Default | Required |
 |------|-------------|------|---------|:--------:|
-| <a name="input_alert"></a> [alert](#input\_alert) | Alerting policies to add to the dashboard. | `string` | `""` | no |
+| <a name="input_alerts"></a> [alerts](#input\_alerts) | Alerting policies to add to the dashboard. | `list(string)` | `[]` | no |
 | <a name="input_labels"></a> [labels](#input\_labels) | Additional labels to apply to the dashboard. | `map` | `{}` | no |
 | <a name="input_service_name"></a> [service\_name](#input\_service\_name) | Name of the service(s) to monitor | `string` | n/a | yes |
 

dashboard/service/dashboard.tf

@@ -17,8 +17,10 @@ module "resources" {
 }
 
 module "alerts" {
+  for_each = toset(var.alerts)
+
   source = "../sections/alerts"
-  alert  = var.alert
+  alert  = ech.key
   title  = "Alert"
 }
 
@@ -27,7 +29,7 @@ module "width" { source = "../sections/width" }
 module "layout" {
   source = "../sections/layout"
   sections = concat(
-    var.alert == "" ? [] : [module.alerts.section],
+    [for x in var.alerts : module.alerts[x].section],
     [
       module.logs.section,
       module.http.section,

dashboard/service/variables.tf

@@ -8,9 +8,9 @@ variable "labels" {
   default     = {}
 }
 
-variable "alert" {
+variable "alerts" {
   description = "Alerting policies to add to the dashboard."
-  type        = string
-  default     = ""
+  type        = list(string)
+  default     = []
 }