Merge pull request #52 from pluralsh/reinstall-on-host-change

Force reinstalls on k8s host changes

Author: michaeljguarino

internal/resource/cluster.go

@@ -129,7 +129,9 @@ func (r *clusterResource) Update(ctx context.Context, req resource.UpdateRequest
 		return
 	}
 
-	if !data.HelmRepoUrl.Equal(state.HelmRepoUrl) && data.HasKubeconfig() {
+	reinstallable := !data.HelmRepoUrl.Equal(state.HelmRepoUrl) || !data.GetKubeconfig().Unchanged(state.GetKubeconfig())
+
+	if reinstallable && data.HasKubeconfig() {
 		clusterWithToken, err := r.client.GetClusterWithToken(ctx, data.Id.ValueStringPointer(), nil)
 		if err != nil {
 			resp.Diagnostics.AddError("Client Error", fmt.Sprintf("Unable to fetch cluster deploy token, got error: %s", err))

internal/resource/cluster_model.go

@@ -255,3 +255,15 @@ type KubeconfigExec struct {
 	Env        types.Map    `tfsdk:"env"`
 	APIVersion types.String `tfsdk:"api_version"`
 }
+
+func (k *Kubeconfig) Unchanged(other *Kubeconfig) bool {
+	if k == nil {
+		return other == nil
+	}
+
+	if other == nil {
+		return false
+	}
+
+	return k.Host == other.Host
+}