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

kube-apiserver templates #3247

Closed
chrislovecnm opened this issue Aug 21, 2017 · 4 comments
Closed

kube-apiserver templates #3247

chrislovecnm opened this issue Aug 21, 2017 · 4 comments
Labels
good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines.

Comments

@chrislovecnm
Copy link
Contributor

kube-apiserver is the only non-addon where we are still using templates. We need to move all of the templating into code.

All of this templating fun should be moved to logical code: https://github.com/kubernetes/kops/tree/master/upup/models/config/components/kube-apiserver

@georgebuckerfield
Copy link
Contributor

I started some work on this here:
master...georgebuckerfield:refactor-apiserver-templates

Am I going in the right direction with it? Is there anything else that I should be looking at?

@chrislovecnm
Copy link
Contributor Author

That is awesome. @andrewsykim did you need anything specific for DO?

@andrewsykim
Copy link
Member

andrewsykim commented Aug 26, 2017

@georgebuckerfield thanks for getting started on this! For DO, the cloud provider passed into kops doesn't correlate to the cloud provider passed into kubernetes components. If the cloud provider is DO the --cloud-provider flags should be --cloud-provider=external since DO will depend on out-of-tree cloud providers.

Example: https://github.com/kubernetes/kops/blob/master/pkg/model/components/kubecontrollermanager.go#L107-L108

@chrislovecnm
Copy link
Contributor Author

Probably get this fix in and then do the DO change.

Please get a PR in!

k8s-github-robot pushed a commit that referenced this issue Aug 27, 2017
…mplates

Automatic merge from submit-queue

Refactor apiserver templates

Fixes #3247 by moving the apiserver configuration out of template files and into code.
@rifelpet rifelpet added good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines. and removed good-starter-issue labels Apr 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Denotes an issue ready for a new contributor, according to the "help wanted" guidelines.
Projects
None yet
Development

No branches or pull requests

4 participants