-
-
Notifications
You must be signed in to change notification settings - Fork 605
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
Source defaults from init if found #103
Source defaults from init if found #103
Conversation
- clean old limits - use ulimit for all flavors
|
this is fine with me |
Folks, when will this be merged? I have a broken production, it dies randomly, and looks like this is the reason (see gist: https://gist.github.com/maxim/26278d8bc31e1c8d905b). If it isn't ready to merge, is there a workaround? |
For anyone who needs to workaround this issue right now, before it's merged — follow this gist. It implies that you have a wrapper cookbook or that you can put additional cookbook in your runlist containing this fix. |
@maxim I'd be very surprised if limits would be responsible for the |
…limits.d is ignored This follows the conventions in the official .deb and .rpm packages from elastic/elasticsearch#2059. See: * https://github.com/elasticsearch/elasticsearch/blob/master/src/deb/init.d/elasticsearch#L148-L154 * https://github.com/elasticsearch/elasticsearch/blob/master/src/rpm/init.d/elasticsearch#L75-L83 This is part of a larger changeset by @chrisroberts in #103. Closes: #103 Closes: #109 Related: #101 Related: #102
So, I've created a mashup of @chrisroberts and @ctrabolds versions, so now we have a sane approach to |
@karmi saw your mention on IRC as well. I ran ES on a machine with this configuration for months without problems, and all I did was rebuild identical-config'ed machine with updated ES. As soon as I applied my workaround - the crashing issue was gone, and since then (15 days ago) it's been running perfectly. I'm surprised that you say limits have nothing to do with it, since the docs for
|
…limits.d is ignored This follows the conventions in the official .deb and .rpm packages from elastic/elasticsearch#2059. See: * https://github.com/elasticsearch/elasticsearch/blob/master/src/deb/init.d/elasticsearch#L148-L154 * https://github.com/elasticsearch/elasticsearch/blob/master/src/rpm/init.d/elasticsearch#L75-L83 This is part of a larger changeset by @chrisroberts in sous-chefs#103. Closes: sous-chefs#103 Closes: sous-chefs#109 Related: sous-chefs#101 Related: sous-chefs#102
No description provided.