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.

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

[DOCS] Fix ulimit value in system settings docs #56973

Merged
merged 1 commit into from
May 20, 2020

Conversation

eddieturizo
Copy link
Contributor

As part of #35839 and #37537 the limit was lowered to 65535, but the configuring.asiidoc still states 65536 instead of 65535. Would it be possible to backport this to at least 7.x?

Make sure that the limits match in the documentation, across lines 44 and 49.
@eddieturizo eddieturizo added the >docs General docs changes label May 19, 2020
@eddieturizo eddieturizo requested a review from a team May 19, 2020 21:38
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs (>docs)

@elasticmachine elasticmachine added the Team:Docs Meta label for docs team label May 19, 2020
@jrodewig jrodewig added the :Delivery/Packaging RPM and deb packaging, tar and zip archives, shell and batch scripts label May 20, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (:Core/Infra/Packaging)

@elasticmachine elasticmachine added the Team:Core/Infra Meta label for core/infra team label May 20, 2020
@jrodewig jrodewig added v7.7.1 v7.8.1 v7.9.0 v8.0.0 and removed Team:Core/Infra Meta label for core/infra team labels May 20, 2020
Copy link
Contributor

@jrodewig jrodewig left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks @eddieturizo

@jrodewig jrodewig changed the title Fix ulimit value discrepancy in documentation [DOCS] Fix ulimit value in system settings docs May 20, 2020
@jrodewig jrodewig merged commit 9c97e55 into master May 20, 2020
@jrodewig jrodewig deleted the eddieturizo-limits-conf-update branch May 20, 2020 12:43
jrodewig pushed a commit that referenced this pull request May 20, 2020
Correctly documents the ulimit value as 65535.

The limit was lowered as part of #37537.
jrodewig pushed a commit that referenced this pull request May 20, 2020
Correctly documents the ulimit value as 65535.

The limit was lowered as part of #37537.
jrodewig pushed a commit that referenced this pull request May 20, 2020
Correctly documents the ulimit value as 65535.

The limit was lowered as part of #37537.
jrodewig pushed a commit that referenced this pull request May 20, 2020
Correctly documents the ulimit value as 65535.

The limit was lowered as part of #37537.
jrodewig pushed a commit that referenced this pull request May 20, 2020
Correctly documents the ulimit value as 65535.

The limit was lowered as part of #37537.
@jrodewig
Copy link
Contributor

I've backported this to all our currently maintained branches.

Backport commits

master 9c97e55
7.x 27673aa
7.8 e42b050
7.7 180ad17
7.6 dfc7fd4
6.8 80b6b27

@mark-vieira mark-vieira added the Team:Delivery Meta label for Delivery team label Nov 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Delivery/Packaging RPM and deb packaging, tar and zip archives, shell and batch scripts >docs General docs changes Team:Delivery Meta label for Delivery team Team:Docs Meta label for docs team v6.8.10 v7.6.3 v7.7.1 v7.8.1 v7.9.0 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants