actions on/off OK, server state is now computed

Author: Mia-Cross

internal/services/instance/action_server_action_test.go

@@ -2,13 +2,16 @@ package instance_test
 
 import (
 	"errors"
+	"fmt"
 	"regexp"
 	"strings"
 	"testing"
 
 	"github.com/hashicorp/terraform-plugin-testing/helper/resource"
 	"github.com/hashicorp/terraform-plugin-testing/terraform"
+	instanceSDK "github.com/scaleway/scaleway-sdk-go/api/instance/v1"
 	"github.com/scaleway/terraform-provider-scaleway/v2/internal/acctest"
+	"github.com/scaleway/terraform-provider-scaleway/v2/internal/services/instance"
 )
 
 func TestAccActionServer_Basic(t *testing.T) {
@@ -97,6 +100,163 @@ func TestAccActionServer_Basic(t *testing.T) {
 	})
 }
 
+func TestAccActionServer_On_Off(t *testing.T) {
+	if acctest.IsRunningOpenTofu() {
+		t.Skip("Skipping TestAccActionServer_Basic because action are not yet supported on OpenTofu")
+	}
+
+	tt := acctest.NewTestTools(t)
+	defer tt.Cleanup()
+
+	resource.ParallelTest(t, resource.TestCase{
+		ProtoV6ProviderFactories: tt.ProviderFactories,
+		Steps: []resource.TestStep{
+			{
+				Config: fmt.Sprintf(`
+					resource "scaleway_instance_server" "main" {
+						name = "test-terraform-action-server-on-off"
+						type = "DEV1-S"
+						image = "ubuntu_jammy"
+
+					  	lifecycle {
+							action_trigger {
+						  		events  = [after_create]
+						  		actions = [action.scaleway_instance_server_action.stop]
+							}
+					  	}
+					}
+
+					action "scaleway_instance_server_action" "stop" {
+						config {
+						  	action = "%s"
+							server_id = scaleway_instance_server.main.id
+							wait = true
+						}
+					}`, instanceSDK.ServerActionStopInPlace),
+			},
+			{
+				RefreshState: true,
+				Check: resource.ComposeTestCheckFunc(
+					resource.TestCheckResourceAttr("scaleway_instance_server.main", "state", instance.InstanceServerStateStandby),
+					readActualServerState(tt, "scaleway_instance_server.main", instanceSDK.ServerStateStoppedInPlace.String()),
+				),
+			},
+			{
+				Config: fmt.Sprintf(`
+					resource "scaleway_instance_server" "main" {
+						name = "should-be-started"
+						type = "DEV1-S"
+						image = "ubuntu_jammy"
+
+					  	lifecycle {
+							action_trigger {
+						  		events  = [after_create]
+						  		actions = [action.scaleway_instance_server_action.stop]
+							}
+							action_trigger {
+						  		events  = [after_update]
+						  		actions = [action.scaleway_instance_server_action.start]
+							}
+					  	}
+					}
+
+					action "scaleway_instance_server_action" "stop" {
+						config {
+						  	action = "%s"
+							server_id = scaleway_instance_server.main.id
+							wait = true
+						}
+					}
+
+					action "scaleway_instance_server_action" "start" {
+						config {
+						  	action = "%s"
+							server_id = scaleway_instance_server.main.id
+							wait = true
+						}
+					}`, instanceSDK.ServerActionStopInPlace, instanceSDK.ServerActionPoweron),
+			},
+			{
+				RefreshState: true,
+				Check: resource.ComposeTestCheckFunc(
+					resource.TestCheckResourceAttr("scaleway_instance_server.main", "state", instance.InstanceServerStateStarted),
+					readActualServerState(tt, "scaleway_instance_server.main", instanceSDK.ServerStateRunning.String()),
+				),
+			},
+			{
+				Config: fmt.Sprintf(`
+					resource "scaleway_instance_server" "main" {
+						name = "should-be-powered-off"
+						type = "DEV1-S"
+						image = "ubuntu_jammy"
+
+					  	lifecycle {
+							action_trigger {
+						  		events  = [after_create]
+						  		actions = [action.scaleway_instance_server_action.stop]
+							}
+							action_trigger {
+						  		events  = [after_update]
+						  		actions = [action.scaleway_instance_server_action.poweroff]
+							}
+					  	}
+					}
+
+					action "scaleway_instance_server_action" "stop" {
+						config {
+						  	action = "%s"
+							server_id = scaleway_instance_server.main.id
+							wait = true
+						}
+					}
+
+					action "scaleway_instance_server_action" "poweroff" {
+						config {
+						  	action = "%s"
+							server_id = scaleway_instance_server.main.id
+							wait = true
+						}
+					}`, instanceSDK.ServerActionStopInPlace, instanceSDK.ServerActionPoweroff),
+			},
+			{
+				RefreshState: true,
+				Check: resource.ComposeTestCheckFunc(
+					resource.TestCheckResourceAttr("scaleway_instance_server.main", "state", instance.InstanceServerStateStopped),
+					readActualServerState(tt, "scaleway_instance_server.main", instanceSDK.ServerStateStopped.String()),
+				),
+			},
+		},
+	})
+}
+
+func readActualServerState(tt *acctest.TestTools, n string, expectedState string) resource.TestCheckFunc {
+	return func(state *terraform.State) error {
+		rs, ok := state.RootModule().Resources[n]
+		if !ok {
+			return fmt.Errorf("resource not found: %s", n)
+		}
+
+		api, zone, id, err := instance.NewAPIWithZoneAndID(tt.Meta, rs.Primary.ID)
+		if err != nil {
+			return err
+		}
+
+		server, err := api.GetServer(&instanceSDK.GetServerRequest{
+			Zone:     zone,
+			ServerID: id,
+		})
+		if err != nil {
+			return err
+		}
+
+		if server.Server.State.String() != expectedState {
+			return fmt.Errorf("expected server state to be %q, got %q", expectedState, server.Server.State)
+		}
+
+		return nil
+	}
+}
+
 func TestAccActionServer_UnknownVerb(t *testing.T) {
 	if acctest.IsRunningOpenTofu() {
 		t.Skip("Skipping TestAccActionServer_Basic because action are not yet supported on OpenTofu")

internal/services/instance/helpers_instance.go

@@ -135,6 +135,9 @@ func serverStateFlatten(fromState instance.ServerState) (string, error) {
 
 // serverStateExpand converts terraform state to an API state or return an error.
 func serverStateExpand(rawState string) (instance.ServerState, error) {
+	if rawState == "" {
+		return instance.ServerStateRunning, nil
+	}
 	apiState, exist := map[string]instance.ServerState{
 		InstanceServerStateStopped: instance.ServerStateStopped,
 		InstanceServerStateStandby: instance.ServerStateStoppedInPlace,

internal/services/instance/server.go

@@ -240,7 +240,7 @@ func ResourceServer() *schema.Resource {
 			"state": {
 				Type:        schema.TypeString,
 				Optional:    true,
-				Default:     InstanceServerStateStarted,
+				Computed:    true,
 				Description: "The state of the server should be: started, stopped, standby",
 				ValidateFunc: validation.StringInSlice([]string{
 					InstanceServerStateStarted,
@@ -1130,7 +1130,14 @@ func ResourceInstanceServerUpdate(ctx context.Context, d *schema.ResourceData, m
 	if d.HasChange("state") {
 		targetState, err := serverStateExpand(d.Get("state").(string))
 		if err != nil {
+			//server, err = waitForServer(ctx, api.API, zone, id, d.Timeout(schema.TimeoutUpdate))
+			//if err != nil {
+			//	return diag.FromErr(err)
+			//}
+			//targetState, err = serverStateExpand(d.Get("state").(string))
+			//if err != nil {
 			return diag.FromErr(err)
+			//}
 		}
 		// reach expected state
 		err = reachState(ctx, api, zone, id, targetState)