From 89b8125739adc33c3f1821d62cd719a8e276de0f Mon Sep 17 00:00:00 2001 From: Jarrad Whitaker Date: Thu, 14 Dec 2023 11:21:26 +1100 Subject: [PATCH] ** remove len(addrs)>1 check in syncStatus shutdown() this logic is no longer valid but I don't understand what's going on well enough to justify its removal yet - input welcome if you can tell me if this is/isn't reasonable. --- internal/ingress/status/status.go | 6 ------ 1 file changed, 6 deletions(-) diff --git a/internal/ingress/status/status.go b/internal/ingress/status/status.go index e16b50170..9b8c61192 100644 --- a/internal/ingress/status/status.go +++ b/internal/ingress/status/status.go @@ -121,12 +121,6 @@ func (s *statusSync) Shutdown() { return } - if len(addrs) > 1 { - // leave the job to the next leader - klog.InfoS("leaving status update for next leader") - return - } - if s.isRunningMultiplePods() { klog.V(2).InfoS("skipping Ingress status update (multiple pods running - another one will be elected as master)") return