NTP server choice removed?

elapsed · June 23, 2019, 5:05pm

I’m sure you used to be able to choose which NTP server you wanted to use by typing in the domains?
Was this functionality removed from LuCI?

anon50890781 · June 23, 2019, 5:31pm

LuCI development is managed/done by upstream OpenWRT and not the the TO team.

Perhaps the question is better placed with OpenWRT.

elapsed · June 23, 2019, 5:40pm

According to screenshots I’ve seen from searching, it still exists in OpenWRT. I have not personally verified the latest version however.

Foris also does not seem to allow editing the entries used, you can only view them.

anon50890781 · June 23, 2019, 5:50pm

That is correct but might be worked on

MOX A - ntp servers

time being only via ssh

settings stored in /etc/config/system

config timeserver 'ntp'
	list server '217.31.202.100'
	list server '195.113.144.201'
	list server '195.113.144.238'
	list server '2001:1488:ffff::100'
	list server 'ntp.nic.cz'
	list server '0.openwrt.pool.ntp.org'
	list server '1.openwrt.pool.ntp.org'
	list server '2.openwrt.pool.ntp.org'
	list server '3.openwrt.pool.ntp.org'
	option enabled '1'
	option enable_server '0'

They may not reflect the current LuCI state, which does not provide the option any more - I do remember it used to be possible as it was to install ntpd - which now is handled by systnpd (busybox) however.

There would be also luci-app-ntpc with ntpclient but that will create just an conflict with sysntpd

elapsed · June 23, 2019, 5:54pm

So just to be clear, this was actually removed from LuCI and (for now) the only way to edit them is via SSH?

anon50890781 · June 23, 2019, 5:54pm

that is the short of it indeed

ftmx · June 23, 2019, 6:34pm

You can still change the NTP servers in OpemWRT LuCI in 18.06 and master.

anon50890781 · June 23, 2019, 6:52pm

Oops, that it is interesting. It would imply that TO is patching LuCI and remove the option. Just what purpose that serves is a bit of mystery, preventing the access to the option via UI.

Could you share perhaps a screenshot?

Pepe · June 23, 2019, 7:00pm

Why we would remove the option? We are patching LuCI mostly for branding and no patch removes the option.
In OpenWrt 18.06, we have 5 patches for LuCI and no one does that.

backport: luci-app-transmission to allow user to define custom web ui
branding: remove uhttpd dependency
branding: add webapps
branding: use by default our server for LXC containers
wip: don’t use opkg --force-overwrite

Look here: https://gitlab.labs.nic.cz/turris/turris-build/tree/v4.0/patches/luci

In OpenWrt 19.06 and OpenWrt master, we have just two patches:

branding: remove uhttpd dependency
branding: add webapps
Look here: https://gitlab.labs.nic.cz/turris/turris-build/tree/master/patches/luci/

anon50890781 · June 23, 2019, 8:18pm

That is how it used to look in LuCI

And if am not completely mistaken for that part it requires ntpd being installed and not just busybox’s sysntpd implant.

Pepe · June 23, 2019, 8:29pm

I found the culprit.

https://github.com/openwrt/luci/blob/openwrt-19.07/modules/luci-mod-system/luasrc/model/cbi/admin_system/system.lua#L11

If there is no /usr/sbin/ntpd, then it isn’t shown in LuCI. Once I created symlink, it’s there.

symlink -s /sbin/ntpd /usr/sbin/ntpd

It seems that it should work. I checked to provide NTP server and I see that it should be running:

root@turris:~# netstat -l | grep ntp
udp        0      0 :::ntp                  :::*

This is an issue across all Turris OS versions.

anon50890781 · June 23, 2019, 8:40pm

It runs also without the link being set but at least now shows in LuCI.

Pepe · June 23, 2019, 10:01pm

In LuCI here is a hard coded path, but it’s the same as it should be in OpenWrt:

github.com

openwrt/openwrt/blob/master/package/utils/busybox/files/sysntpd#L7


#!/bin/sh /etc/rc.common
# Copyright (C) 2011 OpenWrt.org


START=98


USE_PROCD=1
PROG=/usr/sbin/ntpd
HOTPLUG_SCRIPT=/usr/sbin/ntpd-hotplug


get_dhcp_ntp_servers() {
	local interfaces="$1"
	local filter="*"
	local interface ntpservers ntpserver


	for interface in $interfaces; do
		[ "$filter" = "*" ] && filter="@.interface='$interface'" || filter="$filter,@.interface='$interface'"
	done

Unfortunately, this was caused by one of our patch, which moves busybox binaries to /{s,}bin from /usr/bin.

I prepared a pull request to fix this in Turris OS:

Pepe · June 24, 2019, 5:54am

Fixed in the Turris OS 3.11.6, upcoming next version of Turris OS 4.0 and as well with Turris OS 5.0 (branch HBD, based on the cooking OpenWrt 19.07) and I shouldn’t forget also Turris OS 6.0, which is daily build on branches master of OpenWrt.

Thank you for your reporting.

Pepe · June 24, 2019, 5:42pm

4 posts were split to a new topic: NTP: add a list to choose from in Foris/LuCI