| Re: [chrony-dev] [PATCH] nm-dispatcher: handle NTP servers from DHCP |
[ Thread Index |
Date Index
| More chrony.tuxfamily.org/chrony-dev Archives
]
- To: chrony-dev@xxxxxxxxxxxxxxxxxxxx
- Subject: Re: [chrony-dev] [PATCH] nm-dispatcher: handle NTP servers from DHCP
- From: Miroslav Lichvar <mlichvar@xxxxxxxxxx>
- Date: Wed, 10 Jun 2020 08:59:27 +0200
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1591772373; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=meYHIUpWonDUtjCl5qzOKGAFpHwTbiM5P3plmT41BdU=; b=Ee3i6RZacy/y/kA7L5Jh3MLvL6GkxvJ95LKuAaEX9Bx1r6mlK0oGSrx/jFpnJJ7Zknsl+l ZzAkkl4ggK3bKhvbnfx5nr2AJRz9Wtbhk36bKRKNmadxYzznREGdFpQ9WwIxxpXS8yyVfa +F5ZvsWDUNIiTXujWU+l1+ROCQ8KIbk=
On Tue, Jun 09, 2020 at 02:09:54PM -0400, Robert Fairley wrote:
> That sounds a good idea! We only need to reload the NTP sources -
> having a directive to specify where the NTP sources are managed makes
> sense. Would the directive point to a file/directory/glob to load
> files containing the NTP server config from?
I was thinking about reusing the confdirs code with its syntax, e.g.
sourcedirs /etc/chrony.d /var/run/chrony-dhcp
I'm not sure how much sense it will make to override the source files
(maybe disable servers from a specific interface?), but it's free and
it's easier to document.
As for your patchset, it would be good to rebase it to not include the
helper. It won't be very useful and we can probably drop it downstream
too. AFAIK nobody is really using the DNS SRV support. It was meant to
be used by the ipa installer, but it doesn't look like they ever used
it. For the static sources we might want to discourage editing the
main config.
There are other tools that rely on NTP sources being specified in the
main config (e.g. the installer). They will need to be updated before
we can move the default servers to a fragment.
--
Miroslav Lichvar
--
To unsubscribe email chrony-dev-request@xxxxxxxxxxxxxxxxxxxx with "unsubscribe" in the subject.
For help email chrony-dev-request@xxxxxxxxxxxxxxxxxxxx with "help" in the subject.
Trouble? Email listmaster@xxxxxxxxxxxxxxxxxxxx.