DevFW-CICD/ingress-nginx-helm
16
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

** remove len(addrs)>1 check in syncStatus shutdown()

Browse source 
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.
This commit is contained in:
Jarrad Whitaker 2023-12-14 11:21:26 +11:00
parent e518ae45c7
commit 89b8125739
No known key found for this signature in database
1 changed files with 0 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
internal/ingress/status/status.go
View file

@ -121,12 +121,6 @@ func (s *statusSync) Shutdown() {
return return
} }
if len(addrs) > 1 {
// leave the job to the next leader
klog.InfoS("leaving status update for next leader")
return
}
if s.isRunningMultiplePods() { if s.isRunningMultiplePods() {
klog.V(2).InfoS("skipping Ingress status update (multiple pods running - another one will be elected as master)") klog.V(2).InfoS("skipping Ingress status update (multiple pods running - another one will be elected as master)")
return return