fix: change alarm_service_resources_enabled default (#83)

Signed-off-by: Bünyamin <[email protected]>

README.md

@@ -69,7 +69,7 @@ Terraform module which creates a ecs app
 | <a name="input_alarm_scheduled"></a> [alarm\_scheduled](#input\_alarm\_scheduled) | This can be used to override scheduled alarm | <pre>object({<br>    alarm_description   = optional(string)<br>    datapoints_to_alarm = optional(number, 1)<br>    evaluation_periods  = optional(number, 1)<br>    period              = optional(number, 60)<br>    threshold           = optional(number, 0)<br>  })</pre> | `{}` | no |
 | <a name="input_alarm_service_resources_cpu_average"></a> [alarm\_service\_resources\_cpu\_average](#input\_alarm\_service\_resources\_cpu\_average) | Average CPUUtilization alarm specs | <pre>object({<br>    datapoints_to_alarm = optional(number, 6)<br>    evaluation_periods  = optional(number, 6)<br>    period              = optional(number, 300)<br>    threshold           = optional(number, 125)<br>  })</pre> | <pre>{<br>  "datapoints_to_alarm": 6,<br>  "evaluation_periods": 6,<br>  "period": 300,<br>  "threshold": 125<br>}</pre> | no |
 | <a name="input_alarm_service_resources_cpu_maximum"></a> [alarm\_service\_resources\_cpu\_maximum](#input\_alarm\_service\_resources\_cpu\_maximum) | Maximum CPUUtilization alarm specs | <pre>object({<br>    datapoints_to_alarm = optional(number, 15)<br>    evaluation_periods  = optional(number, 15)<br>    period              = optional(number, 60)<br>    threshold           = optional(number, 150)<br>  })</pre> | <pre>{<br>  "datapoints_to_alarm": 15,<br>  "evaluation_periods": 15,<br>  "period": 60,<br>  "threshold": 150<br>}</pre> | no |
-| <a name="input_alarm_service_resources_enabled"></a> [alarm\_service\_resources\_enabled](#input\_alarm\_service\_resources\_enabled) | Defines if resource alarms should be created | `bool` | `null` | no |
+| <a name="input_alarm_service_resources_enabled"></a> [alarm\_service\_resources\_enabled](#input\_alarm\_service\_resources\_enabled) | Defines if resource alarms should be created | `bool` | `false` | no |
 | <a name="input_alarm_service_resources_memory_average"></a> [alarm\_service\_resources\_memory\_average](#input\_alarm\_service\_resources\_memory\_average) | Average MemoryUtilization alarm specs | <pre>object({<br>    datapoints_to_alarm = optional(number, 6)<br>    evaluation_periods  = optional(number, 6)<br>    period              = optional(number, 300)<br>    threshold           = optional(number, 125)<br>  })</pre> | <pre>{<br>  "datapoints_to_alarm": 6,<br>  "evaluation_periods": 6,<br>  "period": 300,<br>  "threshold": 125<br>}</pre> | no |
 | <a name="input_alarm_service_resources_memory_maximum"></a> [alarm\_service\_resources\_memory\_maximum](#input\_alarm\_service\_resources\_memory\_maximum) | Maximum MemoryUtilization alarm specs | <pre>object({<br>    datapoints_to_alarm = optional(number, 15)<br>    evaluation_periods  = optional(number, 15)<br>    period              = optional(number, 60)<br>    threshold           = optional(number, 150)<br>  })</pre> | <pre>{<br>  "datapoints_to_alarm": 15,<br>  "evaluation_periods": 15,<br>  "period": 60,<br>  "threshold": 150<br>}</pre> | no |
 | <a name="input_alarm_service_resources_treat_missing_data"></a> [alarm\_service\_resources\_treat\_missing\_data](#input\_alarm\_service\_resources\_treat\_missing\_data) | How to treat missing data, defaults to 'breaching' | `string` | `"breaching"` | no |

variables.tf

@@ -43,7 +43,7 @@ variable "alarm_kinsumer" {
 
 variable "alarm_service_resources_enabled" {
   type        = bool
-  default     = null
+  default     = false
   description = "Defines if resource alarms should be created"
 }