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
fix max elasticsearch memory #2155
Conversation
Signed-off-by: Timothy Ehlers <tehlers@expedia.com>
omnibus/files/private-chef-cookbooks/private-chef/libraries/elasticsearch.rb
Outdated
Show resolved
Hide resolved
Signed-off-by: Timothy Ehlers <tehlers@expedia.com>
152a128
to
1e8ab49
Compare
Added a unit test for this. |
Signed-off-by: Timothy Ehlers <tehlers@expedia.com>
1e8ab49
to
73bc68f
Compare
@tehlers320 This looks good. (edit: nvm, read the original bug report and it answered my question) |
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.
Thanks for the fix and adding these unit tests!
omnibus/files/private-chef-cookbooks/private-chef/libraries/elasticsearch.rb
Outdated
Show resolved
Hide resolved
omnibus/files/private-chef-cookbooks/private-chef/libraries/elasticsearch.rb
Outdated
Show resolved
Hide resolved
Signed-off-by: Timothy Ehlers <tehlers@expedia.com>
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.
Thank you very much for your fix!
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.
This looks great! Thanks for adding a unit test for this. Apparently our test machines don't have 256GB of ram :-).
Just FYI, this is good to merge but we are currently investigating a sqitch+perl issue in the build pipeline so are holding off on merging things until we resolve that. |
rebased and merged from #2159. Thank you for your contribution @tehlers320! |
Signed-off-by: Timothy Ehlers tehlers@expedia.com
Description
reconfigure fails on big servers.
Issues Resolved
#2154
Check List
Aha! Link: https://chef.aha.io/features/SH-434