Francesco Canovai

This is not limited to logical replication, a streaming client has the same problem. I fixed it using @r4tt4 suggestion.

I've prepared a draft PR (#3711) . It is missing user documentation and needs thorough testing, but I'd like some feedback first.

+1. If we have a single, fast and easy procedure to demote a cluster and promote the replica one it will be easier from everyone. And less intermediate steps could...

This seems mostly a documentation and naming issue to me, so let me first summarise the operator behaviour, according to the Cluster configuration and storage. ### Single instance cluster The...

/test level=4

I don't like the idea to generate metrics with different names from the same table, although on different postgres version. Additionally, such changes should affect the grafana dashboard as well.

> What about `minSyncReplicasEnforcement` accepting `preferred` (current default) or `required`? And in a future version make required the default (to facilitate DBAs transition). We could even think of an annotation...

I'd like to see it both at the same level of the min parameter it refers to, and in the `spec.postgresql` section. So, no strong opinion, but I think putting...

On a second though, the interface (and the documentation) will be way too complex. From the documentation, it's really unclear what the point of `minSyncReplicas` is, unless it gets enforced....