Merge pull request #75265 from JacobTanenbaum/ClearExternalIPs

Clear conntrack entries on 0 -> 1 endpoint transition with externalIPs

Merge pull request #75265 from JacobTanenbaum/ClearExternalIPs
aa9cbd11 · Kubernetes Prow Robot · GitHub · 28910bf2 · c3548165 · aa9cbd11
Unverified Commit aa9cbd11 authored Mar 18, 2019 by Kubernetes Prow Robot Committed by GitHub Mar 18, 2019
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 0 deletions

proxier.go pkg/proxy/iptables/proxier.go +3 -0

proxier.go pkg/proxy/ipvs/proxier.go +3 -0

No files found.
--- a/pkg/proxy/iptables/proxier.go
+++ b/pkg/proxy/iptables/proxier.go
@@ -674,6 +674,9 @@ func (proxier *Proxier) syncProxyRules() {
 		if svcInfo, ok := proxier.serviceMap[svcPortName]; ok && svcInfo != nil && svcInfo.GetProtocol() == v1.ProtocolUDP {
 			klog.V(2).Infof("Stale udp service %v -> %s", svcPortName, svcInfo.ClusterIPString())
 			staleServices.Insert(svcInfo.ClusterIPString())
+			for _, extIP := range svcInfo.ExternalIPStrings() {
+				staleServices.Insert(extIP)
+			}
 		}
 	}


--- a/pkg/proxy/ipvs/proxier.go
+++ b/pkg/proxy/ipvs/proxier.go
@@ -741,6 +741,9 @@ func (proxier *Proxier) syncProxyRules() {
 		if svcInfo, ok := proxier.serviceMap[svcPortName]; ok && svcInfo != nil && svcInfo.GetProtocol() == v1.ProtocolUDP {
 			klog.V(2).Infof("Stale udp service %v -> %s", svcPortName, svcInfo.ClusterIPString())
 			staleServices.Insert(svcInfo.ClusterIPString())
+			for _, extIP := range svcInfo.ExternalIPStrings() {
+				staleServices.Insert(extIP)
+			}
 		}
 	}