Skip to content

Clarify limits.conf usage in Ubuntu#5565

Open
eedugon wants to merge 3 commits intomainfrom
eedugon-patch-5
Open

Clarify limits.conf usage in Ubuntu#5565
eedugon wants to merge 3 commits intomainfrom
eedugon-patch-5

Conversation

@eedugon
Copy link
Contributor

@eedugon eedugon commented Mar 19, 2026

The problem mentioned in the admonition does not happen anymore, so it was probably occurring in older versions that got patched.

Currently the mentioned line is already uncommented in Ubuntu by default, so I propose to change the sentence for the user to be aware of the possibility of limits.conf being ignored due to this in some cases.

I've also removed the mention to init.d because init.d scripts do not apply limits.conf, as they do not go through PAM. They start processes directly (typically via start-stop-daemon). Also with modern systemd services limits.conf does not apply either.

The problem mentioned in the admonition does not happen anymore, so it was probably occurring in older versions that got patched.

Currently the mentioned line is already uncommented in Ubuntu by default, so I propose to change the sentence for the user to be aware of the possibility of limits.conf being ignored due to this.
@eedugon eedugon requested a review from a team as a code owner March 19, 2026 11:49
@github-actions
Copy link
Contributor

github-actions bot commented Mar 19, 2026

✅ Vale Linting Results

No issues found on modified lines!


The Vale linter checks documentation changes against the Elastic Docs style guide.

To use Vale locally or report issues, refer to Elastic style guide for Vale.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 19, 2026

🔍 Preview links for changed docs

Copy link
Contributor

@kilfoyle kilfoyle left a comment

Choose a reason for hiding this comment

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

LGTM! 🐅

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.

2 participants