feat: Adds support for alertConfiguration.severityOverride attribute

.changelog/3795.txt

@@ -0,0 +1,11 @@
+```release-note:enhancement
+data-source/mongodbatlas_alert_configuration: Adds `severity_override` attribute
+```
+
+```release-note:enhancement
+data-source/mongodbatlas_alert_configurations: Adds `severity_override` attribute
+```
+
+```release-note:enhancement
+resource/mongodbatlas_alert_configuration: Adds `severity_override` attribute
+```

contributing/testing-best-practices.md

@@ -31,6 +31,7 @@
 - `Basic import tests` are done as the last step in the `basic acceptance tests`, not as a different test, e.g. [basicTestCase](https://github.com/mongodb/terraform-provider-mongodbatlas/blob/66c44e62c9afe04ffe8be0dbccaec682bab830e6/internal/service/searchindex/resource_search_index_test.go#L211). Exceptions apply for more specific import tests, e.g. testing with incorrect IDs. [Import tests](https://developer.hashicorp.com/terraform/plugin/sdkv2/resources/import#resource-acceptance-testing-implementation) verify that the [Terraform Import](https://developer.hashicorp.com/terraform/cli/import) functionality is working fine.
 - Data sources are tested in the same tests as the resources, e.g. [commonChecks](https://github.com/mongodb/terraform-provider-mongodbatlas/blob/66c44e62c9afe04ffe8be0dbccaec682bab830e6/internal/service/searchindex/resource_search_index_test.go#L262-L263).
 - Helper functions such as `resource.TestCheckTypeSetElemNestedAttrs` or `resource.TestCheckTypeSetElemAttr` can be used to check resource and data source attributes more easily, e.g. [resource_serverless_instance_test.go](https://github.com/mongodb/terraform-provider-mongodbatlas/blob/66c44e62c9afe04ffe8be0dbccaec682bab830e6/internal/service/serverlessinstance/resource_serverless_instance_test.go#L61).
+- Note: before running the acceptance tests, make sure you set the environment variable `TF_ACC=1`.
 
 ### Cloud Gov tests
 

docs/resources/alert_configuration.md

@@ -150,6 +150,9 @@ resource "mongodbatlas_alert_configuration" "test" {
 
 
   -> **NOTE:** If `event_type` is set to `OUTSIDE_METRIC_THRESHOLD` or `OUTSIDE_SERVERLESS_METRIC_THRESHOLD`, the `metric_threshold_config` field must also be configured.
+* `severity_override` - (Optional) Severity of the event. For the list of accepted values please read the [Create One Alert Configuration in One Project](https://www.mongodb.com/docs/api/doc/atlas-admin-api-v2/operation/operation-creategroupalertconfig) API documentation
+
+  -> **NOTE:** To reset the value of this attribute to default value, you must explicity set it to ERROR (default in MongoDB Atlas)
 
 ### Matchers
 Rules to apply when matching an object against this alert configuration. Only entities that match all these rules are checked for an alert condition. You can filter using the matchers array only when the eventTypeName specifies an event for a host, replica set, or sharded cluster.

internal/service/alertconfiguration/data_source.go

@@ -27,6 +27,7 @@ type TFAlertConfigurationDSModel struct {
 	EventType             types.String                      `tfsdk:"event_type"`
 	Created               types.String                      `tfsdk:"created"`
 	Updated               types.String                      `tfsdk:"updated"`
+	SeverityOverride      types.String                      `tfsdk:"severity_override"`
 	Matcher               []TfMatcherModel                  `tfsdk:"matcher"`
 	MetricThresholdConfig []TfMetricThresholdConfigModel    `tfsdk:"metric_threshold_config"`
 	ThresholdConfig       []TfThresholdConfigModel          `tfsdk:"threshold_config"`
@@ -242,6 +243,9 @@ var alertConfigDSSchemaAttributes = map[string]schema.Attribute{
 			},
 		},
 	},
+	"severity_override": schema.StringAttribute{
+		Optional: true,
+	},
 }
 
 func (d *alertConfigurationDS) Schema(ctx context.Context, req datasource.SchemaRequest, resp *datasource.SchemaResponse) {
@@ -339,6 +343,10 @@ func outputAlertConfigurationResourceHcl(label string, alert *admin.GroupAlertsC
 		appendBlockWithCtyValues(resource, "notification", []string{}, convertNotificationToCtyValues(&notifications[i]))
 	}
 
+	if alert.SeverityOverride != nil {
+		resource.SetAttributeValue("severity_override", cty.StringVal(*alert.SeverityOverride))
+	}
+
 	return string(f.Bytes())
 }
 

internal/service/alertconfiguration/model.go

@@ -107,6 +107,7 @@ func NewTFAlertConfigurationModel(apiRespConfig *admin.GroupAlertsConfig, currSt
 		ThresholdConfig:       NewTFThresholdConfigModel(apiRespConfig.Threshold, currState.ThresholdConfig),
 		Notification:          NewTFNotificationModelList(apiRespConfig.GetNotifications(), currState.Notification),
 		Matcher:               NewTFMatcherModelList(apiRespConfig.GetMatchers(), currState.Matcher),
+		SeverityOverride:      types.StringPointerValue(apiRespConfig.SeverityOverride),
 	}
 }
 
@@ -298,6 +299,7 @@ func NewTfAlertConfigurationDSModel(apiRespConfig *admin.GroupAlertsConfig, proj
 		ThresholdConfig:       NewTFThresholdConfigModel(apiRespConfig.Threshold, []TfThresholdConfigModel{}),
 		Notification:          NewTFNotificationModelList(apiRespConfig.GetNotifications(), []TfNotificationModel{}),
 		Matcher:               NewTFMatcherModelList(apiRespConfig.GetMatchers(), []TfMatcherModel{}),
+		SeverityOverride:      types.StringPointerValue(apiRespConfig.SeverityOverride),
 	}
 }