Skip to content
This repository has been archived by the owner on Jun 29, 2022. It is now read-only.

Add a knob to disable self hosted kubelet #425

Merged
merged 4 commits into from
May 15, 2020

Conversation

surajssd
Copy link
Member

No description provided.

pkg/platform/aws/aws.go Outdated Show resolved Hide resolved
pkg/platform/packet/packet.go Outdated Show resolved Hide resolved
@invidian
Copy link
Member

I think we also need to document, that disabling self-hosted kubelet makes it difficult to upgrade, as kubelet needs to be upgraded manually for every node.

@surajssd surajssd force-pushed the surajssd/add-knob-to-disable-self-hosted-kubelet branch 2 times, most recently from 4505b70 to 2be6c79 Compare May 14, 2020 08:10
@surajssd surajssd marked this pull request as ready for review May 14, 2020 08:10
@surajssd
Copy link
Member Author

@invidian

I think we also need to document, that disabling self-hosted kubelet makes it difficult to upgrade, as kubelet needs to be upgraded manually for every node.

Filed #429

@surajssd surajssd force-pushed the surajssd/add-knob-to-disable-self-hosted-kubelet branch 13 times, most recently from 73ebf14 to 7ce11c7 Compare May 15, 2020 13:31
@surajssd surajssd requested a review from invidian May 15, 2020 14:30
Copy link
Member

@invidian invidian left a comment

Choose a reason for hiding this comment

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

Just some styling nits, otherwise looks good.

This flag adds a variable `disable_self_hosted_kubelet` to bootkube so
that installation of self hosted kubelet can be disabled.

Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
This commit adds a boolean variable to controller config called
`disable_self_hosted_kubelet`.

Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
This commit adds a boolean variable to controller config called
`disable_self_hosted_kubelet`.

Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
This commit adds a boolean variable to controller config called
`disable_self_hosted_kubelet`.

Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
@surajssd surajssd force-pushed the surajssd/add-knob-to-disable-self-hosted-kubelet branch from 7ce11c7 to 101934d Compare May 15, 2020 16:27
@surajssd surajssd requested a review from invidian May 15, 2020 16:50
@surajssd surajssd merged commit 9e3eaf6 into master May 15, 2020
@surajssd surajssd deleted the surajssd/add-knob-to-disable-self-hosted-kubelet branch May 15, 2020 21:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants