Skip to content

Commit

Permalink
fix: retry Conflict errors when upgrading k8s manifests
Browse files Browse the repository at this point in the history
Fixes #5985

Signed-off-by: Andrey Smirnov <[email protected]>
  • Loading branch information
smira committed Jul 29, 2022
1 parent 1db097f commit 0cdf222
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion pkg/cluster/kubernetes/talos_managed.go
Original file line number Diff line number Diff line change
Expand Up @@ -518,7 +518,7 @@ func syncManifests(ctx context.Context, objects []*unstructured.Unstructured, cl

err = retry.Constant(3*time.Minute, retry.WithUnits(10*time.Second), retry.WithErrorLogging(true)).RetryWithContext(ctx, func(ctx context.Context) error {
resp, diff, skipped, err = updateManifest(ctx, mapper, k8sClient, obj, options.DryRun)
if kubernetes.IsRetryableError(err) {
if kubernetes.IsRetryableError(err) || apierrors.IsConflict(err) {
return retry.ExpectedError(err)
}

Expand Down

0 comments on commit 0cdf222

Please sign in to comment.