Skip to content

Conversation

@liulinC
Copy link
Collaborator

@liulinC liulinC commented May 30, 2025

commit 6ac37d7 add localhost name to /etc/hosts
There problem is, instead of appending FQDN to the end of the line, the entry should in following format:
IP FQDN [alias1] [alias2]

Besides, another non-regression is found.
The host need to change its static IP and DNS to join AD, However, then the Host record is not send to DNS, cause the pool member can not resovle each other.

To resovle the issue, a task is scheduled to sync the hostname to DNS.

@liulinC liulinC force-pushed the private/linl/dev branch 2 times, most recently from 3a8b018 to 33be96b Compare May 30, 2025 11:42

let winbind_machine_pwd_timeout = ref (2. *. 7. *. 24. *. 3600.)

let winbind_dns_sync_interval = ref 3600.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This reminds me I really need to test my changes to replace the scheduler to actually use Mtime.Span instead of floats, and open e PR

commit 6ac37d7 add localhost name to /etc/hosts
There problem is, instead of appending FQDN to the end of the line,
the entry should in following format:
IP FQDN [alias1] [alias2]

Besides, another non-regression is found.
The host need to change its static IP and DNS to join AD,
However, then the Host record is not send to DNS, cause the pool
member can not resovle each other.

To resovle the issue, a task is scheduled to sync the hostname
to DNS.

Signed-off-by: Lin Liu <[email protected]>
@liulinC liulinC force-pushed the private/linl/dev branch from 33be96b to cba2f1d Compare June 4, 2025 09:44
@liulinC liulinC enabled auto-merge June 4, 2025 09:58
@liulinC liulinC added this pull request to the merge queue Jun 4, 2025
Merged via the queue into xapi-project:master with commit 35c353c Jun 4, 2025
16 checks passed
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.

3 participants