From 44a7cb6f5af92c3329209f9d2c9fc21d7910718f Mon Sep 17 00:00:00 2001
From: Aaron Bieber <aaron@bolddaemon.com>
Date: Mon, 21 Oct 2019 21:07:00 -0600
Subject: [PATCH] Only restart services when the corresponding file has been
 modified recently

---
 README.md               | 10 ++++++++--
 boxctl.sh               | 23 ++++++++++++++++++-----
 examples/host1/services |  2 +-
 man/boxctl.8            | 10 ++++++++--
 4 files changed, 35 insertions(+), 10 deletions(-)

diff --git a/README.md b/README.md
index ae612ea..700fb6f 100644
--- a/README.md
+++ b/README.md
@@ -86,8 +86,14 @@ The options are as follows:
 
 > An optional file that contains a list of services to enable on the remote
 > host.
-> If a service is already enabled, it will be restarted each run of
-> **boxctl**.
+> Each line in the file should contain "name:path".
+> Where "name" is the name of a service (httpd for example) and "path" is the
+> full path to a file that when modified, should trigger a restart of the named
+> service.
+> For example, the entry "httpd:/etc/httpd.conf" would restart
+> httpd(8)
+> if the mtime on "/etc/httpd.conf" is less than 100 seconds from the current
+> time.
 
 *$CWD/groups*
 
diff --git a/boxctl.sh b/boxctl.sh
index bd7bdd5..9007181 100755
--- a/boxctl.sh
+++ b/boxctl.sh
@@ -69,10 +69,19 @@ EOF
 )
 
 SERVICE_START_RESTART=$(cat <<EOF
-/usr/sbin/rcctl enable %s; \
-/usr/sbin/rcctl check %s && \
-	/usr/sbin/rcctl restart %s || \
+/usr/sbin/rcctl enable %s;
+if /usr/sbin/rcctl check %s; then
+	AGE=\$(stat -s %s | awk -v now=\$(date +%%s) \
+		'{split(\$10,a,"="); print now - a[2]}')
+	if [ \$AGE -lt 100 ]; then
+		logger -t boxctl "restarting %s (\$AGE seconds old)"
+		/usr/sbin/rcctl restart %s
+	else
+		logger -t boxctl "not restarting %s (\$AGE seconds old)"
+	fi
+else
 	/usr/sbin/rcctl start %s
+fi
 EOF
 )
 
@@ -258,10 +267,14 @@ fi
 
 if [ -f ./services ]; then
 	msg 0 "enabling services $(wc -l services | awk '{print $1 " " $2}')"
+	local _svc _chfile
 	for service in $(cat services); do
-		msg 1 "\tenabling/restarting ${service}"
+		read _svc _chfile <<EOF
+			$(echo $service | sed 's/:/ /g')
+EOF
+		msg 1 "\tenabling/restarting ${_svc}"
 		cmd="$(printf "$SERVICE_START_RESTART" \
-			$service $service $service $service)"
+			$_svc $_svc $_chfile $_svc $_svc $_svc $_svc)"
 		_ssh ${RUN_USER}@${SERVER} "${cmd}"
 	done
 fi
diff --git a/examples/host1/services b/examples/host1/services
index 71e8b8e..8738ce0 100644
--- a/examples/host1/services
+++ b/examples/host1/services
@@ -1 +1 @@
-httpd
+httpd:/etc/httpd.conf
diff --git a/man/boxctl.8 b/man/boxctl.8
index 5c38de0..b7bc15d 100644
--- a/man/boxctl.8
+++ b/man/boxctl.8
@@ -83,8 +83,14 @@ If this file does not exist, no files are copied.
 .It Pa $CWD/services
 An optional file that contains a list of services to enable on the remote
 host.
-If a service is already enabled, it will be restarted each run of
-.Nm .
+Each line in the file should contain "name:path".
+Where "name" is the name of a service (httpd for example) and "path" is the
+full path to a file that when modified, should trigger a restart of the named
+service.
+For example, the entry "httpd:/etc/httpd.conf" would restart
+.Xr httpd 8
+if the mtime on "/etc/httpd.conf" is less than 100 seconds from the current
+time.
 .It Pa $CWD/groups
 A ":" delimited list of groups to be added to the remote host.
 Entries should follow a "group:gid" pattern.