Skip to content

[DOCS] Updates TLS tutorial for use with basic licenses and new cluster formation settings #358

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

Merged
merged 7 commits into from
Jun 19, 2019

Conversation

lcawl
Copy link
Contributor

@lcawl lcawl commented Jun 6, 2019

This PR updates the TLS tutorial (https://www.elastic.co/guide/en/elastic-stack-overview/master/encrypting-internode-communications.html) such that it:

  1. No longer uses out-dated zen settings
  2. Changes the order of the steps such that they work for nodes with basic licenses.
  3. Re-uses some existing content via tagged sections so that we don't have to maintain the same information in multiple places.

Copy link
Contributor

@DaveCTurner DaveCTurner left a comment

Choose a reason for hiding this comment

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

I suggested a change re. cluster.initial_master_nodes. The details of setting up security are outside what I can definitively review, so I'll leave that to Tim.

Copy link
Contributor

@tvernum tvernum left a comment

Choose a reason for hiding this comment

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

LGTM, with a couple of minor points.

. (Optional) Restart {kib}. For example, if you installed
{kib} with a `.tar.gz` package, run the following command from the {kib}
directory:
. Create passwords for the built-in users and .
Copy link
Contributor

Choose a reason for hiding this comment

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

This sentence finished halfway through.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for catching that!

@lcawl lcawl merged commit 65bb58e into elastic:master Jun 19, 2019
@lcawl lcawl deleted the tutorial-zen branch June 19, 2019 23:53
lcawl added a commit that referenced this pull request Jun 20, 2019
lcawl added a commit that referenced this pull request Jun 20, 2019
lcawl added a commit that referenced this pull request Jun 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants