-
Notifications
You must be signed in to change notification settings - Fork 119
instructions for es multinode + kib external access #1022
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
Conversation
deploy-manage/deploy/self-managed/_snippets/join-existing-cluster.md
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think I forgot to apply my review comments last week before your day started and I started to run tests in my afternoon :)
Sharing them now, as I think they are still relevant. Please ignore otherwise.
I'll start a new review now checking the changes we talked about the procedure itself.
deploy-manage/deploy/self-managed/_snippets/join-existing-cluster.md
Outdated
Show resolved
Hide resolved
deploy-manage/deploy/self-managed/_snippets/join-existing-cluster.md
Outdated
Show resolved
Hide resolved
deploy-manage/deploy/self-managed/install-elasticsearch-with-zip-on-windows.md
Outdated
Show resolved
Hide resolved
deploy-manage/deploy/self-managed/_snippets/clean-up-multinode.md
Outdated
Show resolved
Hide resolved
deploy-manage/deploy/self-managed/_snippets/clean-up-multinode.md
Outdated
Show resolved
Hide resolved
deploy-manage/deploy/self-managed/_snippets/reset-superuser-rpm-deb.md
Outdated
Show resolved
Hide resolved
deploy-manage/deploy/self-managed/install-elasticsearch-with-debian-package.md
Outdated
Show resolved
Hide resolved
deploy-manage/deploy/self-managed/install-elasticsearch-with-debian-package.md
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks really great. I've left a few comments for your consideration.
If you need another review let me know!
Harvested all of the content from the old install stack tutorial that was missing from the other docs.
namely:
elasticsearch-reconfigure-node
tool for RPM and debian)^ both of these were added as optional steps to the es and kibana install tutorials. I also simplified and deduplicated initial TCP config info across the install guides
Wonder if the kibana instructions should instead be part of the config step, but not sure if the correct
server.host
value needs to be present when kibana is enrolled. this preserves the order from the old tutorial (config then start).I also rounded up the ports from the old image and added them in the installation overviews - not necessarily the best spot but maybe an adequate one? took the content from a knowledge article and added the relevant settings
open to feedback about whether this is a good addition or not
closes https://github.com/elastic/docs-projects/issues/462