ensure monitor does not notify when monitor resolves

Author: lalo-galvan

aws/acm/README.md

@@ -31,7 +31,7 @@ No modules.
 | <a name="input_alert_critical_priority"></a> [alert\_critical\_priority](#input\_alert\_critical\_priority) | Priority for alerts within critical threshold (P1-P5, uses monitor defaults if not specified) | `string` | `null` | no |
 | <a name="input_alert_message"></a> [alert\_message](#input\_alert\_message) | Message to prepend to alert notifications | `string` | `"Alert"` | no |
 | <a name="input_alert_nodata_priority"></a> [alert\_nodata\_priority](#input\_alert\_nodata\_priority) | Priority for alerts within warning threshold (P1-P5, uses monitor defaults if not specified) | `string` | `null` | no |
-| <a name="input_base_tags"></a> [base\_tags](#input\_base\_tags) | Base tags (key:value format) to add to this type of check (combined with `local.tags` and `var.additional_tags`, generally you should not change this) | `list(string)` | <pre>[<br/>  "resource:ec2"<br/>]</pre> | no |
+| <a name="input_base_tags"></a> [base\_tags](#input\_base\_tags) | Base tags (key:value format) to add to this type of check (combined with `local.tags` and `var.additional_tags`, generally you should not change this) | `list(string)` | <pre>[<br/>  "resource:acm"<br/>]</pre> | no |
 | <a name="input_certificate_renewal_failure_check_enabled"></a> [certificate\_renewal\_failure\_check\_enabled](#input\_certificate\_renewal\_failure\_check\_enabled) | Whether to enable the certificate renewal failure check | `bool` | `true` | no |
 | <a name="input_cost_center"></a> [cost\_center](#input\_cost\_center) | Cost Center of the monitored resource (leave blank to omit tag) | `string` | `null` | no |
 | <a name="input_dashboard_link"></a> [dashboard\_link](#input\_dashboard\_link) | Dashboard link to include in message | `string` | `null` | no |

aws/acm/variables.tf

@@ -8,7 +8,7 @@ variable "additional_tags" {
 }
 
 variable "base_tags" {
-  default     = ["resource:ec2"]
+  default     = ["resource:acm"]
   description = "Base tags (key:value format) to add to this type of check (combined with `local.tags` and `var.additional_tags`, generally you should not change this)"
   type        = list(string)
 }

common/common.tf

@@ -340,7 +340,6 @@ END
 ${local.alert_context}
 **Alert Information**
 {{#is_alert}} ${local.notify_on_alert} {{/is_alert}}
-{{#is_recovery}} ${local.notify_on_recovery} {{/is_recovery}}
 END
 
   event_alert_base_message = <<END
@@ -369,29 +368,6 @@ Environment: {{env.name}}
 
 Please investigate and take necessary actions.
 {{/is_alert}}
-
-{{#is_recovery}}
-Recovery: Monitor has returned to a normal state.
-
-Current value: {{value}}
-Threshold: {{threshold}}
-
-Environment: {{env.name}}
-
-No further action is required.
-
-  {{#is_match "env.name" "prod" "prd"}}
-    {{#is_match "event.tags.datadog_managed" "critical"}}
-      ${local.notify_on_crit}
-    {{/is_match}}
-    {{#is_match "event.tags.datadog_managed" "true"}}
-      ${local.notify_on_prod}
-    {{/is_match}}
-  {{/is_match}}
-  {{^is_match "env.name" "prod" "prd"}}
-      ${local.notify_on_nonprod}
-  {{/is_match}}
-{{/is_recovery}}
 END
 
   service_group_by = join(",", formatlist("\"%s\"", var.group_by))