Skip to content

Commit

Permalink
Fixed reconcile not running on all KIngress after ksvc deletion (knat…
Browse files Browse the repository at this point in the history
…ive-extensions#1170) (#121)

Co-authored-by: Knative Prow Robot <[email protected]>
Co-authored-by: ryutoyasugi <[email protected]>
  • Loading branch information
3 people authored Dec 6, 2023
1 parent d26c449 commit f608629
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pkg/reconciler/ingress/controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -106,10 +106,10 @@ func NewController(ctx context.Context, cmw configmap.Watcher) *controller.Impl

r.resyncConflicts = func() {
impl.FilteredGlobalResync(func(obj interface{}) bool {
lbReady := obj.(*v1alpha1.Ingress).Status.GetCondition(v1alpha1.IngressConditionLoadBalancerReady).GetReason()
lbReady := obj.(*v1alpha1.Ingress).Status.GetCondition(v1alpha1.IngressConditionLoadBalancerReady)
// Force reconcile all Kourier ingresses that are either not reconciled yet
// (and thus might end up in a conflict) or already in conflict.
return isKourierIngress(obj) && (lbReady == "" || lbReady == conflictReason)
return isKourierIngress(obj) && (lbReady == nil || lbReady.GetReason() == conflictReason)
}, ingressInformer.Informer())
}

Expand Down

0 comments on commit f608629

Please sign in to comment.