Join GitHub today
GitHub is home to over 31 million developers working together to host and review code, manage projects, and build software together.
Sign upAuto set memory-chunk base on the available free memory detected #2188
Comments
beorn7
added
the
reviewed/duplicate
label
Nov 14, 2016
This comment has been minimized.
This comment has been minimized.
|
This is less trivial than suggested here. |
beorn7
closed this
Nov 14, 2016
This comment has been minimized.
This comment has been minimized.
lock
bot
commented
Mar 24, 2019
|
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
lock
bot
locked and limited conversation to collaborators
Mar 24, 2019
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
netroby commentedNov 14, 2016
•
edited
We know we can using golang runtime package to get memstats information. so we know the free memory available.
Now the default
storage.local.memory-chunksbe 1024*1024. It not good if the installed target machine have not enough memory. like a VM with only 512MB memory.prometheus/cmd/prometheus/config.go
Line 117 in 6703404
So would we please add the auto detected memory?
We should not using more then 85% availble free memory. or we will got OOM .