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

added kubelet args key #13

Closed
wants to merge 0 commits into from
Closed

Conversation

carpenike
Copy link
Contributor

I think I got them all, let me know if I missed any.

Signed-off-by: Ryan Holt ryan@ryanholt.net

@carpenike
Copy link
Contributor Author

Closes #12

@xanmanning xanmanning self-requested a review February 27, 2020 09:17
@xanmanning xanmanning self-assigned this Feb 27, 2020
@xanmanning xanmanning added the enhancement New feature or request label Feb 27, 2020
@xanmanning
Copy link
Member

Thanks @carpenike - looks like it is covered in terms of how to implement it. I shall test it on my VM cluster.

Any chance an example usage can be put in the README.md somewhere?, eg. under a heading "Important note about k3s_kubelet_args" ?

I may adjust the .travis-ci.yml file for testing this before merging this in, hopefully this evening (GMT+0), I'll add it to this branch.

@carpenike
Copy link
Contributor Author

carpenike commented Feb 27, 2020

Yup that makes sense. My particular use case is for integration with a a cloud provider. IE: https://itnext.io/how-to-integrate-k3s-with-the-cloud-controller-36bd5020b8f7

I’ll put something into the readme today.

@xanmanning
Copy link
Member

Balls, I did not mean to do that.

@xanmanning
Copy link
Member

My sincerest apologies @carpenike - I shall recreate this.

@xanmanning xanmanning mentioned this pull request Feb 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants