Correctly report success in .status on Update (zalando#469)

redbaron · sdudoladov · commit ed6acc117804 · 2019-01-31T13:09:17.000+01:00
diff --git a/pkg/cluster/cluster.go b/pkg/cluster/cluster.go
@@ -494,7 +494,7 @@ func (c *Cluster) Update(oldSpec, newSpec *acidv1.Postgresql) error {
 	defer func() {
 		if updateFailed {
 			c.setStatus(acidv1.ClusterStatusUpdateFailed)
-		} else if c.Status != acidv1.ClusterStatusRunning {
+		} else {
 			c.setStatus(acidv1.ClusterStatusRunning)
 		}
 	}()

Original file line number	Diff line number	Diff line change
`@@ -494,7 +494,7 @@ func (c Cluster) Update(oldSpec, newSpec acidv1.Postgresql) error {`
`494`	`494`	`defer func() {`
`495`	`495`	`if updateFailed {`
`496`	`496`	`c.setStatus(acidv1.ClusterStatusUpdateFailed)`
`497`		`- } else if c.Status != acidv1.ClusterStatusRunning {`
	`497`	`+ } else {`
`498`	`498`	`c.setStatus(acidv1.ClusterStatusRunning)`
`499`	`499`	`}`
`500`	`500`	`}()`