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

Allow for a Fully-Qualified Hostname #931

Open
wants to merge 18 commits into
base: master
Choose a base branch
from

Conversation

tayterz2
Copy link
Contributor

Problem:
When using domain names in hostnames, inform RancherD and the Dashboard that this is the proper hostname.

Solution:
Add hostname to rancherd node adds, and perform "hostnamectl" instead of "hostname" to pull the FQDN from the hosts.

Related Issue:
harvester/harvester#7312

Test plan:
Install Harvester with an FQDN as hostname (taylor.rancher.io) and observe that same hostname in the "hosts" and "node" fields throughout.

Add check to make sure DHCP route error only shows for DHCP setups
@bk201 bk201 requested review from Yu-Jack and tserong February 4, 2025 01:06
@tayterz2 tayterz2 requested a review from Yu-Jack February 14, 2025 17:10
@Yu-Jack
Copy link
Contributor

Yu-Jack commented Feb 24, 2025

@tayterz2 BTW, please use rebase master, thank you!

@tayterz2
Copy link
Contributor Author

@Yu-Jack - I rebased 2 days ago, and currently github reports this branch as caught up. What am I missing?

Thanks!
Taylor

@Yu-Jack
Copy link
Contributor

Yu-Jack commented Feb 25, 2025

Hi @tayterz2 I think you can rebase again. Or just open another branch based on master, cherry-pick your commit and force push it to this branch again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants