fix when syncing standby discription (#2513)

FxKu · web-flow · commit 3bad9aaded6e · 2024-01-12T10:41:17.000+01:00
diff --git a/pkg/cluster/cluster.go b/pkg/cluster/cluster.go
@@ -981,7 +981,7 @@ func (c *Cluster) Update(oldSpec, newSpec *acidv1.Postgresql) error {
 	}()
 
 	// add or remove standby_cluster section from Patroni config depending on changes in standby section
-	if reflect.DeepEqual(oldSpec.Spec.StandbyCluster, newSpec.Spec.StandbyCluster) {
+	if !reflect.DeepEqual(oldSpec.Spec.StandbyCluster, newSpec.Spec.StandbyCluster) {
 		if err := c.syncStandbyClusterConfiguration(); err != nil {
 			return fmt.Errorf("could not set StandbyCluster configuration options: %v", err)
 		}
diff --git a/pkg/cluster/sync.go b/pkg/cluster/sync.go
@@ -85,7 +85,7 @@ func (c *Cluster) Sync(newSpec *acidv1.Postgresql) error {
 	}
 
 	// add or remove standby_cluster section from Patroni config depending on changes in standby section
-	if reflect.DeepEqual(oldSpec.Spec.StandbyCluster, newSpec.Spec.StandbyCluster) {
+	if !reflect.DeepEqual(oldSpec.Spec.StandbyCluster, newSpec.Spec.StandbyCluster) {
 		if err := c.syncStandbyClusterConfiguration(); err != nil {
 			return fmt.Errorf("could not sync StandbyCluster configuration: %v", err)
 		}

Original file line number	Diff line number	Diff line change
`@@ -981,7 +981,7 @@ func (c Cluster) Update(oldSpec, newSpec acidv1.Postgresql) error {`
`981`	`981`	`}()`
`982`	`982`
`983`	`983`	`// add or remove standby_cluster section from Patroni config depending on changes in standby section`
`984`		`- if reflect.DeepEqual(oldSpec.Spec.StandbyCluster, newSpec.Spec.StandbyCluster) {`
	`984`	`+ if !reflect.DeepEqual(oldSpec.Spec.StandbyCluster, newSpec.Spec.StandbyCluster) {`
`985`	`985`	`if err := c.syncStandbyClusterConfiguration(); err != nil {`
`986`	`986`	`return fmt.Errorf("could not set StandbyCluster configuration options: %v", err)`
`987`	`987`	`}`
Original file line number	Diff line number	Diff line change
`@@ -85,7 +85,7 @@ func (c Cluster) Sync(newSpec acidv1.Postgresql) error {`
`85`	`85`	`}`
`86`	`86`
`87`	`87`	`// add or remove standby_cluster section from Patroni config depending on changes in standby section`
`88`		`- if reflect.DeepEqual(oldSpec.Spec.StandbyCluster, newSpec.Spec.StandbyCluster) {`
	`88`	`+ if !reflect.DeepEqual(oldSpec.Spec.StandbyCluster, newSpec.Spec.StandbyCluster) {`
`89`	`89`	`if err := c.syncStandbyClusterConfiguration(); err != nil {`
`90`	`90`	`return fmt.Errorf("could not sync StandbyCluster configuration: %v", err)`
`91`	`91`	`}`