Skip to content

Update bootstrap-checks.asciidoc#48975

Merged
jrodewig merged 2 commits intoelastic:masterfrom
daixiang0:patch-1
Nov 27, 2019
Merged

Update bootstrap-checks.asciidoc#48975
jrodewig merged 2 commits intoelastic:masterfrom
daixiang0:patch-1

Conversation

@daixiang0
Copy link
Copy Markdown
Contributor

@daixiang0 daixiang0 commented Nov 12, 2019

Comment how to update max size of virtual memory more clearly.

Signed-off-by: Xiang Dai 764524258@qq.com

Comment how to update max size if virtual memory more clearly.

Signed-off-by: Xiang Dai <764524258@qq.com>
@alpar-t alpar-t added :Docs >docs General docs changes labels Nov 12, 2019
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-docs (>docs)

@jrodewig jrodewig added the :Core/Infra/Core Core issues without another label label Nov 12, 2019
@elasticmachine
Copy link
Copy Markdown
Collaborator

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

@jrodewig jrodewig removed the :Docs label Nov 12, 2019
@daixiang0
Copy link
Copy Markdown
Contributor Author

any update? @elastic/es-core-infra

@jrodewig
Copy link
Copy Markdown
Contributor

@elasticmachine test this please.

@jrodewig
Copy link
Copy Markdown
Contributor

Thanks for your contribution @daixiang0. I left some suggestions for your consideration.

After you address them, we can work on merging these changes in.

Signed-off-by: Xiang Dai <764524258@qq.com>
@daixiang0
Copy link
Copy Markdown
Contributor Author

@jrodewig done.

@daixiang0 daixiang0 requested a review from jrodewig November 27, 2019 10:48
@jrodewig
Copy link
Copy Markdown
Contributor

@elasticmachine test this please

Copy link
Copy Markdown
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 @daixiang0. I'll get this merged and backported.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Core/Infra/Core Core issues without another label >docs General docs changes v7.4.3 v7.5.1 v7.6.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants