-
Notifications
You must be signed in to change notification settings - Fork 1
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
Enhance Readme #55
Enhance Readme #55
Conversation
fixes #54
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.
Very nice, especially the suffix rules and the mentioning of the "Contributing" chapter.
We can link it since there are anchors.
Thanks, I always wondered if it wouldn't be easier to link to sections instead of just referencing them. But to be honest, I was too lazy to look it up. 😅 TIL: Linking in Markup |
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.
Ok. Approved :-)
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.
LGTM!
* Support of SAN for etcd * Removed trailing spaces * Update README.md Co-authored-by: Thomas Widhalm <[email protected]> * Update tasks/main.yml Co-authored-by: Thomas Widhalm <[email protected]> * Update tasks/main.yml Co-authored-by: Thomas Widhalm <[email protected]> * Update README.md Co-authored-by: Thomas Widhalm <[email protected]> * Fixed syntax * Check for instance key (#52) * Check for instance key * Use new ca_client_ca_dir * Add check for server certificate fixes #51 * Enhance Readme (#55) * Enhance Readme fixes #54 * Added link to "Contributing" chapter Co-authored-by: DanOPT <[email protected]> * Added tasks to create additional peer and server certificates for etcd * Resolved merge conflict of README.md * Resolved tasks/main.yml * Removed trailing spaces * Updated README.md * Update README.md * Added first part of molecule verification * Finished molecule verification tasks * Get IP addresses of nodes by group name for etcd certificates * Fixed trailing spaces * Fixed groups in molecule.yml * Converted groups to array * Added packages and start Networkmanager for EL in prepare.yml * Added empty line EOF to prepare.yml * Changed volume to rw instead r * Added cgroupns_mode to molecule.yml * Changed actions to v3 * Removed CentOS 7 from molecule tests * Improved default(omit) to default(omit, true) Co-authored-by: Thomas Widhalm <[email protected]>
I hope to make more clear what the role is about and how to add more functionality.
fixes #54