-
Notifications
You must be signed in to change notification settings - Fork 181
Specify heap size cap at 32GB for instances >= 64GB #3936
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
Specify heap size cap at 32GB for instances >= 64GB #3936
Conversation
kilfoyle
left a comment
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.
LGTM! 🦖
Thanks @claudia-correia!
florent-leborgne
left a comment
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.
LGTM 🚢
|
@kilfoyle @florent-leborgne Thanks for the review! Just FYI, I'm holding off merging this PR until the changes in https://github.com/elastic/cloud/pull/148914 have been released 🙂 |
|
The changes are now in Production, so this can be merged ✅ |
Vale Linting ResultsSummary: 2 warnings, 6 suggestions found
|
| File | Line | Rule | Message |
|---|---|---|---|
| troubleshoot/deployments/cloud-enterprise/node-bootlooping.md | 109 | Elastic.Latinisms | Latin terms and abbreviations are a common source of confusion. Use 'using' instead of ' via '. |
| troubleshoot/monitoring/node-bootlooping.md | 143 | Elastic.Latinisms | Latin terms and abbreviations are a common source of confusion. Use 'using' instead of ' via '. |
💡 Suggestions (6)
| File | Line | Rule | Message |
|---|---|---|---|
| deploy-manage/deploy/elastic-cloud/ec-customize-deployment-components.md | 34 | Elastic.Acronyms | 'JVM' has no definition. |
| deploy-manage/deploy/elastic-cloud/ec-customize-deployment-components.md | 34 | Elastic.Acronyms | 'SSD' has no definition. |
| troubleshoot/deployments/cloud-enterprise/node-bootlooping.md | 109 | Elastic.Acronyms | 'OOM' has no definition. |
| troubleshoot/deployments/cloud-enterprise/node-bootlooping.md | 109 | Elastic.Acronyms | 'OOM' has no definition. |
| troubleshoot/monitoring/node-bootlooping.md | 143 | Elastic.Acronyms | 'OOM' has no definition. |
| troubleshoot/monitoring/node-bootlooping.md | 143 | Elastic.Acronyms | 'OOM' has no definition. |
This PR updates the docs to specify the heap size cap at 32GB for instances with >= 64GB of RAM.
Rel: https://elasticco.atlassian.net/browse/CP-4136