-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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
Default NodeJS max heap limit to 250MB. Override by setting NODE_OPTIONS #5451
Conversation
Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run; then say 'jenkins, test it'. |
1 similar comment
Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run; then say 'jenkins, test it'. |
I've just signed the CLA - not sure how to kick off the check aout |
@mrdavidlaing I'm unable to find your CLA signature. Would you mind submitting another? |
Can you add this to bin/kibana.bat too please? |
@spalger - Here is the signed CLA from my company. |
Ah, business signatures are a little more complex. Thanks for that! |
…NS - Windows support
Okay, thinking about this a second time it feels wrong to set the value to anything by default. I'm thinking we should just inject NODE_OPTIONS so that we can use node's (probably intelligent) default but let others pick a value that's better for them. |
+1, I think we should keep node's default, and document the fact it exists and how to lower it. |
+1 |
Can someone please resolve and merge this pull request? I have been quite anxiously waiting for this fix. It is quite annoying that I have to start my kibana docker container everyday after the OOM Killer kills it. How can I help? |
@@ -21,5 +21,6 @@ if [ ! -x "$NODE" ]; then | |||
exit 1 | |||
fi | |||
|
|||
exec "${NODE}" "${DIR}/src/cli" ${@} | |||
NODE_OPTIONS="${NODE_OPTIONS:=--max-old-space-size=250}" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove this line, we decided not to set a default.
Feature implemented in #5598 |
@w33ble - Thanks for following up and merging a similar fix. Sorry for the radio silence. |
This is great, I was just looking for a clean way to run node in debug mode when using our npm scripts and grunt tasks that execute bin/kibana. This fix works great for that use case as well. |
If you have to optimize resources then node runs out of memory ( |
Did you guys really just ship a broken systemd service in your release?
|
Also, not only is that an invalid comment, the option doesn't work anyhow....
|
@ageis could you open a new issue for the problems you're having with the service? It'll help us track a fix. I'm guessing that's a problem in 5.4.0, but please also note the version you're having problems with in the new issue. |
@ageis, our |
@tylersmalley Yes. I had a previous installation of Kibana 4 (when it was in
I removed everything related to that installation a long time ago though and had been running cleanly on version 5 for quite a while. |
Relates to #5170 - this limits NodeJS to using only 250MB of heap before the garbage collector kicks in - useful in low memory environments.
To override, set the
NODE_OPTIONS
environment variable, eg, to allow Kibana to use 512MB of heap, set