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: Try to improve the docs about custom CAs #1315

Merged
merged 4 commits into from Feb 8, 2019
Merged

Conversation

scholzj
Copy link
Member

@scholzj scholzj commented Feb 7, 2019

Type of change

  • Documentation

Description

After some experience with multiple users try to get custom CAs working, I prepared an minor update to the relevant section of the documentation which important things which are missing or not obvious:

  • Everything should be prepared before cluster deployment
  • Handling for Intermediate CAs and CA chains needs to be explained
  • The OpenSSL command is IMHO not needed. This is the same what CO does automatically. So why would we tell users to use that.
  • Move the kubectl label and oc label commands to separate step. Many users including me missed them because they were hidden at the end of a very long command. This should hopefully make it a bit more noticeable

Further feedback would be welcome.

Checklist

  • Update documentation

Copy link
Member

@ppatierno ppatierno left a comment

Choose a reason for hiding this comment

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

Just a minor.

Copy link
Contributor

@ncbaratta ncbaratta left a comment

Choose a reason for hiding this comment

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

ltgm

@scholzj scholzj merged commit b7fd531 into master Feb 8, 2019
@scholzj scholzj deleted the docs-custom-cas branch February 8, 2019 17:57
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.

None yet

3 participants