wait_for_default_service_account for Namespace

Handle waiter code within an afterCreate hook and enable all "dualstack"
tests except generateName for documented reason.

Author: appilon

go.mod

@@ -4,6 +4,7 @@ go 1.21.5
 
 require (
 	github.com/Masterminds/semver v1.5.0
+	github.com/cenkalti/backoff/v4 v4.2.1
 	github.com/getkin/kin-openapi v0.111.0
 	github.com/google/go-cmp v0.6.0
 	github.com/hashicorp/go-hclog v1.6.3

go.sum

@@ -34,6 +34,8 @@ github.com/bgentry/speakeasy v0.1.0 h1:ByYyxL9InA1OWqxJqqp2A5pYHUrCiAL6K3J+LKSsQ
 github.com/bgentry/speakeasy v0.1.0/go.mod h1:+zsyZBPWlz7T6j88CTgSN5bM796AkVf0kBD4zp0CCIs=
 github.com/bufbuild/protocompile v0.4.0 h1:LbFKd2XowZvQ/kajzguUp2DC9UEIQhIq77fZZlaQsNA=
 github.com/bufbuild/protocompile v0.4.0/go.mod h1:3v93+mbWn/v3xzN+31nwkJfrEpAUwp+BagBSZWx+TP8=
+github.com/cenkalti/backoff/v4 v4.2.1 h1:y4OZtCnogmCPw98Zjyt5a6+QwPLGkiQsYW5oUqylYbM=
+github.com/cenkalti/backoff/v4 v4.2.1/go.mod h1:Y3VNntkOUPxTVeUxJ/G5vcM//AlwfmyYozVcomhLiZE=
 github.com/census-instrumentation/opencensus-proto v0.2.1/go.mod h1:f6KPmirojxKA12rnyqOA5BBL4O983OfeGPqjHWSTneU=
 github.com/chai2010/gettext-go v1.0.2 h1:1Lwwip6Q2QGsAdl/ZKPCwTe9fe0CjlUbqj5bFNSjIRk=
 github.com/chai2010/gettext-go v1.0.2/go.mod h1:y+wnP2cHYaVj19NZhYKAwEMH2CI1gNHeQQ+5AjwawxA=
@@ -175,10 +177,6 @@ github.com/hashicorp/terraform-exec v0.21.0 h1:uNkLAe95ey5Uux6KJdua6+cv8asgILFVW
 github.com/hashicorp/terraform-exec v0.21.0/go.mod h1:1PPeMYou+KDUSSeRE9szMZ/oHf4fYUmB923Wzbq1ICg=
 github.com/hashicorp/terraform-json v0.22.1 h1:xft84GZR0QzjPVWs4lRUwvTcPnegqlyS7orfb5Ltvec=
 github.com/hashicorp/terraform-json v0.22.1/go.mod h1:JbWSQCLFSXFFhg42T7l9iJwdGXBYV8fmmD6o/ML4p3A=
-github.com/hashicorp/terraform-plugin-codegen-kubernetes v0.1.0 h1:cPPZitB7w5qYPbSad0IkhDaXZc7LpDhsgm4g8Kj2cOY=
-github.com/hashicorp/terraform-plugin-codegen-kubernetes v0.1.0/go.mod h1:AOwjDhbkWel/4J4fnLB59l52cGmV13FjkqKxaNVJjok=
-github.com/hashicorp/terraform-plugin-codegen-kubernetes v0.1.1-0.20241022191930-51e5c0aa016a h1:NjEx7QP4I2xXV0hNYft98cTJtz5bOrR/ig3bbqPzPsI=
-github.com/hashicorp/terraform-plugin-codegen-kubernetes v0.1.1-0.20241022191930-51e5c0aa016a/go.mod h1:AfADi0tmCqyqPiz4Yyprr//qe+Kwz5HdxWaERDcDCl4=
 github.com/hashicorp/terraform-plugin-codegen-kubernetes v0.1.1-0.20241022192615-7f55d8b45646 h1:hMR+nw3FRKeRVhBrLf9WjAK8TWPjmTIsxdA8Qlgi6uk=
 github.com/hashicorp/terraform-plugin-codegen-kubernetes v0.1.1-0.20241022192615-7f55d8b45646/go.mod h1:AfADi0tmCqyqPiz4Yyprr//qe+Kwz5HdxWaERDcDCl4=
 github.com/hashicorp/terraform-plugin-docs v0.16.0 h1:UmxFr3AScl6Wged84jndJIfFccGyBZn52KtMNsS12dI=

internal/framework/helper/dynamic.go

@@ -0,0 +1,31 @@
+package helper
+
+import (
+	"fmt"
+
+	"github.com/hashicorp/terraform-plugin-framework/types"
+)
+
+func DynamicToBool(d types.Dynamic) bool {
+	// treat null or unknown as unset/falsy
+	if d.IsUnknown() || d.IsNull() {
+		return false
+	}
+	switch value := d.UnderlyingValue().(type) {
+	case types.Bool:
+		return value.ValueBool()
+	case types.String:
+		// Terraform performs some conversions on assignment we should honor
+		// https://developer.hashicorp.com/terraform/language/expressions/types#type-conversion
+		switch value.ValueString() {
+		case "true":
+			return true
+		case "false":
+			return false
+		default:
+			panic(fmt.Errorf("%v is not a bool", value.ValueString()))
+		}
+	default:
+		panic(fmt.Errorf("%v is not a bool", value))
+	}
+}

internal/framework/provider/client/resource_interface.go

@@ -0,0 +1,36 @@
+package client
+
+import (
+	"k8s.io/apimachinery/pkg/runtime/schema"
+	"k8s.io/client-go/dynamic"
+	"k8s.io/client-go/restmapper"
+)
+
+func ResourceInterface(clientGetter KubernetesClientGetter, kind, apiVersion, namespace string) (dynamic.ResourceInterface, error) {
+	client, err := clientGetter.DynamicClient()
+	if err != nil {
+		return nil, err
+	}
+	discoveryClient, err := clientGetter.DiscoveryClient()
+	if err != nil {
+		return nil, err
+	}
+	agr, err := restmapper.GetAPIGroupResources(discoveryClient)
+	if err != nil {
+		return nil, err
+	}
+	gv, err := schema.ParseGroupVersion(apiVersion)
+	if err != nil {
+		return nil, err
+	}
+	restMapper := restmapper.NewDiscoveryRESTMapper(agr)
+	mapping, err := restMapper.RESTMapping(gv.WithKind(kind).GroupKind(), gv.Version)
+	if err != nil {
+		return nil, err
+	}
+
+	if namespace == "" {
+		return client.Resource(mapping.Resource), nil
+	}
+	return client.Resource(mapping.Resource).Namespace(namespace), nil
+}

internal/framework/provider/corev1/generate_namespace.hcl

@@ -21,6 +21,11 @@ resource "kubernetes_namespace_v1_gen" {
 
     autocrud_options {
       wait_for_deletion = true
+      hooks {
+        after {
+          create = true
+        }
+      }
     }
   }
 
@@ -38,6 +43,10 @@ resource "kubernetes_namespace_v1_gen" {
     "spec"
   ]
 
+  custom_attributes = [
+    "wait_for_default_service_account"
+  ]
+
   required_attributes = [
     "metadata"
   ]

internal/framework/provider/corev1/namespace_crud_gen.go

@@ -40,6 +40,8 @@ func (r *Namespace) Create(ctx context.Context, req resource.CreateRequest, resp
 		return
 	}
 
+	r.AfterCreate(ctx, req, resp, &dataModel)
+
 	diags := resp.State.Set(ctx, &dataModel)
 	resp.Diagnostics.Append(diags...)
 	if resp.Diagnostics.HasError() {

internal/framework/provider/corev1/namespace_hooks.go

@@ -8,8 +8,9 @@ import (
 type NamespaceModel struct {
 	Timeouts timeouts.Value `tfsdk:"timeouts"`
 
-	ID       types.String `tfsdk:"id" manifest:""`
-	Metadata struct {
+	ID                           types.String  `tfsdk:"id" manifest:""`
+	WaitForDefaultServiceAccount types.Dynamic `tfsdk:"wait_for_default_service_account" manifest:""`
+	Metadata                     struct {
 		Annotations     map[string]types.String `tfsdk:"annotations" manifest:"annotations"`
 		GenerateName    types.String            `tfsdk:"generate_name" manifest:"generateName"`
 		Generation      types.Int64             `tfsdk:"generation" manifest:"generation"`

internal/framework/provider/corev1/namespace_model_gen.go

@@ -23,6 +23,10 @@ func (r *Namespace) Schema(ctx context.Context, req resource.SchemaRequest, resp
 				Optional:            true,
 				Computed:            true,
 			},
+			"wait_for_default_service_account": schema.DynamicAttribute{
+				MarkdownDescription: ``,
+				Optional:            true,
+			},
 			"metadata": schema.SingleNestedAttribute{
 				MarkdownDescription: `Standard object's metadata. More info: https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#metadata`,
 				Required:            true,