fix(DMVP-3291): Deleted thresholds for network in and out alerts

dasmeta · Jan 9, 2024 · d0dd30a · d0dd30a
1 parent 5c3a459
commit d0dd30a
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 4 deletions.
diff --git a/README.md b/README.md
@@ -64,7 +64,7 @@ No requirements.
 
 | Name | Description | Type | Default | Required |
 |------|-------------|------|---------|:--------:|
-| <a name="input_alarms"></a> [alarms](#input\_alarms) | Alarms are enabled by default. You need to set SNS topic name to send alarms. Use custom\_values to customize alarms. | <pre>object({<br>    enabled       = optional(bool, true)<br>    sns_topic     = string<br>    custom_values = optional(any, {})<br>    restarts = optional(object({<br>      enabled = bool<br>      }), {<br>      enabled = true<br>    })<br>    replicas = optional(object({<br>      enabled = bool<br>      }), {<br>      enabled = true<br>    })<br>    network_in = optional(object({<br>      enabled = bool<br>      }), {<br>      enabled = true<br>    })<br>    network_out = optional(object({<br>      enabled = bool<br>      }), {<br>      enabled = true<br>    })<br>    maximum_replicas_usage = optional(object({<br>      enabled          = bool<br>      maximum_replicas = number<br>      }), {<br>      enabled          = true<br>      maximum_replicas = 3 //The count of HPA maximum for a service. It will be used as a threshold for HPA maximum alarm.<br>    })<br><br>  })</pre> | n/a | yes |
+| <a name="input_alarms"></a> [alarms](#input\_alarms) | Alarms are enabled by default. You need to set SNS topic name to send alarms. Use custom\_values to customize alarms. | <pre>object({<br>    enabled       = optional(bool, true)<br>    sns_topic     = string<br>    custom_values = optional(any, {})<br>    restarts = optional(object({<br>      enabled = bool<br>      }), {<br>      enabled = true<br>    })<br>    replicas = optional(object({<br>      enabled = bool<br>      }), {<br>      enabled = true<br>    })<br>    network_in = optional(object({<br>      enabled = bool<br>      }), {<br>      enabled = true<br>    })<br>    network_out = optional(object({<br>      enabled = bool<br>      }), {<br>      enabled = true<br>    })<br>    maximum_replicas_usage = optional(object({<br>      enabled          = optional(bool, true)<br>      maximum_replicas = number<br>      }), {<br>      enabled          = true<br>      maximum_replicas = 3 //The count of HPA maximum for a service. It will be used as a threshold for HPA maximum alarm.<br>    })<br><br>  })</pre> | n/a | yes |
 | <a name="input_cluster_name"></a> [cluster\_name](#input\_cluster\_name) | Cluster name | `string` | n/a | yes |
 | <a name="input_deploy_service"></a> [deploy\_service](#input\_deploy\_service) | Wether to deploy the service via helm or not. | `bool` | `true` | no |
 | <a name="input_helm_values"></a> [helm\_values](#input\_helm\_values) | Values which overwrite chart defaults | `any` | `null` | no |

diff --git a/alarms.tf b/alarms.tf
@@ -51,7 +51,6 @@ module "cw_alerts" {
         },
         period            = try(var.alarms.custom_values.network_in.period, 300),
         statistic         = try(var.alarms.custom_values.network_in.statistic, "avg"),
-        threshold         = try(var.alarms.custom_values.network_in.threshold, 350000)
         equation          = try(var.alarms.custom_values.network_in.equation, "ltlgtu")
         anomaly_detection = true
       },
@@ -68,7 +67,6 @@ module "cw_alerts" {
         },
         period            = try(var.alarms.custom_values.network_out.period, 300),
         statistic         = try(var.alarms.custom_values.network_out.statistic, "avg"),
-        threshold         = try(var.alarms.custom_values.network_out.threshold, 90)
         equation          = try(var.alarms.custom_values.network_out.equation, "ltlgtu")
         anomaly_detection = true
       },

diff --git a/variables.tf b/variables.tf
@@ -46,7 +46,7 @@ variable "alarms" {
       enabled = true
     })
     maximum_replicas_usage = optional(object({
-      enabled          = bool
+      enabled          = optional(bool, true)
       maximum_replicas = number
       }), {
       enabled          = true