Page MenuHomePhabricator

Rewrite NTP subsystem in new XML style interface definition
Closed, ResolvedPublicFEATURE REQUEST


Rewrite the current interface files as XML interface and Python script to move towards new VyConfig


Difficulty level
Normal (likely a few hours)
Why the issue appeared?
Will be filled on close

Event Timeline

c-po created this task.May 6 2018, 6:57 PM
c-po claimed this task.
c-po triaged this task as Normal priority.
c-po updated the task description. (Show Details)
c-po changed Difficulty level from Unknown (require assessment) to Normal (likely a few hours).
c-po moved this task from Need Triage to In Progress on the VyOS 1.2 Crux board.

@dmbaturin @UnicronNL @syncer Is there any reason why NTP should be restarted when DNS changes?

The generated /etc/ntp.conf file only holds IP addresses.

syncer added a comment.May 6 2018, 7:15 PM

in theory we need to use hostnames there along with ips

c-po added a comment.EditedMay 6 2018, 9:05 PM

Feedback welcome:


look slike we can adjust this by using the hostname. If I remember correctly, most of the distros are using NTP with the pool and the corresponding hostnames.

I'm now running this config ...

### Autogenerated by on Sun, 06 May 2018 23:14:12 ###
driftfile /var/lib/ntp/ntp.drift
# By default, only allow ntpd to query time sources, ignore any
# incoming requests.
restrict default ignore

# Local users have unrestricted access, allowing reconfiguration
# via ntpdc
restrict -6 ::1

# Server configuration
server -4 iburst
restrict -4 nomodify notrap nopeer noquery

# Client configuration:
restrict mask nomodify notrap nopeer

# Client configuration: 2001:db9::/64
restrict 2001:db9:: mask ffff:ffff:ffff:ffff:: nomodify notrap nopeer

lets give it a try over some hours/days.

c-po added a comment.May 7 2018, 8:43 PM

Looks very good!

c-po moved this task from In Progress to Finished on the VyOS 1.2 Crux board.May 7 2018, 8:44 PM
dmbaturin closed this task as Resolved.Sep 25 2018, 6:20 PM