Make sure we don't propagate users between syncs.

PostgreSQL users should not be propagated between syncs through the
internal state of the operator. Instead, they are always read from
the respective sources (i.e. teams API, infrastructure roles secrets).

pkg/cluster/cluster.go

@@ -168,6 +168,11 @@ func (c *Cluster) setStatus(status spec.PostgresStatus) {
 // initUsers populates c.systemUsers and c.pgUsers maps.
 func (c *Cluster) initUsers() error {
 	c.setProcessName("initializing users")
+
+	// clear our the previous state of the cluster users (in case we are running a sync).
+	c.systemUsers = map[string]spec.PgUser{}
+	c.pgUsers = map[string]spec.PgUser{}
+
 	c.initSystemUsers()
 
 	if err := c.initInfrastructureRoles(); err != nil {