pendulum-project / ntpd-rs

A full-featured implementation of the Network Time Protocol, including NTS support.
https://trifectatech.org
Other
749 stars 42 forks source link

Add NTS-enabled indicator for ntp-ctl status #1647

Closed michielp1807 closed 3 weeks ago

michielp1807 commented 3 weeks ago

ntp-ctl status now shows for each source if that source is NTS-enabled, e.g.:

Sources:
time.tweede.golf:4460/34.147.28.4:123 [NTS] (1): -0.000101±0.000141(±0.008505)s
    poll interval: 128s, missing polls: 1
    root dispersion: 0.000031s, root delay:0.006897s
    NTS cookies: 7/8 available

if NTS is disabled, nothing changes:

Sources:
ntpd-rs.pool.ntp.org:123/185.51.192.62:123 (1): +0.047054±0.079489(±0.006319)s
    poll interval: 16s, missing polls: 0
    root dispersion: 0.000183s, root delay:0.002304s

The Prometheus output has not (yet) been updated to include a NTS flag

codecov[bot] commented 3 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 81.22%. Comparing base (a859104) to head (c1df5e0). Report is 5 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1647 +/- ## ========================================== + Coverage 81.08% 81.22% +0.14% ========================================== Files 64 64 Lines 19708 19664 -44 ========================================== - Hits 15981 15973 -8 + Misses 3727 3691 -36 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.