Skip to content

Set default max memory usage for all node.js processes on CircleCi #5478

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

Merged
merged 2 commits into from
Feb 8, 2021

Conversation

archmoj
Copy link
Contributor

@archmoj archmoj commented Feb 8, 2021

@archmoj archmoj added this to the NEXT milestone Feb 8, 2021
@archmoj
Copy link
Contributor Author

archmoj commented Feb 8, 2021

@archmoj
Copy link
Contributor Author

archmoj commented Feb 8, 2021

Also FYI - on a dev branch I also tried setting NODE_OPTIONS in config.yml environments; but that did not help.
So I think 071d76e is the best we could do for now to set this before very first node.js memory allocation.

@archmoj archmoj changed the title Set default max memory usage for all node.js processes Set default max memory usage for all node.js processes on CircleCi Feb 8, 2021
@alexcjohnson
Copy link
Collaborator

Wow, interesting, I would not have thought changing something like that in the build job would affect later jobs. I also had never heard of the word "swappiness" before. So many new things to learn!

💃

@archmoj archmoj merged commit 33c396f into master Feb 8, 2021
@archmoj archmoj deleted the set-node-space-all branch February 8, 2021 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants