Re: [chrony-dev] [PATCH] nm-dispatcher: handle NTP servers from DHCP

[ Thread Index | Date Index | More chrony.tuxfamily.org/chrony-dev Archives ]


On Thu, Jun 18, 2020 at 04:01:27AM -0400, Robert Fairley wrote:
> For now I've left https://src.fedoraproject.org/rpms/chrony/pull-request/3
> with a fragment for `sourcedirs`, as the PR is pending the `sourcedirs`
> being packaged. In the meantime I can take a look to see what changes
> are needed in the installer and others. Otherwise, for now having
> `sourcedirs` appended the main `/etc/chrony.conf` would be fine (with
> respect to allowing overriding the propagation from DHCP, we'd
> later on aim to make this its own fragment, if acceptable).

I think it could be a fragment now, and probably everything else
except the default servers. I'll see if I can add an example of
a fragmented configuration, which downstreams could use.

> So far I have tested these patches with the downstream macro
> substitutions and sysconfig patch applied, and using `sourcedirs`
> to specify `/var/run/chrony-dhcp`, in Fedora Rawhide. On modifying the
> `*.sources` files and `chronyc reload sources` (or
> deactivating/activating the NM interface to re-run the dispatcher,
> the sources from a DHCP server appeared in `chronyc sources`.

Great.

The patches look good, except the commit message still mentions the
helper. Can you please update that?

> +++ b/examples/chrony.nm-dispatcher.dhcp.in
> @@ -0,0 +1,44 @@
> +#!/bin/sh
> +# This is a NetworkManager dispatcher script for chronyd to update
> +# its NTP sources passed from DHCP options. Note that this script is
> +# specific to NetworkManager-dispatcher due to use of the
> +# DHCP4_NTP_SERVERS environment variable. For networkd-dispatcher,
> +# an alternative approach is external means such as a dhclient hook.

Maybe don't mention networkd-dispatcher here at all? If someone adds
support to this script, or adds a new script, it will be mentioned
there.

Thanks,

-- 
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.


Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/