Merge pull request #132 from fluxcd/disable-requeue

Do not requeue when chart is not ready
This commit is contained in:
Hidde Beydals 2020-10-28 17:26:41 +01:00 committed by GitHub
commit 7ab3ed3a55
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 1 deletions

View File

@ -226,7 +226,9 @@ func (r *HelmReleaseReconciler) reconcile(ctx context.Context, log logr.Logger,
msg := fmt.Sprintf("HelmChart '%s/%s' is not ready", hc.GetNamespace(), hc.GetName())
r.event(hr, hr.Status.LastAttemptedRevision, events.EventSeverityInfo, msg)
log.Info(msg)
return v2.HelmReleaseNotReady(hr, v2.ArtifactFailedReason, msg), ctrl.Result{Requeue: true}, nil
// Do not requeue, when the artifact is created the watcher should trigger a
// reconciliation.
return v2.HelmReleaseNotReady(hr, v2.ArtifactFailedReason, msg), ctrl.Result{Requeue: false}, nil
}
// Check dependencies