Repository navigation
Conversation
| command=/usr/bin/ntp-daemon | ||
| command_args="-c ${CONFIG} -l ${LOG_LEVEL}" | ||
| # log the stderror and stdout to the logger program | ||
| output_logger="logger -t $RC_SVCNAME -p daemon.info" |
There was a problem hiding this comment.
should we use daemon.notice instead? is ntpd-rs noisy on stdout or does it actually only give out important messages in which case I think we should use notice.
There was a problem hiding this comment.
We are currently actively working to make ntpd-rs less noisy, so I think we should aspire to daemon.notice I think.
There was a problem hiding this comment.
It would be more appropriate to log to a seperate log file then
5d28ac2 to
6167525
Compare
|
Converting to draft, because of some bugs either in the script, OpenRC or ntpd-rs:
|
92b7255 to
8062aba
Compare
|
Thank you for providing this. To answer some of your questions: |
|
On Thu, 2026-10-08 at 23:59 -0700, David Venhoek wrote:
davidv1992 left a comment (pendulum-project/ntpd-rs#2526)
Thank you for providing this.
To answer some of your questions:
For the observation socket, we are currently in a fairly major migration, and have accepted that currently the main branch is not entirely functional. In particular, observability is in that spot right now where it isn't fully functional. For testing, it may be easier to use the 1.9 version for now.
This *is* with v1.9
As to the /var/lib/ntpd-rs folder, we don't currently use that.
Thanks so should I remove the checkpath logic for it? Also the systemd service has https://github.com/pendulum-project/ntpd-rs/blob/d89fbb3916d1dc3a063c9d8813be026728951fce/docs/examples/conf/ntpd-rs.strict.service#L35 which is why I put the checkpath logic in the first place
|
|
Apologies, forgot we use it to store the cookie keys when running as an NTS server. That is configuration dependent though and will only happen when configured as described in the NTS guide. We include it in the example systemd file to make sure these configurations work out of the box. As for the observability, which configuration are you testing with? It needs to configure an observability path for the socket to get created. |
|
On Fri, 2026-10-09 at 00:58 -0700, David Venhoek wrote:
davidv1992 left a comment (pendulum-project/ntpd-rs#2526)
Apologies, forgot we use it to store the cookie keys when running as an NTS server. That is configuration dependent though and will only happen when configured as described in the NTS guide. We include it in the example systemd file to make sure these configurations work out of the box.
As for the observability, which configuration are you testing with? It needs to configure an observability path for the socket to get created.
Thanks I didn't set observability-path, the toml file I used was
https://codeberg.org/gentoo/gentoo/pulls/497/files#diff-de2e6f052625ce0f5a3b530f90ea655f7f63c522
( I really should have looked more closely and that PR should just use
the default ntp.toml file).
I'll mark this PR as ready once I test everything is working correctly.
|
Add OpenRC service script and a config file to go along with it, these are supposed to be installed to the init.d directory usually in /etc at /etc/init.d/ntpd-rs and /etc/conf.d/ntpd-rs respectively.
The user can customize ntp-daemon's -c and -l commands by editing the /etc/conf.d/ntpd-rs file
With this configuration OpenRC:
RUST_LOG=info /usr/bin/ntp-daemon -c /etc/ntpd-rs/ntp.toml -l infontpd-rs:ntpd-rscap_net_bind_serviceandcap_sys_timeNO_NEW_PRIVSenabledBut before the service actually starts OpenRC:
ntp-ctl validateor fail to start/var/lib/ntpd-rs,/run/ntpd-rsand/var/log/ntpd-rsas ntpd-rs:ntpd-rs if they don't existAnd if the service if restarted OpenRC:
ntp-ctl validateor else fail to restart with a broken configAnd if the service is reloaded via OpenRC with
rc-service ntpd-rs reloadntp-ctl validateor fail to startThats all.