Miroslav Lichvar
Miroslav Lichvar
Looks good. I'd just replace "NTP client" with ntpperf and move the 172.18.0.0/16 box inside to me it clear it's not a single client and that it is simulated (the...
Great. Please add it to the document and commit.
Why not add it as a new file to the repo and reference it in the document?
CFLAGS can be set on the make command line, e.g. `make CFLAGS="-O3 -g"`. For the misaligned access, I'd suggest to allocate a buffer on stack and memcpy the frame there...
As currently documented, this role is meant only for configuring NTP clients and/or PTP slaves. It might make sense to extend it into a NTP server, but we need to...
The role used to have some variables for adding settings directly to the generated config files, but IIRC they were removed in order to hide the implementation-specific details. Later was...
I'm not aware of any best practices for injecting configuration snippets like this. I think it would be good to warn the users that there are no guarantees a working...
`-u chrony` is the default on RHEL. It doesn't need to be set. Other distributions may have different names for the user.
If a new option needs to be added to the role for this, I'd prefer a more general approach specifying directly the additional chronyd options included in /etc/sysconfig/chronyd, e.g. `timesync_chronyd_custom_options`.
Some systems don't have sysconfig, e.g. on Debian the options are in `/etc/default/chrony`, so the I think the name should be general enough to work on all potentially supported distros.