10090
39%

Ran 15 Aug 2019 10:38PM UTC

Jobs 1

Files 435

Run time 32s

Badge

Embed ▾

Committed 15 Aug 2019 10:19PM UTC coverage: 44.187% (-0.02%) from 44.209%

Build # 10090

Build Type

push

travis-ci-com

Committed by

ianvernon

Commit Message

endpoint: Fix proxy port leak on endpoint delete

Deep in the proxy.removeRedirect() logic, a finalize function is created
to defer proxy port release to allow a port to be reused by the proxy
after a grace period. The port is not actually released when the
removeRedirect() function is returned, instead the finalize function is
passed all the way up to the caller.

In the endpoint leave case, this finalize function was being ignored,
meaning that the proxy port used by this endpoint is never released.
After 10,000 endpoint deletions of endpoints that have L7 policy
applied, the endpoint regeneration begins to fail for new endpoints that
are subject to L7 policy. As a result, the CNI times out:

  Failed create pod sandbox: rpc error:
  code = Unknown
  desc = failed to set up sandbox container "xxx" network for pod "yyy":
  NetworkPlugin cni failed to set up pod "yyy" network:
  Unable to create endpoint:
  Put http:///var/run/cilium/cilium.sock/v1/endpoint/cilium-local:0:
  context deadline exceeded

Fix this by running the finalize function upon endpoint leave.

Signed-off-by: Joe Stringer <joe@cilium.io>

Coverage Stats

25427 of 57544 relevant lines covered (44.19%)

1185.8 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	10090.1	15 Aug 2019 10:38PM UTC	0	44.19	Travis Job 10090.1

Source Files on build 10090

Detailed source file information is not available for this build.

cilium / cilium / 10090
39%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 10090

cilium / cilium / 10090 39%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 10090

cilium / cilium / 10090
39%

README BADGES
x