ci: get ci passing

Author: jasondamour

.github/workflows/samples-terraform.yaml

@@ -34,6 +34,10 @@ jobs:
         with:
           go-version: "stable"
       - run: go version
+      - name: Install Dependencies
+        working-directory: ${{ matrix.sample }}
+        run: |
+          go mod tidy
       - name: Build provider
         working-directory: ${{ matrix.sample }}
         run: go build -v ./...

modules/openapi-generator/src/main/java/org/openapitools/codegen/languages/TerraformProviderCodegen.java

@@ -444,6 +444,21 @@ public ModelsMap postProcessModels(ModelsMap objs) {
         for (ModelMap m : objs.getModels()) {
             CodegenModel model = m.getModel();
             for (CodegenProperty prop : model.vars) {
+                // Ensure x-go-datatag is set for all model vars.
+                // The parent AbstractGoCodegen only sets x-go-datatag on
+                // inheritedProperties (oneOf/anyOf entries) when the model has
+                // composed schemas without allOf, leaving model.vars without
+                // json tags. We fix this by generating the tag here when missing.
+                if (!prop.vendorExtensions.containsKey("x-go-datatag")) {
+                    String goDataTag = "json:\"" + prop.baseName;
+                    if (!prop.required) {
+                        goDataTag += ",omitempty";
+                    }
+                    goDataTag += "\"";
+                    goDataTag = " `" + goDataTag + "`";
+                    prop.vendorExtensions.put("x-go-datatag", goDataTag);
+                }
+
                 // Add Terraform-specific vendor extensions
                 if (prop.isReadOnly) {
                     prop.vendorExtensions.put("x-terraform-computed", true);

modules/openapi-generator/src/main/resources/terraform-provider/README.mustache

@@ -5,7 +5,7 @@ This Terraform provider was generated using [OpenAPI Generator](https://openapi-
 ## Requirements
 
 - [Terraform](https://www.terraform.io/downloads.html) >= 1.0
-- [Go](https://golang.org/doc/install) >= 1.21
+- [Go](https://golang.org/doc/install) >= 1.22
 
 ## Building The Provider
 
@@ -39,6 +39,19 @@ If you wish to work on the provider, you'll first need [Go](http://www.golang.or
 
 To compile the provider, run `go install`. This will build the provider and put the provider binary in the `$GOPATH/bin` directory.
 
+For local development and testing, add a `dev_overrides` block to your `~/.terraformrc` so that Terraform uses the locally built binary instead of fetching from the registry:
+
+```hcl
+provider_installation {
+  dev_overrides {
+    "{{providerAddress}}" = "${GOPATH}/bin"
+  }
+  direct {}
+}
+```
+
+With `dev_overrides` configured, you do not need to run `terraform init` for the provider.
+
 To generate or update documentation, run `go generate`.
 
 In order to run the full suite of Acceptance tests, run `make testacc`.

modules/openapi-generator/src/main/resources/terraform-provider/gitignore.mustache

@@ -8,5 +8,6 @@
 *.tfstate
 *.tfstate.*
 .terraform/
-terraform.tfvars
+*.tfvars
+*.tfvars.json
 terraform-provider-{{providerName}}

modules/openapi-generator/src/main/resources/terraform-provider/go.mod.mustache

@@ -1,6 +1,6 @@
 module {{gitHost}}/{{gitUserId}}/{{gitRepoId}}
 
-go 1.22
+go 1.24.0
 
 toolchain go1.24.4
 

modules/openapi-generator/src/main/resources/terraform-provider/provider.mustache

@@ -78,9 +78,16 @@ func (p *{{providerName}}Provider) Configure(ctx context.Context, req provider.C
 		return
 	}
 
-	endpoint := config.Endpoint.ValueString()
-	if endpoint == "" {
-		endpoint = "{{{basePath}}}"
+	if config.Endpoint.IsUnknown() {
+		resp.Diagnostics.AddWarning(
+			"Unknown Endpoint Value",
+			"The provider endpoint value is not yet known. Defaulting to the spec-defined base path.",
+		)
+	}
+
+	endpoint := "{{{basePath}}}"
+	if !config.Endpoint.IsNull() && !config.Endpoint.IsUnknown() {
+		endpoint = config.Endpoint.ValueString()
 	}
 
 	c := client.NewClient(endpoint)

modules/openapi-generator/src/main/resources/terraform-provider/resource.mustache

@@ -162,7 +162,7 @@ func (r *{{resourceClassName}}Resource) Read(ctx context.Context, req resource.R
 {{/hasRead}}
 {{^hasRead}}
 func (r *{{resourceClassName}}Resource) Read(ctx context.Context, req resource.ReadRequest, resp *resource.ReadResponse) {
-	resp.Diagnostics.AddError("Not Supported", "Read is not supported for {{resourceName}}")
+	// No read endpoint available; keep existing state as-is.
 }
 {{/hasRead}}
 
@@ -202,7 +202,13 @@ func (r *{{resourceClassName}}Resource) Update(ctx context.Context, req resource
 {{/hasUpdate}}
 {{^hasUpdate}}
 func (r *{{resourceClassName}}Resource) Update(ctx context.Context, req resource.UpdateRequest, resp *resource.UpdateResponse) {
-	resp.Diagnostics.AddError("Not Supported", "Update is not supported for {{resourceName}}")
+	// No update endpoint available; persist the planned values into state.
+	var plan {{resourceClassName}}Model
+	resp.Diagnostics.Append(req.Plan.Get(ctx, &plan)...)
+	if resp.Diagnostics.HasError() {
+		return
+	}
+	resp.Diagnostics.Append(resp.State.Set(ctx, &plan)...)
 }
 {{/hasUpdate}}
 
@@ -231,7 +237,7 @@ func (r *{{resourceClassName}}Resource) Delete(ctx context.Context, req resource
 {{/hasDelete}}
 {{^hasDelete}}
 func (r *{{resourceClassName}}Resource) Delete(ctx context.Context, req resource.DeleteRequest, resp *resource.DeleteResponse) {
-	resp.Diagnostics.AddError("Not Supported", "Delete is not supported for {{resourceName}}")
+	// No delete endpoint available; remove the resource from state.
 }
 {{/hasDelete}}