From 3f1c4313c314d5327b03a8c5d5b81f2d68af2f34 Mon Sep 17 00:00:00 2001
From: karencfv <karencfv@users.noreply.github.com>
Date: Thu, 2 Jul 2026 15:53:16 +1200
Subject: [PATCH] [ntp-admin] Service should explicitly depend on chrony setup

---
 smf/chrony-setup/manifest.xml | 2 +-
 smf/ntp-admin/manifest.xml    | 5 +++++
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/smf/chrony-setup/manifest.xml b/smf/chrony-setup/manifest.xml
index 8127ca80dfc..915d3f55822 100644
--- a/smf/chrony-setup/manifest.xml
+++ b/smf/chrony-setup/manifest.xml
@@ -33,7 +33,7 @@
     -->
     <propval name="boundary_pool" type="astring" value="" />
     <!--
-        Upstream NTP server. May be specifid more than once. (At least one is
+        Upstream NTP server. May be specified more than once. (At least one is
         required for boundary NTP zones; internal NTP zones instead use
         `boundary_pool` above to find the boundary NTP servers.)
     -->
diff --git a/smf/ntp-admin/manifest.xml b/smf/ntp-admin/manifest.xml
index fdb97923e10..d12cbffc3cf 100644
--- a/smf/ntp-admin/manifest.xml
+++ b/smf/ntp-admin/manifest.xml
@@ -21,6 +21,11 @@
   <service_fmri value='svc:/oxide/ntp:default' />
   </dependency>
 
+  <dependency name='chrony-setup' grouping='require_all' restart_on='refresh'
+    type='service'>
+  <service_fmri value='svc:/oxide/chrony-setup:default' />
+  </dependency>
+
   <exec_method type='method' name='start'
     exec='/opt/oxide/lib/svc/manifest/ntp-admin.sh'
     timeout_seconds='0' />