use patch instead of update

gkech · gkech · commit 6c9d857df814 · 2025-07-09T14:01:18.000+03:00
diff --git a/internal/controller/postgrescluster/postgres.go b/internal/controller/postgrescluster/postgres.go
@@ -613,6 +613,8 @@ func (r *Reconciler) updateCustomSecretLabels(
 		return errors.Wrap(err, fmt.Sprintf("failed to get user %s secret %s", userName, secretName))
 	}
 
+	orig := secret.DeepCopy()
+
 	requiredLabels := map[string]string{
 		naming.LabelCluster:      cluster.Name,
 		naming.LabelPostgresUser: userName,
@@ -631,7 +633,7 @@ func (r *Reconciler) updateCustomSecretLabels(
 	}
 
 	if needsUpdate {
-		return errors.WithStack(r.Client.Update(ctx, secret))
+		return errors.WithStack(r.Client.Patch(ctx, secret.DeepCopy(), client.MergeFrom(orig)))
 	}
 
 	return nil

Original file line number	Diff line number	Diff line change
`@@ -613,6 +613,8 @@ func (r *Reconciler) updateCustomSecretLabels(`
`613`	`613`	`return errors.Wrap(err, fmt.Sprintf("failed to get user %s secret %s", userName, secretName))`
`614`	`614`	`}`
`615`	`615`
	`616`	`+ orig := secret.DeepCopy()`
	`617`	`+`
`616`	`618`	`requiredLabels := map[string]string{`
`617`	`619`	`naming.LabelCluster: cluster.Name,`
`618`	`620`	`naming.LabelPostgresUser: userName,`
`@@ -631,7 +633,7 @@ func (r *Reconciler) updateCustomSecretLabels(`
`631`	`633`	`}`
`632`	`634`
`633`	`635`	`if needsUpdate {`
`634`		`- return errors.WithStack(r.Client.Update(ctx, secret))`
	`636`	`+ return errors.WithStack(r.Client.Patch(ctx, secret.DeepCopy(), client.MergeFrom(orig)))`
`635`	`637`	`}`
`636`	`638`
`637`	`639`	`return nil`