-
-
Notifications
You must be signed in to change notification settings - Fork 32
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
Potential memory leak #423
Comments
Hello @armin-malek , Thanks for your issue. I never used this provider so I can't say if it is coming from this or not.
Regards, |
we have close to 8 million files in the bucket |
I think I have an idea of why. People requesting different files all the time ? Or are they the same for multiple minutes ? |
it's like an image hosting platform so yeah it should be mostly random objects |
Ok so I think I know why. This must come from prometheus metrics which are saving metrics per "request path". This can be patched for sure. |
I'm not familiar with the code base but maybe an option to disable the metrics in the config file should do the trick |
I've just released a new version. The 4.13.0 release have a configuration dedicated for this. Here is an example: # Metrics configuration
metrics:
# Disable router path save in HTTP metrics
disableRouterPath: true That should fix your problem. PS: Out of this, do you use s3-proxy to make a tile map server ? |
thanks, it has solved the problem!
|
out of this too |
Cool to hear this ! That's great! Yes that's a feature to log everything. You can change the log level to error if you want only errors. All log levels from uber/zap logger are supported. |
This issue is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 10 days |
after startup, it slowly starts using more memory, in a few hours it reaches over 1GB
because it works just fine with much less memory usage at startup, I think it's caused by a memory leak
To Reproduce
here is the config file I'm using my provider is Idrive e2
Expected behavior
less and stable ram usage
Version and platform (please complete the following information):
The text was updated successfully, but these errors were encountered: