minor changes

pkg/cluster/sync.go

@@ -404,7 +404,7 @@ func (c *Cluster) syncStatefulSet() error {
 		}
 
 		// empty config probably means cluster is not fully initialized yet, e.g. restoring from backup
-		// to not attempt a restart in such situation
+		// do not attempt a restart
 		if !reflect.DeepEqual(patroniConfig, emptyPatroniConfig) || len(pgParameters) > 0 {
 			instanceRestartRequired, err = c.checkAndSetGlobalPostgreSQLConfiguration(&pod, patroniConfig, pgParameters)
 			if err != nil {

pkg/util/patroni/patroni.go

@@ -238,8 +238,12 @@ func (p *Patroni) Restart(server *v1.Pod) error {
 	if err != nil {
 		return err
 	}
-	memberData, _ := p.GetMemberData(server)
+	memberData, err := p.GetMemberData(server)
+	if err != nil {
+		return err
+	}
 
+	// do restart only when it is pending
 	if !memberData.PendingRestart {
 		return nil
 	}