Skip to content
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.

Sign up for GitHub

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

Jump to bottom

ulimit settings not used with start-stop-daemon #109

Closed
ralphm opened this issue Jul 1, 2013 · 3 comments
Closed

ulimit settings not used with start-stop-daemon #109

ralphm opened this issue Jul 1, 2013 · 3 comments

Comments

@ralphm
Copy link

ralphm commented Jul 1, 2013

Pull request #91 introduced the use of start-stop-daemon to daemonize elasticsearch on Debian-family machines. However, start-stop-daemon currently does not use PAM, and thus also does not apply user-specific ulimit settings in /etc/security/limits.conf.

http://superuser.com/questions/454465/make-ulimits-work-with-start-stop-daemon gives some more context around this issue.
@chrisroberts
Copy link

@ralphm Hi! Take a quick look at #103 :)

@ralphm
Copy link
Author

ralphm commented Jul 1, 2013

@chrisroberts ah, cool, I apparently missed that ticket :-/. From the looks of it, those changes for #103 would indeed solve this issue. Thanks!

@zimbatm zimbatm mentioned this issue Jul 26, 2013
Closed
karmi pushed a commit that referenced this issue Aug 1, 2013
@chrisroberts @karmi
Run ulimit in the init script to prevent errors when /etc/security/…
8719d4f 
…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
karmi added a commit that referenced this issue Aug 1, 2013
@karmi
Merge branch 'limits' into 'master'
16b3f03 
Refactoring the way we set limits on the system:

* Use separate file in `/etc/limits.d/`
* Run `ulimit` in the init script defensively

Related issues: #118, #109, #103
@karmi
Copy link
Contributor

karmi commented Aug 1, 2013

@ralphm Should be fixed now, please see 16b3f03

@karmi karmi closed this as completed Aug 1, 2013
thoughtcroft pushed a commit to ninefold/chef-elasticsearch that referenced this issue Jan 30, 2015
@chrisroberts
Run ulimit in the init script to prevent errors when /etc/security/…
ac21241 
…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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@karmi @ralphm @chrisroberts