MON-499 Variables for no_data_timeframe removed from monitors which have notify_no_data=false. Notify_no_data enabled on one monitor for SSL (service check).

network/http/ssl/monitors-ssl.tf

@@ -18,7 +18,7 @@ EOQ
 
   new_host_delay      = var.new_host_delay
   no_data_timeframe   = var.invalid_ssl_certificate_no_data_timeframe
-  notify_no_data      = false
+  notify_no_data      = true
   notify_audit        = false
   locked              = false
   timeout_h           = 0

network/tls/README.md

@@ -45,7 +45,6 @@ Creates DataDog monitors with the following checks:
 | invalid\_tls\_certificate\_enabled | Flag to enable TLS certificate expiration monitor | string | `"true"` | no |
 | invalid\_tls\_certificate\_extra\_tags | Extra tags for TLS certificate expiration monitor | list(string) | `[]` | no |
 | invalid\_tls\_certificate\_message | Custom message for TLS certificate expiration monitor | string | `""` | no |
-| invalid\_tls\_certificate\_no\_data\_timeframe | TLS certificate expiration monitor no data timeframe | string | `"10"` | no |
 | invalid\_tls\_certificate\_threshold\_warning | TLS certificate expiration monitor (warning threshold) | string | `"3"` | no |
 | message | Message sent when an alert is triggered | string | n/a | yes |
 | new\_host\_delay | Delay in seconds before monitor new resource | string | `"300"` | no |
@@ -53,7 +52,6 @@ Creates DataDog monitors with the following checks:
 | tls\_certificate\_expiration\_enabled | Flag to enable TLS certificate expiration monitor | string | `"true"` | no |
 | tls\_certificate\_expiration\_extra\_tags | Extra tags for TLS certificate expiration monitor | list(string) | `[]` | no |
 | tls\_certificate\_expiration\_message | Custom message for TLS certificate expiration monitor | string | `""` | no |
-| tls\_certificate\_expiration\_no\_data\_timeframe | TLS certificate expiration monitor no data timeframe | string | `"10"` | no |
 | tls\_certificate\_expiration\_threshold\_warning | TLS certificate expiration monitor (warning threshold) | string | `"5"` | no |
 
 ## Outputs

network/tls/inputs.tf

@@ -93,12 +93,6 @@ variable "invalid_tls_certificate_threshold_warning" {
   default     = 3
 }
 
-variable "invalid_tls_certificate_no_data_timeframe" {
-  description = "TLS certificate expiration monitor no data timeframe"
-  type        = string
-  default     = 10
-}
-
 variable "invalid_tls_certificate_extra_tags" {
   description = "Extra tags for TLS certificate expiration monitor"
   type        = list(string)
@@ -126,12 +120,6 @@ variable "tls_certificate_expiration_threshold_warning" {
   default     = 5
 }
 
-variable "tls_certificate_expiration_no_data_timeframe" {
-  description = "TLS certificate expiration monitor no data timeframe"
-  type        = string
-  default     = 10
-}
-
 variable "tls_certificate_expiration_extra_tags" {
   description = "Extra tags for TLS certificate expiration monitor"
   type        = list(string)

network/tls/monitors-tls.tf

@@ -52,7 +52,6 @@ EOQ
   }
 
   new_host_delay      = var.new_host_delay
-  no_data_timeframe   = var.invalid_tls_certificate_no_data_timeframe
   notify_no_data      = false
   notify_audit        = false
   locked              = false
@@ -87,7 +86,6 @@ EOQ
   }
 
   new_host_delay      = var.new_host_delay
-  no_data_timeframe   = var.tls_certificate_expiration_no_data_timeframe
   notify_no_data      = false
   notify_audit        = false
   locked              = false