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

Use kubelet extra args --hostname-override value as etcd hostname? #3985

Closed
kke opened this issue Feb 1, 2024 · 4 comments
Closed

Use kubelet extra args --hostname-override value as etcd hostname? #3985

kke opened this issue Feb 1, 2024 · 4 comments
Labels
bug Something isn't working Stale

Comments

@kke
Copy link
Contributor

kke commented Feb 1, 2024

What happened?

As reported in a k0sctl issue, k0s always uses os.Hostname() as the etcd --name value. Maybe the value should be replicated from --kubelet-extra-args="--hostname-override=..."?

@kke kke added the bug Something isn't working label Feb 1, 2024
@twz123
Copy link
Member

twz123 commented Feb 1, 2024

The kubelet runs on workers, not on controllers. So this could only meaningfully apply to controller+worker nodes. Moreover, the os.Hostname() is also used in some other places, i.e. for events emitted by k0s. I agree, the node name handling in k0s is a bit messy and scattered around in many places, including autopilot. Fixing this would require some thought, though.

Is this a cosmetic thing, or would this fix a real problem?

@kke
Copy link
Contributor Author

kke commented Feb 1, 2024

k0sctl has spec: host: hostname: for overriding the hostname. This only sets the --hostname-override but it doesn't help with etcd.

@twz123
Copy link
Member

twz123 commented Feb 1, 2024

You could also pass --name to the etcd extra args, then.

Copy link
Contributor

github-actions bot commented Mar 2, 2024

The issue is marked as stale since no activity has been recorded in 30 days

@github-actions github-actions bot added the Stale label Mar 2, 2024
@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Mar 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Stale
Projects
None yet
Development

No branches or pull requests

2 participants