[openwrt/openwrt] busybox: sysntpd: add trigger to reload server

LEDE Commits lede-commits at lists.infradead.org
Wed Jun 30 00:25:03 PDT 2021


zorun pushed a commit to openwrt/openwrt.git, branch openwrt-21.02:
https://git.openwrt.org/9fa925362f285614e90c2d7a286efd9d3d41db45

commit 9fa925362f285614e90c2d7a286efd9d3d41db45
Author: Alexey Dobrovolsky <dobrovolskiy.alexey at gmail.com>
AuthorDate: Sun Jun 6 02:25:02 2021 +0300

    busybox: sysntpd: add trigger to reload server
    
    sysntpd server becomes unavailable if the index of the bound
    interface changes. So let's add an interface trigger to reload sysntpd.
    
    This patch also adds the ability for the sysntpd script to handle
    uci interface name from configuration.
    
    Fixes: 4da60500ebd2 ("busybox: sysntpd: option to bind server to iface")
    Signed-off-by: Alexey Dobrovolsky <dobrovolskiy.alexey at gmail.com>
    Reviewed-by: Philip Prindeville <philipp at redfish-solutions.com>
    (cherry picked from commit 88114f617ae7bffe13d19d7b9575659a3d3cd9b6)
---
 package/utils/busybox/files/sysntpd | 24 ++++++++++++++++++++++--
 1 file changed, 22 insertions(+), 2 deletions(-)

diff --git a/package/utils/busybox/files/sysntpd b/package/utils/busybox/files/sysntpd
index c4c311c242..074f14b8f8 100755
--- a/package/utils/busybox/files/sysntpd
+++ b/package/utils/busybox/files/sysntpd
@@ -56,7 +56,14 @@ start_ntpd_instance() {
 	procd_set_param command "$PROG" -n -N
 	if [ "$enable_server" = "1" ]; then
 		procd_append_param command -l
-		[ -n "$interface" ] && procd_append_param command -I $interface
+		[ -n "$interface" ] && {
+			local ifname
+
+			network_get_device ifname "$interface" || \
+				ifname="$interface"
+			procd_append_param command -I "$ifname"
+			procd_append_param netdev "$ifname"
+		}
 	fi
 	[ -x "$HOTPLUG_SCRIPT" ] && procd_append_param command -S "$HOTPLUG_SCRIPT"
 	for peer in $server; do
@@ -79,11 +86,12 @@ start_ntpd_instance() {
 }
 
 start_service() {
+	. /lib/functions/network.sh
 	validate_ntp_section ntp start_ntpd_instance
 }
 
 service_triggers() {
-	local script name use_dhcp
+	local script name use_dhcp enable_server interface
 
 	script=$(readlink -f "$initscript")
 	name=$(basename ${script:-$initscript})
@@ -106,5 +114,17 @@ service_triggers() {
 		fi
 	}
 
+	config_get_bool enable_server ntp enable_server 0
+	config_get interface ntp interface
+
+	[ $enable_server -eq 1 ] && [ -n "$interface" ] && {
+		local ifname
+
+		network_get_device ifname "$interface" || \
+			ifname="$interface"
+		procd_add_interface_trigger "interface.*" "$ifname" \
+			/etc/init.d/"$name" reload
+	}
+
 	procd_add_validation validate_ntp_section
 }



More information about the lede-commits mailing list