Alerting fields should be Elastic durations, not Golang durations

Author: tobio

internal/kibana/alerting.go

@@ -71,7 +71,7 @@ func ResourceAlertingRule() *schema.Resource {
 			Description:  "The check interval, which specifies how frequently the rule conditions are checked. The interval must be specified in seconds, minutes, hours or days.",
 			Type:         schema.TypeString,
 			Required:     true,
-			ValidateFunc: utils.StringIsDuration,
+			ValidateFunc: utils.StringIsElasticDuration,
 		},
 		"actions": {
 			Description: "An action that runs under defined conditions.",
@@ -120,7 +120,7 @@ func ResourceAlertingRule() *schema.Resource {
 									Description:  "Defines how often an alert generates repeated actions. This custom action interval must be specified in seconds, minutes, hours, or days. For example, 10m or 1h. This property is applicable only if `notify_when` is `onThrottleInterval`. NOTE: This is a rule level property; if you update the rule in Kibana, it is automatically changed to use action-specific `throttle` values.",
 									Type:         schema.TypeString,
 									Optional:     true,
-									ValidateFunc: utils.StringIsDuration,
+									ValidateFunc: utils.StringIsElasticDuration,
 								},
 							},
 						},
@@ -198,7 +198,7 @@ func ResourceAlertingRule() *schema.Resource {
 			Description:  "Deprecated in 8.13.0. Defines how often an alert generates repeated actions. This custom action interval must be specified in seconds, minutes, hours, or days. For example, 10m or 1h. This property is applicable only if `notify_when` is `onThrottleInterval`. NOTE: This is a rule level property; if you update the rule in Kibana, it is automatically changed to use action-specific `throttle` values.",
 			Type:         schema.TypeString,
 			Optional:     true,
-			ValidateFunc: utils.StringIsDuration,
+			ValidateFunc: utils.StringIsElasticDuration,
 		},
 		"scheduled_task_id": {
 			Description: "ID of the scheduled task that will execute the alert.",

internal/kibana/alerting_test.go

@@ -52,7 +52,7 @@ func TestAccResourceAlertingRule(t *testing.T) {
 					resource.TestCheckResourceAttr("elasticstack_kibana_alerting_rule.test_rule", "consumer", "alerts"),
 					resource.TestCheckResourceAttr("elasticstack_kibana_alerting_rule.test_rule", "notify_when", "onActiveAlert"),
 					resource.TestCheckResourceAttr("elasticstack_kibana_alerting_rule.test_rule", "rule_type_id", ".index-threshold"),
-					resource.TestCheckResourceAttr("elasticstack_kibana_alerting_rule.test_rule", "interval", "10m"),
+					resource.TestCheckResourceAttr("elasticstack_kibana_alerting_rule.test_rule", "interval", "1d"),
 					resource.TestCheckResourceAttr("elasticstack_kibana_alerting_rule.test_rule", "enabled", "false"),
 					resource.TestCheckResourceAttr("elasticstack_kibana_alerting_rule.test_rule", "tags.0", "first"),
 					resource.TestCheckResourceAttr("elasticstack_kibana_alerting_rule.test_rule", "tags.1", "second"),
@@ -244,7 +244,7 @@ resource "elasticstack_kibana_alerting_rule" "test_rule" {
 	termField           = "name"
   })
   rule_type_id = ".index-threshold"
-  interval     = "10m"
+  interval     = "1d"
   enabled      = false
   tags         = ["first", "second"]
 }
@@ -439,7 +439,7 @@ resource "elasticstack_kibana_alerting_rule" "test_rule" {
         days        = [1,2,3]
       timezone    = "Africa/Accra"
       hours_start = "01:00"
-      hours_end   = "07:00" 
+      hours_end   = "07:00"
       }
     }
   }
@@ -514,7 +514,7 @@ resource "elasticstack_kibana_alerting_rule" "test_rule" {
         days        = [7]
       timezone    = "Pacific/Honolulu"
       hours_start = "02:00"
-      hours_end   = "03:00" 
+      hours_end   = "03:00"
       }
     }
   }