Skip to content

Commit

Permalink
kubelet: don't specify hostname-override if cloudprovider is set to aws
Browse files Browse the repository at this point in the history
This is only applicable to <1.27 kubelet since 1.27 removes the aws
in-tree cloud provider.
  • Loading branch information
etungsten committed Nov 8, 2023
1 parent dbf9ed1 commit c6ae2a0
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 0 deletions.
2 changes: 2 additions & 0 deletions packages/kubernetes-1.23/kubelet-exec-start-conf
Original file line number Diff line number Diff line change
Expand Up @@ -27,9 +27,11 @@ ExecStart=/usr/bin/kubelet \
--image-credential-provider-config /etc/kubernetes/kubelet/credential-provider-config.yaml \
{{/if}}
{{/if}}
{{#unless (eq settings.kubernetes.cloud-provider "aws")}}
{{#if settings.kubernetes.hostname-override}}
--hostname-override {{settings.kubernetes.hostname-override}} \
{{/if}}
{{/unless}}
--node-ip ${NODE_IP} \
--node-labels "${NODE_LABELS}" \
--register-with-taints "${NODE_TAINTS}" \
Expand Down
2 changes: 2 additions & 0 deletions packages/kubernetes-1.24/kubelet-exec-start-conf
Original file line number Diff line number Diff line change
Expand Up @@ -26,9 +26,11 @@ ExecStart=/usr/bin/kubelet \
--image-credential-provider-config /etc/kubernetes/kubelet/credential-provider-config.yaml \
{{/if}}
{{/if}}
{{#unless (eq settings.kubernetes.cloud-provider "aws")}}
{{#if settings.kubernetes.hostname-override}}
--hostname-override {{settings.kubernetes.hostname-override}} \
{{/if}}
{{/unless}}
--node-ip ${NODE_IP} \
--node-labels "${NODE_LABELS}" \
--register-with-taints "${NODE_TAINTS}" \
Expand Down
2 changes: 2 additions & 0 deletions packages/kubernetes-1.25/kubelet-exec-start-conf
Original file line number Diff line number Diff line change
Expand Up @@ -26,9 +26,11 @@ ExecStart=/usr/bin/kubelet \
--image-credential-provider-config /etc/kubernetes/kubelet/credential-provider-config.yaml \
{{/if}}
{{/if}}
{{#unless (eq settings.kubernetes.cloud-provider "aws")}}
{{#if settings.kubernetes.hostname-override}}
--hostname-override {{settings.kubernetes.hostname-override}} \
{{/if}}
{{/unless}}
--node-ip ${NODE_IP} \
--node-labels "${NODE_LABELS}" \
--register-with-taints "${NODE_TAINTS}" \
Expand Down
2 changes: 2 additions & 0 deletions packages/kubernetes-1.26/kubelet-exec-start-conf
Original file line number Diff line number Diff line change
Expand Up @@ -26,9 +26,11 @@ ExecStart=/usr/bin/kubelet \
--image-credential-provider-config /etc/kubernetes/kubelet/credential-provider-config.yaml \
{{/if}}
{{/if}}
{{#unless (eq settings.kubernetes.cloud-provider "aws")}}
{{#if settings.kubernetes.hostname-override}}
--hostname-override {{settings.kubernetes.hostname-override}} \
{{/if}}
{{/unless}}
--node-ip ${NODE_IP} \
--node-labels "${NODE_LABELS}" \
--register-with-taints "${NODE_TAINTS}" \
Expand Down

0 comments on commit c6ae2a0

Please sign in to comment.