adjustsments on the logic

gkech · gkech · commit 7e7e05b1d33b · 2025-07-09T18:05:06.000+03:00
diff --git a/internal/controller/postgrescluster/postgres.go b/internal/controller/postgrescluster/postgres.go
@@ -583,17 +583,15 @@ func (r *Reconciler) reconcilePostgresUserSecrets(
 
 		if err == nil {
 			userSecrets[userName], err = r.generatePostgresUserSecret(cluster, user, secret)
-		}
-		if err == nil {
 			err = errors.WithStack(r.apply(ctx, userSecrets[userName]))
 		}
 	}
 
 	return specUsers, userSecrets, err
 }
 
-// updateCustomSecretLabels checks if a custom secret exists and updates it
-// with required labels if they are missing that enabled the
+// updateCustomSecretLabels checks if a custom secret exists - can be created manually through kubectl apply
+// and updates it with required labels if they are missing that enabled the
 // naming.AsSelector(naming.ClusterPostgresUsers(cluster.Name)) to identify them.
 func (r *Reconciler) updateCustomSecretLabels(
 	ctx context.Context, cluster *v1beta1.PostgresCluster, user v1beta1.PostgresUserSpec,

Original file line number	Diff line number	Diff line change
`@@ -583,17 +583,15 @@ func (r *Reconciler) reconcilePostgresUserSecrets(`
`583`	`583`
`584`	`584`	`if err == nil {`
`585`	`585`	`userSecrets[userName], err = r.generatePostgresUserSecret(cluster, user, secret)`
`586`		`- }`
`587`		`- if err == nil {`
`588`	`586`	`err = errors.WithStack(r.apply(ctx, userSecrets[userName]))`
`589`	`587`	`}`
`590`	`588`	`}`
`591`	`589`
`592`	`590`	`return specUsers, userSecrets, err`
`593`	`591`	`}`
`594`	`592`
`595`		`-// updateCustomSecretLabels checks if a custom secret exists and updates it`
`596`		`-// with required labels if they are missing that enabled the`
	`593`	`+// updateCustomSecretLabels checks if a custom secret exists - can be created manually through kubectl apply`
	`594`	`+// and updates it with required labels if they are missing that enabled the`
`597`	`595`	`// naming.AsSelector(naming.ClusterPostgresUsers(cluster.Name)) to identify them.`
`598`	`596`	`func (r *Reconciler) updateCustomSecretLabels(`
`599`	`597`	`ctx context.Context, cluster *v1beta1.PostgresCluster, user v1beta1.PostgresUserSpec,`