search

tuenti / kube2lb

Dinamically configure load balancers to expose Kubernetes services
Apache License 2.0
22 stars 5 forks source link

Memory leak #13

Open jsoriano opened 7 years ago

jsoriano commented 7 years ago

We have seen in graphs that memory usage of kube2lb continuously grows, there is a memory leak somewhere.

jsoriano commented 7 years ago

Profiling I have seen that there is increasing used memory by http2ClientConn.readLoop method, but I'm not very confident about my go profiling skills. This could explain why we see different behaviours in clusters with and without SSL, as go http client doesn't use HTTP2 without SSL.

There was a leak on this method for HTTP1 clients (https://github.com/golang/go/commit/58c1c011a694cc9a813b34c83eebab223edae1fd)

jsoriano commented 7 years ago

Other reported leaks that could be related: https://github.com/golang/go/issues/21229, https://github.com/golang/go/issues/21543