-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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.
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
Suspected Memory Leak in linkerd-proxy #2345
Comments
Fairly sure this is caused by #2331. The polling-based fs watch implementation probably has a slow leak in it somewhere. |
@christianhuening Can you try out the latest edge-19.2.5 release to see if it resolves this issue? Let us know how it goes. |
@ihcsim updated, I'll report back, should I see another high mem use |
That one didn't reappear. Closing for now |
@christianhuening Great, I'm glad to hear that this was fixed! We should probably open a new ticket to resolve the underlying issue with the polling fs watch implementation. However, since fixing #2331 has resolved the issue that was causing us proxies to fall back to polling unnecessarily, it might be somewhat less urgent. |
Bug Report
What is the issue?
linkerd-proxy
container is using more than 35GB of memoryWe're using TLS and auto-inject, running with Linkerd2 2.2 stable
How can it be reproduced?
Run a Pod with TLS. Happens occasionally
Logs, error output, etc
linkerd-proxy container output:
Metrics output (process_* only):
(If the output is long, please create a gist and
paste the link here.)
linkerd check
outputEnvironment
Possible solution
N/A
Additional context
The text was updated successfully, but these errors were encountered: