Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH 1/1] package/chrony: add 50-chrony.list for systemd-timedated
@ 2026-09-04 13:41 TIAN Yuanhao
  2026-09-06 11:30 ` TIAN Yuanhao
                   ` (2 more replies)
  0 siblings, 3 replies; 5+ messages in thread
From: TIAN Yuanhao @ 2026-09-04 13:41 UTC (permalink / raw)
  To: buildroot; +Cc: TIAN Yuanhao, Fabrice Fontaine, Scott Fan

systemd-timedated discovers NTP services from ntp-units.d/*.list.
Buildroot's chrony package never installs one: on a systemd image
without timesyncd, 'timedatectl status' reports

    NTP service: n/a

and 'timedatectl set-ntp on' fails with

    Failed to set ntp: NTP not supported

even though chronyd is running.

Install 50-chrony.list, following the convention set by Debian,
Fedora and Arch, so timedatectl can manage chrony.

Cc: Fabrice Fontaine <fontaine.fabrice@gmail.com>
Cc: Peter Korsgaard <peter@korsgaard.com>
Cc: Scott Fan <fancp2007@gmail.com>
Signed-off-by: TIAN Yuanhao <tianyuanhao3@163.com>
---
 package/chrony/50-chrony.list | 1 +
 package/chrony/chrony.mk      | 3 +++
 2 files changed, 4 insertions(+)
 create mode 100644 package/chrony/50-chrony.list

diff --git a/package/chrony/50-chrony.list b/package/chrony/50-chrony.list
new file mode 100644
index 0000000000..6b9cca00bb
--- /dev/null
+++ b/package/chrony/50-chrony.list
@@ -0,0 +1 @@
+chrony.service
diff --git a/package/chrony/chrony.mk b/package/chrony/chrony.mk
index d1745419a2..b2311df667 100644
--- a/package/chrony/chrony.mk
+++ b/package/chrony/chrony.mk
@@ -85,6 +85,9 @@ endef
 define CHRONY_INSTALL_INIT_SYSTEMD
 	$(INSTALL) -D -m 644 package/chrony/chrony.service \
 		$(TARGET_DIR)/usr/lib/systemd/system/chrony.service
+	# advertise chrony.service to systemd-timedated
+	$(INSTALL) -D -m 644 $(CHRONY_PKGDIR)/50-chrony.list \
+		$(TARGET_DIR)/usr/lib/systemd/ntp-units.d/50-chrony.list
 endef
 
 $(eval $(generic-package))
-- 
2.54.0

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* [Buildroot] [PATCH 1/1] package/chrony: add 50-chrony.list for systemd-timedated
  2026-09-04 13:41 [Buildroot] [PATCH 1/1] package/chrony: add 50-chrony.list for systemd-timedated TIAN Yuanhao
@ 2026-09-06 11:30 ` TIAN Yuanhao
  2026-09-06 12:52   ` TIAN Yuanhao
  2026-10-03 16:35 ` Thomas Petazzoni via buildroot
  2026-10-10  9:23 ` Thomas Perale via buildroot
  2 siblings, 1 reply; 5+ messages in thread
From: TIAN Yuanhao @ 2026-09-06 11:30 UTC (permalink / raw)
  To: buildroot; +Cc: TIAN Yuanhao, Fabrice Fontaine, Scott Fan

systemd-timedated discovers NTP services from ntp-units.d/*.list.
Buildroot's chrony package never installs one: on a systemd image
without timesyncd, 'timedatectl status' reports

    NTP service: n/a

and 'timedatectl set-ntp on' fails with

    Failed to set ntp: NTP not supported

even though chronyd is running.

Install 50-chrony.list, following the convention set by Debian,
Fedora and Arch, so timedatectl can manage chrony.

Cc: Fabrice Fontaine <fontaine.fabrice@gmail.com>
Cc: Peter Korsgaard <peter@korsgaard.com>
Cc: Scott Fan <fancp2007@gmail.com>
Signed-off-by: TIAN Yuanhao <tianyuanhao3@163.com>
---
 package/chrony/50-chrony.list | 1 +
 package/chrony/chrony.mk      | 3 +++
 2 files changed, 4 insertions(+)
 create mode 100644 package/chrony/50-chrony.list

diff --git a/package/chrony/50-chrony.list b/package/chrony/50-chrony.list
new file mode 100644
index 0000000000..6b9cca00bb
--- /dev/null
+++ b/package/chrony/50-chrony.list
@@ -0,0 +1 @@
+chrony.service
diff --git a/package/chrony/chrony.mk b/package/chrony/chrony.mk
index d1745419a2..b2311df667 100644
--- a/package/chrony/chrony.mk
+++ b/package/chrony/chrony.mk
@@ -85,6 +85,9 @@ endef
 define CHRONY_INSTALL_INIT_SYSTEMD
 	$(INSTALL) -D -m 644 package/chrony/chrony.service \
 		$(TARGET_DIR)/usr/lib/systemd/system/chrony.service
+	# advertise chrony.service to systemd-timedated
+	$(INSTALL) -D -m 644 $(CHRONY_PKGDIR)/50-chrony.list \
+		$(TARGET_DIR)/usr/lib/systemd/ntp-units.d/50-chrony.list
 endef
 
 $(eval $(generic-package))
-- 
2.54.0

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [Buildroot] [PATCH 1/1] package/chrony: add 50-chrony.list for systemd-timedated
  2026-09-06 11:30 ` TIAN Yuanhao
@ 2026-09-06 12:52   ` TIAN Yuanhao
  0 siblings, 0 replies; 5+ messages in thread
From: TIAN Yuanhao @ 2026-09-06 12:52 UTC (permalink / raw)
  To: buildroot

It's really weird that I can't find this patch on patchwork.buildroot.org.















At 2026-09-06 19:30:37, "TIAN Yuanhao" <tianyuanhao3@163.com> wrote:
>systemd-timedated discovers NTP services from ntp-units.d/*.list.
>Buildroot's chrony package never installs one: on a systemd image
>without timesyncd, 'timedatectl status' reports
>
>    NTP service: n/a
>
>and 'timedatectl set-ntp on' fails with
>
>    Failed to set ntp: NTP not supported
>
>even though chronyd is running.
>
>Install 50-chrony.list, following the convention set by Debian,
>Fedora and Arch, so timedatectl can manage chrony.
>
>Cc: Fabrice Fontaine <fontaine.fabrice@gmail.com>
>Cc: Peter Korsgaard <peter@korsgaard.com>
>Cc: Scott Fan <fancp2007@gmail.com>
>Signed-off-by: TIAN Yuanhao <tianyuanhao3@163.com>
>---
> package/chrony/50-chrony.list | 1 +
> package/chrony/chrony.mk      | 3 +++
> 2 files changed, 4 insertions(+)
> create mode 100644 package/chrony/50-chrony.list
>
>diff --git a/package/chrony/50-chrony.list b/package/chrony/50-chrony.list
>new file mode 100644
>index 0000000000..6b9cca00bb
>--- /dev/null
>+++ b/package/chrony/50-chrony.list
>@@ -0,0 +1 @@
>+chrony.service
>diff --git a/package/chrony/chrony.mk b/package/chrony/chrony.mk
>index d1745419a2..b2311df667 100644
>--- a/package/chrony/chrony.mk
>+++ b/package/chrony/chrony.mk
>@@ -85,6 +85,9 @@ endef
> define CHRONY_INSTALL_INIT_SYSTEMD
> 	$(INSTALL) -D -m 644 package/chrony/chrony.service \
> 		$(TARGET_DIR)/usr/lib/systemd/system/chrony.service
>+	# advertise chrony.service to systemd-timedated
>+	$(INSTALL) -D -m 644 $(CHRONY_PKGDIR)/50-chrony.list \
>+		$(TARGET_DIR)/usr/lib/systemd/ntp-units.d/50-chrony.list
> endef
> 
> $(eval $(generic-package))
>-- 
>2.54.0
>
>_______________________________________________
>buildroot mailing list
>buildroot@buildroot.org
>https://lists.buildroot.org/mailman/listinfo/buildroot
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Buildroot] [PATCH 1/1] package/chrony: add 50-chrony.list for systemd-timedated
  2026-09-04 13:41 [Buildroot] [PATCH 1/1] package/chrony: add 50-chrony.list for systemd-timedated TIAN Yuanhao
  2026-09-06 11:30 ` TIAN Yuanhao
@ 2026-10-03 16:35 ` Thomas Petazzoni via buildroot
  2026-10-10  9:23 ` Thomas Perale via buildroot
  2 siblings, 0 replies; 5+ messages in thread
From: Thomas Petazzoni via buildroot @ 2026-10-03 16:35 UTC (permalink / raw)
  To: TIAN Yuanhao; +Cc: buildroot, Fabrice Fontaine, Scott Fan

On Fri, Sep 04, 2026 at 06:41:28AM -0700, TIAN Yuanhao wrote:
> systemd-timedated discovers NTP services from ntp-units.d/*.list.
> Buildroot's chrony package never installs one: on a systemd image
> without timesyncd, 'timedatectl status' reports
> 
>     NTP service: n/a
> 
> and 'timedatectl set-ntp on' fails with
> 
>     Failed to set ntp: NTP not supported
> 
> even though chronyd is running.
> 
> Install 50-chrony.list, following the convention set by Debian,
> Fedora and Arch, so timedatectl can manage chrony.
> 
> Cc: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> Cc: Peter Korsgaard <peter@korsgaard.com>
> Cc: Scott Fan <fancp2007@gmail.com>
> Signed-off-by: TIAN Yuanhao <tianyuanhao3@163.com>

Applied to master, thanks!

Best regards,

Thomas Petazzoni
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [Buildroot] [PATCH 1/1] package/chrony: add 50-chrony.list for systemd-timedated
  2026-09-04 13:41 [Buildroot] [PATCH 1/1] package/chrony: add 50-chrony.list for systemd-timedated TIAN Yuanhao
  2026-09-06 11:30 ` TIAN Yuanhao
  2026-10-03 16:35 ` Thomas Petazzoni via buildroot
@ 2026-10-10  9:23 ` Thomas Perale via buildroot
  2 siblings, 0 replies; 5+ messages in thread
From: Thomas Perale via buildroot @ 2026-10-10  9:23 UTC (permalink / raw)
  To: TIAN Yuanhao; +Cc: Thomas Perale, buildroot

In reply of:
> systemd-timedated discovers NTP services from ntp-units.d/*.list.
> Buildroot's chrony package never installs one: on a systemd image
> without timesyncd, 'timedatectl status' reports
> 
>     NTP service: n/a
> 
> and 'timedatectl set-ntp on' fails with
> 
>     Failed to set ntp: NTP not supported
> 
> even though chronyd is running.
> 
> Install 50-chrony.list, following the convention set by Debian,
> Fedora and Arch, so timedatectl can manage chrony.
> 
> Cc: Fabrice Fontaine <fontaine.fabrice@gmail.com>
> Cc: Peter Korsgaard <peter@korsgaard.com>
> Cc: Scott Fan <fancp2007@gmail.com>
> Signed-off-by: TIAN Yuanhao <tianyuanhao3@163.com>

Applied to 2025.02.x & 2026.08.x. Thanks

> ---
>  package/chrony/50-chrony.list | 1 +
>  package/chrony/chrony.mk      | 3 +++
>  2 files changed, 4 insertions(+)
>  create mode 100644 package/chrony/50-chrony.list
> 
> diff --git a/package/chrony/50-chrony.list b/package/chrony/50-chrony.list
> new file mode 100644
> index 0000000000..6b9cca00bb
> --- /dev/null
> +++ b/package/chrony/50-chrony.list
> @@ -0,0 +1 @@
> +chrony.service
> diff --git a/package/chrony/chrony.mk b/package/chrony/chrony.mk
> index d1745419a2..b2311df667 100644
> --- a/package/chrony/chrony.mk
> +++ b/package/chrony/chrony.mk
> @@ -85,6 +85,9 @@ endef
>  define CHRONY_INSTALL_INIT_SYSTEMD
>  	$(INSTALL) -D -m 644 package/chrony/chrony.service \
>  		$(TARGET_DIR)/usr/lib/systemd/system/chrony.service
> +	# advertise chrony.service to systemd-timedated
> +	$(INSTALL) -D -m 644 $(CHRONY_PKGDIR)/50-chrony.list \
> +		$(TARGET_DIR)/usr/lib/systemd/ntp-units.d/50-chrony.list
>  endef
>  
>  $(eval $(generic-package))
> -- 
> 2.54.0
> 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2026-10-10  9:23 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-09-04 13:41 [Buildroot] [PATCH 1/1] package/chrony: add 50-chrony.list for systemd-timedated TIAN Yuanhao
2026-09-06 11:30 ` TIAN Yuanhao
2026-09-06 12:52   ` TIAN Yuanhao
2026-10-03 16:35 ` Thomas Petazzoni via buildroot
2026-10-10  9:23 ` Thomas Perale via buildroot

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox