refactor: use go template for kubelet systemd config #2320
refactor: use go template for kubelet systemd config #2320
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2320 +/- ##
=========================================
Coverage ? 71.73%
=========================================
Files ? 142
Lines ? 24993
Branches ? 0
=========================================
Hits ? 17928
Misses ? 5934
Partials ? 1131 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
KUBELET_REGISTER_NODE=--register-node=true | ||
KUBELET_REGISTER_WITH_TAINTS=--register-with-taints=node-role.kubernetes.io/master=true:NoSchedule | ||
{{end}} | ||
{{else}} | ||
KUBELET_REGISTER_SCHEDULABLE={{WrapAsVariable "registerSchedulable"}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was this cruft? I don't see it referenced anywhere else.
@@ -367,13 +362,9 @@ MASTER_CONTAINER_ADDONS_PLACEHOLDER | |||
{{if IsAzureStackCloud }} | |||
AZURE_ENVIRONMENT_FILEPATH=/etc/kubernetes/azurestackcloud.json | |||
{{end}} | |||
{{if IsKubernetesVersionGe "1.6.0"}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
K8s 1.6.0 was like a hundred years ago.
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jackfrancis, mboersma The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Reason for Change:
Move kubelet systemd configuration to go templating to be more maintainable, flexible, and agile.
Issue Fixed:
Requirements:
Notes: