Skip to content

Commit ca91b26

Browse files
committed
Retry failed verifications with exponential backoff
Signed-off-by: Stefan Prodan <[email protected]>
1 parent 082028e commit ca91b26

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

controllers/ocirepository_controller.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -413,7 +413,9 @@ func (r *OCIRepositoryReconciler) reconcileSource(ctx context.Context, obj *sour
413413
if obj.Spec.Verify == nil {
414414
// Remove old observations if verification was disabled
415415
conditions.Delete(obj, sourcev1.SourceVerifiedCondition)
416-
} else if !obj.GetArtifact().HasRevision(revision) || conditions.GetObservedGeneration(obj, sourcev1.SourceVerifiedCondition) != obj.Generation {
416+
} else if !obj.GetArtifact().HasRevision(revision) ||
417+
conditions.GetObservedGeneration(obj, sourcev1.SourceVerifiedCondition) != obj.Generation ||
418+
conditions.IsFalse(obj, sourcev1.SourceVerifiedCondition) {
417419
provider := obj.Spec.Verify.Provider
418420
err := r.verifyOCISourceSignature(ctx, obj, url, keychain)
419421
if err != nil {
@@ -422,7 +424,6 @@ func (r *OCIRepositoryReconciler) reconcileSource(ctx context.Context, obj *sour
422424
sourcev1.VerificationError,
423425
)
424426
conditions.MarkFalse(obj, sourcev1.SourceVerifiedCondition, e.Reason, e.Err.Error())
425-
conditions.MarkFalse(obj, meta.ReconcilingCondition, e.Reason, e.Err.Error())
426427
return sreconcile.ResultEmpty, e
427428
}
428429

0 commit comments

Comments
 (0)