Skip to content

Update documentation for systemd configuration#26950

Merged
jasontedor merged 1 commit intoelastic:masterfrom
nicosierra:patch-1
Oct 10, 2017
Merged

Update documentation for systemd configuration#26950
jasontedor merged 1 commit intoelastic:masterfrom
nicosierra:patch-1

Conversation

@nicosierra
Copy link
Copy Markdown

The documentation to override limits for the RPM/ Debian packages on systems that use systemd is not accurate. Link

I spend quite a long time figuring out how to override them, since by defining a file called elasticsearch.conf I was not having the limits overriden.

As a matter of fact, when running sudo systemctl edit elasticsearch, the override file is actually created as override.conf.

@elasticmachine
Copy link
Copy Markdown
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@jasontedor jasontedor changed the base branch from 5.6 to master October 10, 2017 17:03
@jasontedor
Copy link
Copy Markdown
Member

Thanks @nicosierra, I rebased your commit on master and changed the base branch of this PR to master. I will pull this in shortly.

Copy link
Copy Markdown
Member

@jasontedor jasontedor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@jasontedor jasontedor merged commit d6fc4af into elastic:master Oct 10, 2017
jasontedor pushed a commit that referenced this pull request Oct 10, 2017
This commit clarifies how to apply an override to the systemd unit file
for Elasticsearch.

Relates #26950
jasontedor pushed a commit that referenced this pull request Oct 10, 2017
This commit clarifies how to apply an override to the systemd unit file
for Elasticsearch.

Relates #26950
jasontedor pushed a commit that referenced this pull request Oct 10, 2017
This commit clarifies how to apply an override to the systemd unit file
for Elasticsearch.

Relates #26950
@jasontedor
Copy link
Copy Markdown
Member

Thanks @nicosierra.

@nicosierra
Copy link
Copy Markdown
Author

Thanks for this project @jasontedor!

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.

3 participants