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

(maint) Fix some typos in the readme #72

Merged
merged 1 commit into from
Mar 26, 2018

Conversation

lucywyman
Copy link
Contributor

No description provided.

@lucywyman lucywyman requested a review from ccaum March 2, 2018 21:44
Copy link
Contributor

@ccaum ccaum left a comment

Choose a reason for hiding this comment

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

This is great! Thanks, @lucywyman! If you could just remove the link added to line 44 for kubetool, then this is good to go from my perspective.

README.md Outdated
all the security parameters, the bootstrap token, and other configurations for
your Kubernetes cluster into a Hiera file. The tool is available as a Docker
This module includes a configuration tool called
[kubetool](https://github.com/abema/kubetool) to auto generate all the security
Copy link
Contributor

Choose a reason for hiding this comment

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

Not that kubetool :)
Our kubetool is specific to the module and in the tooling/kube_tool directory.

@lucywyman
Copy link
Contributor Author

Oh my goodness, I completely forgot about this! Apologies! @ccaum fixed :)

@ccaum
Copy link
Contributor

ccaum commented Mar 24, 2018

Oh wow I forgot about this too. Thanks @lucywyman! @davejrt or @scotty-c, LGTM. Can you merge?

@scotty-c
Copy link
Contributor

LGTM

@scotty-c scotty-c merged commit 6501f3a into puppetlabs:master Mar 26, 2018
@davejrt davejrt added this to the 1.1.0 milestone Apr 15, 2018
lionce pushed a commit to lionce/puppetlabs-kubernetes that referenced this pull request Dec 19, 2019
Use https instead of ssh for the clone
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants