Skip to content

Commit c4f5619

Browse files
authored
INTMDB-572: HELP-42215 - PATCH Update Advanced Configuration Options for One Cluster / Advance Cluster (#1066)
* improve detection of empty struct * Make cluster update when actual changes are present
1 parent c7357d1 commit c4f5619

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

mongodbatlas/resource_mongodbatlas_advanced_cluster.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -654,6 +654,7 @@ func resourceMongoDBAtlasAdvancedClusterUpdate(ctx context.Context, d *schema.Re
654654
clusterName := ids["cluster_name"]
655655

656656
cluster := new(matlas.AdvancedCluster)
657+
clusterChangeDetect := new(matlas.AdvancedCluster)
657658

658659
if d.HasChange("backup_enabled") {
659660
cluster.BackupEnabled = pointy.Bool(d.Get("backup_enabled").(bool))
@@ -714,7 +715,7 @@ func resourceMongoDBAtlasAdvancedClusterUpdate(ctx context.Context, d *schema.Re
714715
timeout := d.Timeout(schema.TimeoutUpdate)
715716

716717
// Has changes
717-
if !reflect.DeepEqual(cluster, matlas.Cluster{}) {
718+
if !reflect.DeepEqual(cluster, clusterChangeDetect) {
718719
err := resource.RetryContext(ctx, timeout, func() *resource.RetryError {
719720
_, _, err := updateAdvancedCluster(ctx, conn, cluster, projectID, clusterName, timeout)
720721
if err != nil {

mongodbatlas/resource_mongodbatlas_cluster.go

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -804,6 +804,8 @@ func resourceMongoDBAtlasClusterUpdate(ctx context.Context, d *schema.ResourceDa
804804
clusterName := ids["cluster_name"]
805805

806806
cluster := new(matlas.Cluster)
807+
clusterChangeDetect := new(matlas.Cluster)
808+
clusterChangeDetect.AutoScaling = &matlas.AutoScaling{Compute: &matlas.Compute{}}
807809

808810
if d.HasChange("bi_connector") {
809811
cluster.BiConnector, _ = expandBiConnector(d)
@@ -939,7 +941,7 @@ func resourceMongoDBAtlasClusterUpdate(ctx context.Context, d *schema.ResourceDa
939941
"cluster_name": updatedCluster.Name,
940942
"provider_name": updatedCluster.ProviderSettings.ProviderName,
941943
}))
942-
} else if !reflect.DeepEqual(cluster, matlas.Cluster{}) {
944+
} else if !reflect.DeepEqual(cluster, clusterChangeDetect) {
943945
err := resource.RetryContext(ctx, timeout, func() *resource.RetryError {
944946
_, _, err := updateCluster(ctx, conn, cluster, projectID, clusterName, timeout)
945947

0 commit comments

Comments
 (0)