Merge pull request #60 from dasmeta/DMVP-changes

fix(DMVP-changes): Add slo-sli-sla alerts

README.md

@@ -40,6 +40,7 @@ No providers.
 | Name | Source | Version |
 |------|--------|---------|
 | <a name="module_alerts"></a> [alerts](#module\_alerts) | ./modules/alerts/ | n/a |
+| <a name="module_alerts_slo_sli_sla"></a> [alerts\_slo\_sli\_sla](#module\_alerts\_slo\_sli\_sla) | ./modules/alerts/ | n/a |
 | <a name="module_aws_cloudwatch_log_metric_filter"></a> [aws\_cloudwatch\_log\_metric\_filter](#module\_aws\_cloudwatch\_log\_metric\_filter) | dasmeta/modules/aws//modules/cloudwatch-log-metric | 1.7.0 |
 | <a name="module_eks_monitoring_dashboard"></a> [eks\_monitoring\_dashboard](#module\_eks\_monitoring\_dashboard) | ./modules/dashboard/ | n/a |
 | <a name="module_health-check"></a> [health-check](#module\_health-check) | ./modules/alerts/ | n/a |
@@ -59,6 +60,7 @@ No resources.
 | <a name="input_create_alerts"></a> [create\_alerts](#input\_create\_alerts) | Create Alert | `bool` | `true` | no |
 | <a name="input_eks_monitroing_dashboard"></a> [eks\_monitroing\_dashboard](#input\_eks\_monitroing\_dashboard) | Dashboard for monitoring EKS cluster | `any` | `[]` | no |
 | <a name="input_enable_log_base_metrics"></a> [enable\_log\_base\_metrics](#input\_enable\_log\_base\_metrics) | n/a | `bool` | `true` | no |
+| <a name="input_expression_alert"></a> [expression\_alert](#input\_expression\_alert) | Add multiple metrics in one alert and add expression. | `any` | `{}` | no |
 | <a name="input_health_checks"></a> [health\_checks](#input\_health\_checks) | Health\_checks endpoints and paths | `any` | `[]` | no |
 | <a name="input_log_base_metrics"></a> [log\_base\_metrics](#input\_log\_base\_metrics) | Log Base Metrics | `any` | `[]` | no |
 | <a name="input_name"></a> [name](#input\_name) | Dashboard name | `string` | n/a | yes |

health-checks-and-alerts.tf

@@ -25,3 +25,15 @@ module "alerts" {
   sns_topic = local.sns_topic_name
   alerts    = var.alerts
 }
+
+module "alerts_slo_sli_sla" {
+  source = "./modules/alerts/"
+
+  count = var.expression_alert != {} ? 1 : 0
+
+  sns_topic        = local.sns_topic_name
+  expression_alert = var.expression_alert
+
+  enable_insufficient_data_actions = false
+  enable_ok_actions                = false
+}

vaiables.tf

@@ -44,6 +44,12 @@ variable "alerts" {
   description = "Alerts"
 }
 
+variable "expression_alert" {
+  type        = any
+  default     = {}
+  description = "Add multiple metrics in one alert and add expression."
+}
+
 variable "application_channel_alerts" {
   type        = any
   default     = []