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

Update docker.asciidoc #47651

Merged
merged 1 commit into from
Oct 19, 2019
Merged

Update docker.asciidoc #47651

merged 1 commit into from
Oct 19, 2019

Conversation

harsha-s
Copy link
Contributor

@harsha-s harsha-s commented Oct 7, 2019

added nofile section in ulimits to fix below error:
es02 | ERROR: [1] bootstrap checks failed
es02 | [1]: max file descriptors [4096] for elasticsearch process is too low, increase to at least [65535]

  • Have you signed the contributor license agreement?
  • Have you followed the contributor guidelines?
  • If submitting code, have you built your formula locally prior to submission with gradle check?
  • If submitting code, is your pull request against master? Unless there is a good reason otherwise, we prefer pull requests against master and will backport as needed.
  • If submitting code, have you checked that your submission is for an OS and architecture that we support?
  • If you are submitting this code for a class then read our policy for that.

added nofile section in ulimits to fix below error:
es02 | ERROR: [1] bootstrap checks failed
es02 | [1]: max file descriptors [4096] for elasticsearch process is too low, increase to at least [65535]
@polyfractal polyfractal added the :Delivery/Packaging RPM and deb packaging, tar and zip archives, shell and batch scripts label Oct 15, 2019
@elasticmachine
Copy link
Collaborator

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

@rjernst
Copy link
Member

rjernst commented Oct 15, 2019

It looks like this is almost the same as #47650 but slightly different docs pages. I seem to recall some recent issue or work around this as well, but can't find it. @pugnascotia Can you please decide how to proceed here?

@pugnascotia
Copy link
Contributor

@rjernst I think you're referring to PR #47715. I think we can merge this PR, and @debadair can resolve the resulting conflict / merge in the ulimit changes when she picks up the #47715 again.

@rjernst
Copy link
Member

rjernst commented Oct 16, 2019

Yes that's the one! ++ to merge

pugnascotia
pugnascotia previously approved these changes Oct 19, 2019
@pugnascotia
Copy link
Contributor

@elasticmachine test this please

@pugnascotia pugnascotia dismissed their stale review October 19, 2019 14:15

Forgot to run CI first!

@pugnascotia pugnascotia merged commit f9227da into elastic:master Oct 19, 2019
@mark-vieira mark-vieira added the Team:Delivery Meta label for Delivery team label Nov 11, 2020
@elasticmachine elasticmachine added the Team:Docs Meta label for docs team label Jul 27, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-docs (Team:Docs)

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 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants