some fixes

pluralsh · Apr 17, 2024 · 93f792c · 93f792c
1 parent 123351f
commit 93f792c
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 1 deletion.
diff --git a/example/byok/main.tf b/example/byok/main.tf
@@ -25,6 +25,7 @@ resource "plural_cluster" "byok" {
       abc = false
     })
   })
+  helm_repo_url = "https://pluralsh.github.io/deployment-operator"
   tags = {
     "managed-by" = "terraform-provider-plural"
   }

diff --git a/internal/resource/cluster.go b/internal/resource/cluster.go
@@ -71,7 +71,7 @@ func (r *clusterResource) Create(ctx context.Context, req resource.CreateRequest
 			return
 		}
 
-		handler, err := NewOperatorHandler(ctx, data.GetKubeconfig(), data.HelmRepoUrl.String(), data.HelmValues.ValueStringPointer(), r.consoleUrl)
+		handler, err := NewOperatorHandler(ctx, data.GetKubeconfig(), data.HelmRepoUrl.ValueString(), data.HelmValues.ValueStringPointer(), r.consoleUrl)
 		if err != nil {
 			resp.Diagnostics.AddError("Client Error", fmt.Sprintf("Unable to init operator handler, got error: %s", err))
 			return

diff --git a/internal/resource/cluster_schema.go b/internal/resource/cluster_schema.go
@@ -121,6 +121,7 @@ func (r *clusterResource) schema() schema.Schema {
 				Optional:            true,
 				Computed:            true,
 				Default:             stringdefault.StaticString(console.RepoUrl),
+				PlanModifiers:       []planmodifier.String{stringplanmodifier.RequiresReplace()},
 			},
 			"helm_values": schema.StringAttribute{
 				Description:         "Additional Helm values you'd like to use in deployment agent Helm installs. This is useful for BYOK clusters that need to use custom images or other constructs.",