-
Notifications
You must be signed in to change notification settings - Fork 7
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
Out of memory after 3 months of use #32
Comments
Thanks for your report. You don't have any detailed GC statistics, or do you? My suspicion is that it is mainly the (old) TCP stack that in certain scenarios (with real-world Internet traffic) leaks memory. Unfortunately the new TCP stack has still some other issues and thus is not ready for primtle time yet. An interesting data point would be how often you updated the data in the unikernel (via the /hook url, doing a git pull on the data repository) -- if at all. (The reason behind that question is to exclude the git-client from the considerations of memory usage.) What I can say: thanks for testing, sorry that it behaves for you in that way, and we're working hard to get towards a more performant and less leaky stack. :) |
Sadly not. Is there a way to get more information when this type of critical error happens? (something like an option
No worries, whenever have an alpha version that ready to test i'll be happy to give the new stack a try.
Unless some robot used
No problem at all, thanks for creating unipi! |
actually, replying to myself here: i can simply set |
There is the |
So the metrics won't suffice (as far as I know). It is more stuff like memtrace that would be useful -- but then we bite into the Cstruct.t apple and memtrace isn't too useful for these bigarray allocations. Below the line, I think it would be great to get the new TCP stack out of the door, and spend more time on improving that. |
I deployed a unipi instance in early May, and a few days ago i tried to access it only to find this error message in the log when i realised the service was down:
Info:
The text was updated successfully, but these errors were encountered: