Skip to content

Commit

Permalink
Merge pull request #101358 from sbangari/localendpointrefcount
Browse files Browse the repository at this point in the history
Ref counting is only applicable to Remote endpoints
  • Loading branch information
k8s-ci-robot committed May 5, 2021
2 parents 4f1bfe3 + 32b2cd2 commit 73c1b2e
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pkg/proxy/winkernel/proxier.go
Expand Up @@ -354,7 +354,7 @@ func newSourceVIP(hns HostNetworkService, network string, ip string, mac string,

func (ep *endpointsInfo) Cleanup() {
Log(ep, "Endpoint Cleanup", 3)
if ep.refCount != nil {
if !ep.GetIsLocal() && ep.refCount != nil {
*ep.refCount--

// Remove the remote hns endpoint, if no service is referring it
Expand Down Expand Up @@ -1157,10 +1157,10 @@ func (proxier *Proxier) syncProxyRules() {
} else {
// We only share the refCounts for remote endpoints
ep.refCount = proxier.endPointsRefCount.getRefCount(newHnsEndpoint.hnsID)
*ep.refCount++
}

ep.hnsID = newHnsEndpoint.hnsID
*ep.refCount++

Log(ep, "Endpoint resource found", 3)
}
Expand Down

0 comments on commit 73c1b2e

Please sign in to comment.