refactor(monitor_alert_v2): remove deprecated trigger_after_minutes field

Author: tembleking

sysdig/resource_sysdig_monitor_alert_v2_metric.go

@@ -33,18 +33,10 @@ func resourceSysdigMonitorAlertV2Metric() *schema.Resource {
 		},
 
 		Schema: createScopedSegmentedAlertV2Schema(createAlertV2Schema(map[string]*schema.Schema{
-			"trigger_after_minutes": {
-				Type:         schema.TypeInt,
-				Optional:     true,
-				Computed:     true, // computed if range_seconds is defined
-				Deprecated:   "Use range_seconds instead",
-				ValidateFunc: validation.IntAtLeast(1),
-			},
 			"range_seconds": {
 				Type:         schema.TypeInt,
 				Optional:     true,
 				Computed:     true, // computed if trigger_after_minutes is defined
-				ExactlyOneOf: []string{"trigger_after_minutes"},
 				ValidateFunc: validation.IntAtLeast(60),
 			},
 			"duration_seconds": {
@@ -236,15 +228,6 @@ func buildAlertV2MetricStruct(d *schema.ResourceData) (*v2.AlertV2Metric, error)
 		config.Range = d.Get("range_seconds").(int)
 	}
 
-	if d.HasChange("trigger_after_minutes") {
-		// GetOk returns true even if the value is stored only in the state and not in the user config:
-		// to avoid applying a trigger_after_minutes old value from the state even if the user removed it from the config
-		// we use HasChange that is true only if the user has changed (or created) it - and so it must be in the config
-		if attr, ok := d.GetOk("trigger_after_minutes"); ok && attr != nil {
-			config.Range = minutesToSeconds(d.Get("trigger_after_minutes").(int))
-		}
-	}
-
 	if attr, ok := d.GetOk("duration_seconds"); ok && attr != nil {
 		config.Duration = d.Get("duration_seconds").(int)
 	}
@@ -276,7 +259,6 @@ func updateAlertV2MetricState(d *schema.ResourceData, alert *v2.AlertV2Metric) e
 		return err
 	}
 
-	_ = d.Set("trigger_after_minutes", secondsToMinutes(alert.Config.Range))
 	_ = d.Set("range_seconds", alert.Config.Range)
 	_ = d.Set("duration_seconds", alert.Config.Duration)
 

website/docs/r/monitor_alert_v2_metric.md

@@ -73,8 +73,7 @@ These arguments are common to all alerts in Sysdig Monitor.
 
 * `name` - (Required) The name of the alert rule. It must be unique.
 * `description` - (Optional) The description of Monitor alert.
-* `range_seconds` - (Optional, Required if `trigger_after_minutes` is not defined): The rolling time aggregation period in which the relevant metric data is evaluated.
-* `trigger_after_minutes` - (Optional, Deprecated) The rolling time aggregation period in which the relevant metric data is evaluated. Deprecated: use `range_seconds` instead.
+* `range_seconds` - (Optional): The rolling time aggregation period in which the relevant metric data is evaluated.
 * `duration_seconds` - (Optional) A time interval that defines for how long a condition should be met before an alert occurrence is created.
 * `group` - (Optional) Used to group alert rules in the UI. This value must be a lowercase string.
 * `severity` - (Optional) Severity of the Monitor alert. It must be `high`, `medium`, `low` or `info`. Default: `low`.