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

Implementing kubelet GC settings in kops #1243

Closed
markine opened this issue Dec 22, 2016 · 6 comments
Closed

Implementing kubelet GC settings in kops #1243

markine opened this issue Dec 22, 2016 · 6 comments

Comments

@markine
Copy link

markine commented Dec 22, 2016

I'm looking to experiment with garbage collection on a cluster created by kops. I need to specify custom flags for kubelet (/etc/sysconfig/kubelet). Two issues:

How do I edit the Kubelet part of the cluster spec? I tried manually downloading cluster.spec from S3, editing it, and uploading it back. 'kube update' did not recognize any changes made to the kubelet stanza.

What needs to happen for kops to process the GC flags? The GC config flags are here: http://kubernetes.io/docs/admin/garbage-collection/. These settings are commented out of the kubelet spec in kops:

ImageGCHighThresholdPercent :

//ImageGCHighThresholdPercent int32 `json:"imageGCHighThresholdPercent"`

ImageGCLowThresholdPercent:

//ImageGCLowThresholdPercent int32 `json:"imageGCLowThresholdPercent"`

Thank you.

@chrislovecnm
Copy link
Contributor

Take a look at #1083

Also take a look at

@chrislovecnm
Copy link
Contributor

The API docs under developer, also I cannot find the issue for the process that we are working on how to contribute

@chrislovecnm
Copy link
Contributor

@chrislovecnm
Copy link
Contributor

#1194 <- start on a howto contribute.

Also we have dev office hours tomorrow, see the readme

@markine
Copy link
Author

markine commented Dec 23, 2016

@chrislovecnm Thank you for a fast reply and the info!

Could you please answer the question re: expected method of modifying the full cluster.spec? I could not get 'kops update cluster' to pick up changes to existing fields in the spec, albeit I only tried one or two fields.

@chrislovecnm
Copy link
Contributor

@markine you are trying to update the api, need api values, and need to regenerate the api. Feel free to push a PR and we can comment on the code!

@justinsb justinsb modified the milestone: 1.5.0 Dec 28, 2016
justinsb added a commit to justinsb/kops that referenced this issue Jan 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants