Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Memory leak #13

Open
jsoriano opened this issue Jun 13, 2017 · 2 comments
Open

Memory leak #13

jsoriano opened this issue Jun 13, 2017 · 2 comments
Labels
bug
Milestone
next

Comments

@jsoriano
Copy link
Contributor

jsoriano commented Jun 13, 2017 •
edited
Loading

We have seen in graphs that memory usage of kube2lb continuously grows, there is a memory leak somewhere.
@jsoriano jsoriano added the bug label Jun 13, 2017
@jsoriano jsoriano added this to the next milestone Jun 13, 2017
@jsoriano jsoriano self-assigned this Jun 13, 2017
This was referenced Jun 15, 2017
Merged
Merged
@jsoriano jsoriano added wip and removed wip labels Jun 15, 2017
@jsoriano jsoriano modified the milestones: next, v2.1.3 Aug 7, 2017
@jsoriano jsoriano mentioned this issue Aug 21, 2017
Merged
@jsoriano
Copy link
Contributor Author

jsoriano commented Aug 21, 2017 •
edited
Loading

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 (golang/go@58c1c01)

@jsoriano
Copy link
Contributor Author

Other reported leaks that could be related: golang/go#21229, golang/go#21543

@jsoriano jsoriano modified the milestones: 2.3.0, next Dec 21, 2017
@jsoriano jsoriano removed their assignment Apr 3, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
next
Development

No branches or pull requests
1 participant
@jsoriano