feat(resources): implement PostgresFlexInstance list + delete

List via postgresflex/v2api ListInstances; Remove soft-deletes active
instances and force-deletes ones already in Deleted state so nuke runs
converge past the retention window.

Author: qaiser42

README.md

@@ -37,15 +37,15 @@ Legend: ✅ list + delete via real STACKIT SDK · 🟡 registered, lister return
 | Object Storage | `ObjectStorageBucket` | 🟡 | `services/objectstorage` |
 | Object Storage | `ObjectStorageObject` | 🟡 | `services/objectstorage` |
 | SKE | `SKECluster` | 🟡 | `services/ske` |
-| PostgresFlex | `PostgresFlexInstance` | 🟡 | `services/postgresflex` |
+| PostgresFlex | `PostgresFlexInstance` | ✅ | `services/postgresflex/v2api` |
 | MongoDBFlex | `MongoDBFlexInstance` | 🟡 | `services/mongodbflex` |
 | Redis | `RedisInstance` | 🟡 | `services/redis` |
 | OpenSearch | `OpenSearchInstance` | 🟡 | `services/opensearch` |
 | RabbitMQ | `RabbitMQInstance` | 🟡 | `services/rabbitmq` |
 | LoadBalancer | `LoadBalancer` | 🟡 | `services/loadbalancer` |
 | DNS | `DNSZone` | 🟡 | `services/dns` |
 
-**5 of 20 resources fully working.** The CLI / config / auth / libnuke engine are functional; the per-resource SDK wiring lands incrementally. Pick one above and follow [`resources/compute-server.go`](resources/compute-server.go) as the reference pattern — see [Contributing](docs/contributing.md).
+**6 of 20 resources fully working.** The CLI / config / auth / libnuke engine are functional; the per-resource SDK wiring lands incrementally. Pick one above and follow [`resources/compute-server.go`](resources/compute-server.go) as the reference pattern — see [Contributing](docs/contributing.md).
 
 ## Features
 

go.mod

@@ -7,6 +7,7 @@ require (
 	github.com/sirupsen/logrus v1.9.4
 	github.com/stackitcloud/stackit-sdk-go/core v0.26.0
 	github.com/stackitcloud/stackit-sdk-go/services/iaas v1.12.2
+	github.com/stackitcloud/stackit-sdk-go/services/postgresflex v1.11.0
 	github.com/urfave/cli/v3 v3.10.1
 	gopkg.in/yaml.v3 v3.0.1
 )

go.sum

@@ -31,16 +31,12 @@ github.com/stackitcloud/stackit-sdk-go/core v0.26.0 h1:jQEb9gkehfp6VCP6TcYk7BI10
 github.com/stackitcloud/stackit-sdk-go/core v0.26.0/go.mod h1:WU1hhxnjXw2EV7CYa1nlEvNpMiRY6CvmIOaHuL3pOaA=
 github.com/stackitcloud/stackit-sdk-go/services/iaas v1.12.2 h1:hWtbUy0UOhw1cE1riCNjvtmbF3zDl717DIcpEEMF9ps=
 github.com/stackitcloud/stackit-sdk-go/services/iaas v1.12.2/go.mod h1:AbPN9BGkdjc+tVsXEX9Vr8BPDjdlDmG26K1FwCKZQVU=
+github.com/stackitcloud/stackit-sdk-go/services/postgresflex v1.11.0 h1:cuI4NhuFhaZ3tTkBpUM7nt2odKFJkyCcphT/3gGb9CE=
+github.com/stackitcloud/stackit-sdk-go/services/postgresflex v1.11.0/go.mod h1:yzlakB+f8ur4yAHR6lyCABO+HcEtZG3G2Faj6m5/uW8=
 github.com/stevenle/topsort v0.2.0 h1:LLWgtp34HPX6/RBDRS0kElVxGOTzGBLI1lSAa5Lb46k=
 github.com/stevenle/topsort v0.2.0/go.mod h1:ck2WG2/ZrOr6dLApQ/5Xrqy5wv3T0qhKYWE7r9tkibc=
 github.com/stretchr/testify v1.11.1 h1:7s2iGBzp5EwR7/aIZr8ao5+dra3wiQyKjjFuvgVKu7U=
 github.com/stretchr/testify v1.11.1/go.mod h1:wZwfW3scLgRK+23gO65QZefKpKQRnfz6sD981Nm4B6U=
-github.com/urfave/cli/v3 v3.9.0 h1:AV9lIiPv3ukYnxunaCUsHnEozptYmDN2F0+yWqLMn/c=
-github.com/urfave/cli/v3 v3.9.0/go.mod h1:ysVLtOEmg2tOy6PknnYVhDoouyC/6N42TMeoMzskhso=
-github.com/urfave/cli/v3 v3.9.1 h1:OLU13atWZ0M+a4xmyBuBNOLZsSRYXyPeMeNjOvgYP54=
-github.com/urfave/cli/v3 v3.9.1/go.mod h1:ysVLtOEmg2tOy6PknnYVhDoouyC/6N42TMeoMzskhso=
-github.com/urfave/cli/v3 v3.10.0 h1:0aU8yOObVDMkM13Cj4G+zb4P0PdeJMec65f81Ak1ioM=
-github.com/urfave/cli/v3 v3.10.0/go.mod h1:ysVLtOEmg2tOy6PknnYVhDoouyC/6N42TMeoMzskhso=
 github.com/urfave/cli/v3 v3.10.1 h1:7Kx9H50hrHbRbyxgO1KP6/BcbiGRz0uYh5YyQ30JEEY=
 github.com/urfave/cli/v3 v3.10.1/go.mod h1:ysVLtOEmg2tOy6PknnYVhDoouyC/6N42TMeoMzskhso=
 golang.org/x/sync v0.14.0 h1:woo0S4Yywslg6hp4eUFjTVOyKt0RookbpAHG4c1HmhQ=

resources/postgresflex-instance.go

@@ -4,10 +4,14 @@ import (
 	"context"
 	"fmt"
 
+	"github.com/sirupsen/logrus"
+
 	"github.com/ekristen/libnuke/pkg/registry"
 	"github.com/ekristen/libnuke/pkg/resource"
 	"github.com/ekristen/libnuke/pkg/types"
 
+	postgresflexv2 "github.com/stackitcloud/stackit-sdk-go/services/postgresflex/v2api"
+
 	"github.com/qaiser42/stackit-nuke/pkg/stackit"
 )
 
@@ -22,20 +26,89 @@ func init() {
 	})
 }
 
+// PostgresFlexInstance is a STACKIT PostgreSQL Flex instance.
+//
+// API: github.com/stackitcloud/stackit-sdk-go/services/postgresflex/v2api
+//
+// Endpoints used:
+//   - GET    /v2/projects/{projectId}/regions/{region}/instances
+//   - DELETE /v2/projects/{projectId}/regions/{region}/instances/{instanceId}
+//   - DELETE /v2/projects/{projectId}/regions/{region}/instances/{instanceId}/force
+//
+// Deleting an instance is a soft delete: it stays listed with status
+// "Deleted" during the retention window. Remove therefore force-deletes
+// instances already in that state so a nuke run converges instead of
+// re-deleting the same instance forever.
 type PostgresFlexInstance struct {
 	*BaseResource `property:",inline"`
-	ID            string
-	Name          string
+
+	api postgresflexv2.DefaultAPI
+
+	ID     string
+	Name   string
+	Status string
 }
 
-func (r *PostgresFlexInstance) Remove(_ context.Context) error {
-	return fmt.Errorf("PostgresFlexInstance.Remove not yet implemented")
+func (r *PostgresFlexInstance) Remove(ctx context.Context) error {
+	if r.api == nil {
+		return fmt.Errorf("PostgresFlexInstance.Remove: api client not set")
+	}
+	if r.Status == "Deleted" {
+		return r.api.ForceDeleteInstance(ctx, r.ProjectID, r.Region, r.ID).Execute()
+	}
+	return r.api.DeleteInstance(ctx, r.ProjectID, r.Region, r.ID).Execute()
 }
+
 func (r *PostgresFlexInstance) Properties() types.Properties { return PropsFromStruct(r) }
 func (r *PostgresFlexInstance) String() string               { return r.Name }
 
 type PostgresFlexInstanceLister struct{}
 
-func (l *PostgresFlexInstanceLister) List(_ context.Context, _ any) ([]resource.Resource, error) {
-	return []resource.Resource{}, nil
+func (l *PostgresFlexInstanceLister) List(ctx context.Context, o any) ([]resource.Resource, error) {
+	opts := o.(*stackit.ListerOpts)
+	logger := logrus.WithFields(logrus.Fields{
+		"resource": PostgresFlexInstanceResource,
+		"project":  opts.ProjectID,
+		"region":   opts.Region,
+	})
+
+	if opts.Credentials == nil {
+		return nil, fmt.Errorf("missing STACKIT credentials")
+	}
+
+	client, err := postgresflexv2.NewAPIClient(stackitConfigOpts(opts)...)
+	if err != nil {
+		return nil, fmt.Errorf("build postgresflex client: %w", err)
+	}
+
+	logger.Trace("listing postgresflex instances")
+	resp, err := client.DefaultAPI.ListInstances(ctx, opts.ProjectID, opts.Region).Execute()
+	if err != nil {
+		return nil, fmt.Errorf("list postgresflex instances: %w", err)
+	}
+
+	out := make([]resource.Resource, 0, len(resp.GetItems()))
+	for _, i := range resp.GetItems() {
+		id, ok := i.GetIdOk()
+		if !ok || id == nil {
+			continue
+		}
+		name, _ := i.GetNameOk()
+		status, _ := i.GetStatusOk()
+
+		out = append(out, &PostgresFlexInstance{
+			BaseResource: &BaseResource{
+				OrganizationID: opts.OrganizationID,
+				ProjectID:      opts.ProjectID,
+				Region:         opts.Region,
+			},
+			api:    client.DefaultAPI,
+			ID:     *id,
+			Name:   stringDeref(name),
+			Status: stringDeref(status),
+		})
+	}
+
+	logger.WithField("count", len(out)).Trace("list complete")
+	return out, nil
 }

resources/postgresflex-instance_test.go

@@ -0,0 +1,67 @@
+package resources
+
+import (
+	"context"
+	"strings"
+	"testing"
+
+	"github.com/ekristen/libnuke/pkg/registry"
+
+	"github.com/qaiser42/stackit-nuke/pkg/stackit"
+)
+
+func TestPostgresFlexInstance_StringAndProperties(t *testing.T) {
+	r := &PostgresFlexInstance{
+		BaseResource: &BaseResource{ProjectID: "proj-1", Region: "eu01"},
+		ID:           "pg-abc",
+		Name:         "orders-db",
+		Status:       "Ready",
+	}
+
+	if got := r.String(); got != "orders-db" {
+		t.Errorf("String() = %q, want orders-db", got)
+	}
+
+	props := r.Properties()
+	rendered := props.String()
+	for _, want := range []string{"orders-db", "pg-abc", "proj-1", "eu01", "Ready"} {
+		if !strings.Contains(rendered, want) {
+			t.Errorf("Properties() = %q; missing %q", rendered, want)
+		}
+	}
+}
+
+func TestPostgresFlexInstance_RemoveWithoutAPIErrors(t *testing.T) {
+	r := &PostgresFlexInstance{
+		BaseResource: &BaseResource{ProjectID: "p", Region: "r"},
+		ID:           "id",
+	}
+	err := r.Remove(context.Background())
+	if err == nil {
+		t.Fatal("expected error when api is unset")
+	}
+	if !strings.Contains(err.Error(), "api client not set") {
+		t.Errorf("unexpected error: %v", err)
+	}
+}
+
+func TestPostgresFlexInstanceLister_RequiresCredentials(t *testing.T) {
+	l := &PostgresFlexInstanceLister{}
+	_, err := l.List(context.Background(), &stackit.ListerOpts{ProjectID: "p", Region: "r"})
+	if err == nil {
+		t.Fatal("expected error when credentials missing")
+	}
+}
+
+func TestPostgresFlexInstanceRegistration(t *testing.T) {
+	regs := registry.GetRegistration(PostgresFlexInstanceResource)
+	if regs == nil {
+		t.Fatal("PostgresFlexInstance not registered")
+	}
+	if regs.Scope != stackit.ProjectScope {
+		t.Errorf("scope = %q, want %q", regs.Scope, stackit.ProjectScope)
+	}
+	if regs.Lister == nil || regs.Resource == nil {
+		t.Error("lister or resource nil in registration")
+	}
+}

resources/registry_test.go

@@ -72,11 +72,12 @@ func TestAllResourcesUseProjectScope(t *testing.T) {
 // They are excluded from the stub-emptiness check below; they get their own
 // per-resource tests instead.
 var realImpls = map[string]bool{
-	"ComputeServer":    true,
-	"ComputeVolume":    true,
-	"NetworkInterface": true,
-	"Network":          true,
-	"SecurityGroup":    true,
+	"ComputeServer":        true,
+	"ComputeVolume":        true,
+	"NetworkInterface":     true,
+	"Network":              true,
+	"SecurityGroup":        true,
+	"PostgresFlexInstance": true,
 }
 
 func TestListersReturnEmpty(t *testing.T) {