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

chore: add kube scheduler server args & plumb through to config #54

Merged
merged 1 commit into from
Oct 3, 2023

Conversation

lukebond
Copy link
Contributor

@lukebond lukebond commented Oct 2, 2023

i wanted to set some args on k3s using --kube-scheduler-arg (see docs) but found that it wasn't exposed. in this PR i believe i have exposed that flag such that it will get plumbed through into the config file pushed onto the control plane hosts. i wasn't sure how to go about testing it so i'm hoping you can give me some pointers, or approve if you know it to be okay.

Copy link
Contributor

@mkmik mkmik left a comment

Choose a reason for hiding this comment

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

LGTM!

@zawachte
Copy link
Collaborator

zawachte commented Oct 2, 2023

looks good. thanks!

@zawachte zawachte merged commit e4ea03a into k3s-io:main Oct 3, 2023
3 checks passed
@lukebond
Copy link
Contributor Author

lukebond commented Oct 3, 2023

thank you. is it worth a release for this? is that just a matter of adding a tag i think? anything i can do to help LMK

@zawachte
Copy link
Collaborator

zawachte commented Oct 3, 2023

thank you. is it worth a release for this? is that just a matter of adding a tag i think? anything i can do to help LMK

Just created a release. Let's see how it goes ... I know that @ekarlso was still tinkering.

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.

3 participants