feat: add Cockpit TriggerTestAlert action

Author: jremy42

internal/services/cockpit/action_trigger_test_alert_action.go

@@ -19,6 +19,7 @@ var (
 
 type TriggerTestAlertAction struct {
 	regionalAPI *cockpit.RegionalAPI
+	meta        *meta.Meta
 }
 
 func (a *TriggerTestAlertAction) Configure(ctx context.Context, req action.ConfigureRequest, resp *action.ConfigureResponse) {
@@ -38,6 +39,7 @@ func (a *TriggerTestAlertAction) Configure(ctx context.Context, req action.Confi
 
 	client := m.ScwClient()
 	a.regionalAPI = cockpit.NewRegionalAPI(client)
+	a.meta = m
 }
 
 func (a *TriggerTestAlertAction) Metadata(ctx context.Context, req action.MetadataRequest, resp *action.MetadataResponse) {
@@ -61,8 +63,8 @@ func (a *TriggerTestAlertAction) Schema(ctx context.Context, req action.SchemaRe
 				Description: "ID of the Project",
 			},
 			"region": schema.StringAttribute{
-				Required:    true,
-				Description: "Region to target",
+				Optional:    true,
+				Description: "Region to target. If not provided, will use the default region from the provider configuration",
 			},
 		},
 	}
@@ -96,18 +98,27 @@ func (a *TriggerTestAlertAction) Invoke(ctx context.Context, req action.InvokeRe
 		return
 	}
 
-	if data.Region.IsNull() || data.Region.ValueString() == "" {
-		resp.Diagnostics.AddError(
-			"Missing region",
-			"The region attribute is required to trigger a test alert.",
-		)
-
-		return
+	var region scw.Region
+	if !data.Region.IsNull() && data.Region.ValueString() != "" {
+		region = scw.Region(data.Region.ValueString())
+	} else {
+		// Use default region from provider configuration
+		defaultRegion, exists := a.meta.ScwClient().GetDefaultRegion()
+		if !exists {
+			resp.Diagnostics.AddError(
+				"Missing region",
+				"The region attribute is required to trigger a test alert. Please provide it explicitly or configure a default region in the provider.",
+			)
+
+			return
+		}
+
+		region = defaultRegion
 	}
 
 	err := a.regionalAPI.TriggerTestAlert(&cockpit.RegionalAPITriggerTestAlertRequest{
 		ProjectID: data.ProjectID.ValueString(),
-		Region:    scw.Region(data.Region.ValueString()),
+		Region:    region,
 	}, scw.WithContext(ctx))
 
 	if err != nil {
@@ -119,4 +130,3 @@ func (a *TriggerTestAlertAction) Invoke(ctx context.Context, req action.InvokeRe
 		return
 	}
 }
-

internal/services/cockpit/action_trigger_test_alert_action_test.go

@@ -29,19 +29,27 @@ func TestAccActionCockpitTriggerTestAlert_Basic(t *testing.T) {
 
 					resource "scaleway_cockpit_alert_manager" "main" {
 						project_id = scaleway_account_project.project.id
+					}
+
+					resource "scaleway_cockpit_source" "metrics" {
+						project_id     = scaleway_account_project.project.id
+						name           = "test-metrics-source"
+						type           = "metrics"
+						retention_days = 31
 
 						lifecycle {
 							action_trigger {
 								events  = [after_create]
 								actions = [action.scaleway_cockpit_trigger_test_alert_action.main]
 							}
 						}
+
+						depends_on = [scaleway_cockpit_alert_manager.main]
 					}
 
 					action "scaleway_cockpit_trigger_test_alert_action" "main" {
 						config {
 							project_id = scaleway_account_project.project.id
-							region     = scaleway_cockpit_alert_manager.main.region
 						}
 					}
 				`,
@@ -54,19 +62,27 @@ func TestAccActionCockpitTriggerTestAlert_Basic(t *testing.T) {
 
 					resource "scaleway_cockpit_alert_manager" "main" {
 						project_id = scaleway_account_project.project.id
+					}
+
+					resource "scaleway_cockpit_source" "metrics" {
+						project_id     = scaleway_account_project.project.id
+						name           = "test-metrics-source"
+						type           = "metrics"
+						retention_days = 31
 
 						lifecycle {
 							action_trigger {
 								events  = [after_create]
 								actions = [action.scaleway_cockpit_trigger_test_alert_action.main]
 							}
 						}
+
+						depends_on = [scaleway_cockpit_alert_manager.main]
 					}
 
 					action "scaleway_cockpit_trigger_test_alert_action" "main" {
 						config {
 							project_id = scaleway_account_project.project.id
-							region     = scaleway_cockpit_alert_manager.main.region
 						}
 					}
 
@@ -100,4 +116,3 @@ func TestAccActionCockpitTriggerTestAlert_Basic(t *testing.T) {
 		},
 	})
 }
-

provider/framework.go

@@ -140,8 +140,6 @@ func (p *ScalewayProvider) Actions(_ context.Context) []func() action.Action {
 
 	res = append(res, instance.NewServerAction)
 	res = append(res, cockpit.NewTriggerTestAlertAction)
-	res = append(res, cockpit.NewResetGrafanaUserPasswordAction)
-	res = append(res, cockpit.NewSyncGrafanaDataSourcesAction)
 
 	return res
 }