feat(alerts): add notify_on_acknwoledge, deprecated opts at notification channel level (#679)

* add new attribute `notify_on_acknwoledge` on all alert resources, it
should be used if one want to send a notification for an alert that has
been manually acknowledged by a user;
* deprecate `notify_when_ok` and `notify_when_resolved` on notification
channel resources: users should define these behaviors at alert level
now.

Author: dbonf

sysdig/internal/client/v2/model.go

@@ -628,7 +628,7 @@ type NotificationChannelConfigV2 struct {
 }
 
 type NotificationChannelOptionsV2 struct {
-	NotifyOnAcknowledge        bool                          `json:"notifyOnAcknowledge,omitempty"`
+	NotifyOnAcknowledge        *bool                         `json:"notifyOnAcknowledge,omitempty"`
 	NotifyOnResolve            bool                          `json:"notifyOnResolve"`
 	ReNotifyEverySec           *int                          `json:"reNotifyEverySec"`
 	CustomNotificationTemplate *CustomNotificationTemplateV2 `json:"customNotificationTemplate,omitempty"`

sysdig/resource_sysdig_monitor_alert_v2_common.go

@@ -83,6 +83,11 @@ func createAlertV2Schema(original map[string]*schema.Schema) map[string]*schema.
 						Optional: true,
 						Default:  true,
 					},
+					"notify_on_acknowledge": {
+						Type:         schema.TypeString,
+						Optional:     true,
+						ValidateFunc: validation.StringInSlice([]string{"true", "false"}, false),
+					},
 					"main_threshold": {
 						Type:     schema.TypeBool,
 						Optional: true,
@@ -262,6 +267,16 @@ func buildAlertV2CommonStruct(d *schema.ResourceData) *v2.AlertV2Common {
 			}
 
 			newChannel.OverrideOptions.NotifyOnResolve = channelMap["notify_on_resolve"].(bool)
+			if notifyOnAcknowledge, ok := channelMap["notify_on_acknowledge"]; ok && notifyOnAcknowledge.(string) != "" {
+				if notifyOnAcknowledge.(string) == "true" {
+					trueValue := true
+					newChannel.OverrideOptions.NotifyOnAcknowledge = &trueValue
+				} else {
+					falseValue := false
+					newChannel.OverrideOptions.NotifyOnAcknowledge = &falseValue
+				}
+				// else do not set any value for newChannel.OverrideOptions.NotifyOnAcknowledge
+			}
 
 			newChannel.OverrideOptions.Thresholds = []string{}
 			mainThreshold := channelMap["main_threshold"].(bool)
@@ -358,6 +373,14 @@ func updateAlertV2CommonState(d *schema.ResourceData, alert *v2.AlertV2Common) (
 			"notify_on_resolve": ncc.OverrideOptions.NotifyOnResolve,
 		}
 
+		if ncc.OverrideOptions.NotifyOnAcknowledge != nil {
+			if *ncc.OverrideOptions.NotifyOnAcknowledge {
+				config["notify_on_acknowledge"] = "true"
+			} else {
+				config["notify_on_acknowledge"] = "false"
+			}
+		}
+
 		if ncc.OverrideOptions.ReNotifyEverySec != nil {
 			config["renotify_every_minutes"] = secondsToMinutes(*ncc.OverrideOptions.ReNotifyEverySec)
 		} else {

sysdig/resource_sysdig_monitor_notification_channel_common.go

@@ -22,14 +22,16 @@ func createMonitorNotificationChannelSchema(original map[string]*schema.Schema)
 			Default:  false,
 		},
 		"notify_when_ok": {
-			Type:     schema.TypeBool,
-			Optional: true,
-			Default:  false,
+			Type:       schema.TypeBool,
+			Optional:   true,
+			Default:    false,
+			Deprecated: "The notify_when_ok field is deprecated and will be removed in a future version. This flag has been replaced by the `notify_on_resolve` field inside `notification_channels` when defining an alert resource.",
 		},
 		"notify_when_resolved": {
-			Type:     schema.TypeBool,
-			Optional: true,
-			Default:  false,
+			Type:       schema.TypeBool,
+			Optional:   true,
+			Default:    false,
+			Deprecated: "The notify_when_resolved field is deprecated and will be removed in a future version. This flag has been replaced by the `notify_on_acknowledge` field inside `notification_channels` when defining an alert resource.",
 		},
 		"version": {
 			Type:     schema.TypeInt,

sysdig/resource_sysdig_secure_notification_channel_common.go

@@ -22,14 +22,16 @@ func createSecureNotificationChannelSchema(original map[string]*schema.Schema) m
 			Default:  false,
 		},
 		"notify_when_ok": {
-			Type:     schema.TypeBool,
-			Optional: true,
-			Default:  false,
+			Type:       schema.TypeBool,
+			Optional:   true,
+			Default:    false,
+			Deprecated: "The notify_when_ok field is deprecated and will be removed in a future version. This flag has been replaced by the `notify_on_resolve` field inside `notification_channels` when defining an alert resource.",
 		},
 		"notify_when_resolved": {
-			Type:     schema.TypeBool,
-			Optional: true,
-			Default:  false,
+			Type:       schema.TypeBool,
+			Optional:   true,
+			Default:    false,
+			Deprecated: "The notify_when_resolved field is deprecated and will be removed in a future version. This flag has been replaced by the `notify_on_acknowledge` field inside `notification_channels` when defining an alert resource.",
 		},
 		"version": {
 			Type:     schema.TypeInt,

website/docs/r/monitor_alert_v2_change.md

@@ -75,7 +75,8 @@ By defining this field, the user can choose to which notification channels send
 It is a list of objects with the following fields:
 * `id` - (Required) The ID of the notification channel.
 * `renotify_every_minutes` - (Optional) the amount of minutes to wait before re sending the notification to this channel. `0` means no renotification enabled. Default: `0`.
-* `notify_on_resolve` - (Optional) Wether to send a notification when the alert is resolved. Default: `true`.
+* `notify_on_resolve` - (Optional) Whether to send a notification when the alert is resolved. Default: `true`.
+* `notify_on_acknowledge` - (Optional) Whether to send a notification when the alert is acknowledged. If not defined, this option is inherited from the `notify_when_resolved` option from the specific notification channel selected.
 * `main_threshold` - (Optional) Whether this notification channel is used for the main threshold of the alert. Default: `true`.
 * `warning_threshold` - (Optional) Whether this notification channel is used for the warning threshold of the alert. Default: `false`.
 

website/docs/r/monitor_alert_v2_downtime.md

@@ -63,7 +63,8 @@ By defining this field, the user can choose to which notification channels send
 It is a list of objects with the following fields:
 * `id` - (Required) The ID of the notification channel.
 * `renotify_every_minutes` - (Optional) the amount of minutes to wait before re sending the notification to this channel. `0` means no renotification enabled. Default: `0`.
-* `notify_on_resolve` - (Optional) Wether to send a notification when the alert is resolved. Default: `true`.
+* `notify_on_resolve` - (Optional) Whether to send a notification when the alert is resolved. Default: `true`.
+* `notify_on_acknowledge` - (Optional) Whether to send a notification when the alert is acknowledged. If not defined, this option is inherited from the `notify_when_resolved` option from the specific notification channel selected.
 
 ### `custom_notification`
 
@@ -94,7 +95,7 @@ Enables the creation of a capture file of the syscalls during the event.
 * `duration_seconds` - (Optional) Time frame of the capture. Default: `15`.
 * `storage` - (Optional) Custom bucket where to save the capture.
 * `filter` - (Optional) Additional filter to apply to the capture. For example: `proc.name contains nginx`.
-* `enabled` - (Optional) Wether to enable captures. Default: `true`.
+* `enabled` - (Optional) Whether to enable captures. Default: `true`.
 
 ### Downtime alert arguments
 

website/docs/r/monitor_alert_v2_event.md

@@ -73,7 +73,8 @@ By defining this field, the user can choose to which notification channels send
 It is a list of objects with the following fields:
 * `id` - (Required) The ID of the notification channel.
 * `renotify_every_minutes` - (Optional) the amount of minutes to wait before re sending the notification to this channel. `0` means no renotification enabled. Default: `0`.
-* `notify_on_resolve` - (Optional) Wether to send a notification when the alert is resolved. Default: `true`.
+* `notify_on_resolve` - (Optional) Whether to send a notification when the alert is resolved. Default: `true`.
+* `notify_on_acknowledge` - (Optional) Whether to send a notification when the alert is acknowledged. If not defined, this option is inherited from the `notify_when_resolved` option from the specific notification channel selected.
 * `main_threshold` - (Optional) Whether this notification channel is used for the main threshold of the alert. Default: `true`.
 * `warning_threshold` - (Optional) Whether this notification channel is used for the warning threshold of the alert. Default: `false`.
 
@@ -106,7 +107,7 @@ Enables the creation of a capture file of the syscalls during the event.
 * `duration_seconds` - (Optional) Time frame of the capture. Default: `15`.
 * `storage` - (Optional) Custom bucket where to save the capture.
 * `filter` - (Optional) Additional filter to apply to the capture. For example: `proc.name contains nginx`.
-* `enabled` - (Optional) Wether to enable captures. Default: `true`.
+* `enabled` - (Optional) Whether to enable captures. Default: `true`.
 
 ### Event alert arguments
 

website/docs/r/monitor_alert_v2_form_based_prometheus.md

@@ -54,7 +54,8 @@ By defining this field, the user can choose to which notification channels send
 It is a list of objects with the following fields:
 * `id` - (Required) The ID of the notification channel.
 * `renotify_every_minutes` - (Optional) the amount of minutes to wait before re sending the notification to this channel. `0` means no renotification enabled.
-* `notify_on_resolve` - (Optional) Wether to send a notification when the alert is resolved. Default: `true`.
+* `notify_on_resolve` - (Optional) Whether to send a notification when the alert is resolved. Default: `true`.
+* `notify_on_acknowledge` - (Optional) Whether to send a notification when the alert is acknowledged. If not defined, this option is inherited from the `notify_when_resolved` option from the specific notification channel selected.
 * `main_threshold` - (Optional) Whether this notification channel is used for the main threshold of the alert. Default: `true`.
 * `warning_threshold` - (Optional) Whether this notification channel is used for the warning threshold of the alert. Default: `false`.
 

website/docs/r/monitor_alert_v2_group_outlier.md

@@ -75,7 +75,8 @@ By defining this field, the user can choose to which notification channels send
 It is a list of objects with the following fields:
 * `id` - (Required) The ID of the notification channel.
 * `renotify_every_minutes` - (Optional) the amount of minutes to wait before re sending the notification to this channel. `0` means no renotification enabled. Default: `0`.
-* `notify_on_resolve` - (Optional) Wether to send a notification when the alert is resolved. Default: `true`.
+* `notify_on_resolve` - (Optional) Whether to send a notification when the alert is resolved. Default: `true`.
+* `notify_on_acknowledge` - (Optional) Whether to send a notification when the alert is acknowledged. If not defined, this option is inherited from the `notify_when_resolved` option from the specific notification channel selected.
 * `main_threshold` - (Optional) Whether this notification channel is used for the main threshold of the alert. Default: `true`.
 * `warning_threshold` - (Optional) Whether this notification channel is used for the warning threshold of the alert. Default: `false`.
 
@@ -108,7 +109,7 @@ Enables the creation of a capture file of the syscalls during the event.
 * `duration_seconds` - (Optional) Time frame of the capture. Default: `15`.
 * `storage` - (Optional) Custom bucket where to save the capture.
 * `filter` - (Optional) Additional filter to apply to the capture. For example: `proc.name contains nginx`.
-* `enabled` - (Optional) Wether to enable captures. Default: `true`.
+* `enabled` - (Optional) Whether to enable captures. Default: `true`.
 
 ### Group Outlier alert arguments
 

website/docs/r/monitor_alert_v2_metric.md

@@ -90,7 +90,8 @@ By defining this field, the user can choose to which notification channels send
 It is a list of objects with the following fields:
 * `id` - (Required) The ID of the notification channel.
 * `renotify_every_minutes` - (Optional) the amount of minutes to wait before re sending the notification to this channel. `0` means no renotification enabled. Default: `0`.
-* `notify_on_resolve` - (Optional) Wether to send a notification when the alert is resolved. Default: `true`.
+* `notify_on_resolve` - (Optional) Whether to send a notification when the alert is resolved. Default: `true`.
+* `notify_on_acknowledge` - (Optional) Whether to send a notification when the alert is acknowledged. If not defined, this option is inherited from the `notify_when_resolved` option from the specific notification channel selected.
 * `main_threshold` - (Optional) Whether this notification channel is used for the main threshold of the alert. Default: `true`.
 * `warning_threshold` - (Optional) Whether this notification channel is used for the warning threshold of the alert. Default: `false`.
 
@@ -123,7 +124,7 @@ Enables the creation of a capture file of the syscalls during the event.
 * `duration_seconds` - (Optional) Time frame of the capture. Default: `15`.
 * `storage` - (Optional) Custom bucket where to save the capture.
 * `filter` - (Optional) Additional filter to apply to the capture. For example: `proc.name contains nginx`.
-* `enabled` - (Optional) Wether to enable captures. Default: `true`.
+* `enabled` - (Optional) Whether to enable captures. Default: `true`.
 
 ### Metric Threshold alert arguments