Skip to content

Commit e575a0a

Browse files
authored
Do not set --hostname-override on kube-proxy (#4360) (#4365)
1 parent 62137ed commit e575a0a

File tree

1 file changed

+7
-13
lines changed

1 file changed

+7
-13
lines changed

scripts/wrappers/join.py

Lines changed: 7 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -312,38 +312,34 @@ def create_kubeconfig(token, ca, master_ip, api_port, filename, user):
312312
try_set_file_permissions(config)
313313

314314

315-
def update_kubeproxy(token, ca, master_ip, api_port, hostname_override):
315+
def update_kubeproxy(token, ca, master_ip, api_port):
316316
"""
317317
Configure the kube-proxy
318318
319319
:param token: the token to be in the kubeconfig
320320
:param ca: the ca
321321
:param master_ip: the master node IP
322322
:param api_port: the API server port
323-
:param hostname_override: the hostname override in case the hostname is not resolvable
324323
"""
325324
create_kubeconfig(token, ca, master_ip, api_port, "proxy.config", "kubeproxy")
326325
set_arg("--master", None, "kube-proxy")
327-
if hostname_override:
328-
set_arg("--hostname-override", hostname_override, "kube-proxy")
326+
set_arg("--hostname-override", None, "kube-proxy")
329327
service("restart", "proxy")
330328

331329

332-
def update_cert_auth_kubeproxy(token, master_ip, master_port, hostname_override):
330+
def update_cert_auth_kubeproxy(token, master_ip, master_port):
333331
"""
334332
Configure the kube-proxy
335333
336334
:param token: the token to be in the kubeconfig
337335
:param ca: the ca
338336
:param master_ip: the master node IP
339337
:param master_port: the master node port where the cluster agent listens
340-
:param hostname_override: the hostname override in case the hostname is not resolvable
341338
"""
342339
proxy_token = "{}-proxy".format(token)
343340
get_client_cert(master_ip, master_port, "proxy", proxy_token, "/CN=system:kube-proxy", False)
344341
set_arg("--master", None, "kube-proxy")
345-
if hostname_override:
346-
set_arg("--hostname-override", hostname_override, "kube-proxy")
342+
set_arg("--hostname-override", None, "kube-proxy")
347343

348344

349345
def update_kubeproxy_cidr(cidr):
@@ -769,7 +765,7 @@ def join_dqlite_worker_node(info, master_ip, master_port, token):
769765
store_base_kubelet_args(info["kubelet_args"])
770766
update_kubelet_node_ip(info["kubelet_args"], hostname_override)
771767
update_kubelet_hostname_override(info["kubelet_args"])
772-
update_cert_auth_kubeproxy(token, master_ip, master_port, hostname_override)
768+
update_cert_auth_kubeproxy(token, master_ip, master_port)
773769
update_cert_auth_kubelet(token, master_ip, master_port)
774770
subprocess.check_call(
775771
[f"{snap()}/actions/common/utils.sh", "create_worker_kubeconfigs"],
@@ -891,12 +887,10 @@ def join_etcd(connection_parts, verify=True):
891887
update_flannel(info["etcd"], master_ip, master_port, token)
892888

893889
if api_authn_mode == "Token":
894-
update_kubeproxy(
895-
info["kubeproxy"], info["ca"], master_ip, info["apiport"], hostname_override
896-
)
890+
update_kubeproxy(info["kubeproxy"], info["ca"], master_ip, info["apiport"])
897891
update_kubelet(info["kubelet"], info["ca"], master_ip, info["apiport"])
898892
elif api_authn_mode == "Cert":
899-
update_cert_auth_kubeproxy(info["kubeproxy"], master_ip, master_port, hostname_override)
893+
update_cert_auth_kubeproxy(info["kubeproxy"], master_ip, master_port)
900894
update_cert_auth_kubelet(info["kubelet"], master_ip, master_port)
901895
subprocess.check_call(
902896
[

0 commit comments

Comments
 (0)