From 34264e2828eb14a036a646114587b3172f4d51e4 Mon Sep 17 00:00:00 2001 From: Charlie Sharpsteen Date: Thu, 3 Sep 2026 22:37:53 -0700 Subject: [PATCH] Drop ExecReload from SystemD services This commit removes `ExecReload=kill -HUP $MAINPID` from the SystemD service templates. Using `kill` to send `SIGHUP` is redundant when `Type=notify-reload` is set as SystemD will send the signal its self. SystemD also marks the service as "reloaded" if the `ExecReload` command exits with a success code. This can result in `systemctl reload` returning early, before the service is ready to accept traffic. Signed-off-by: Charlie Sharpsteen --- .../lein-ezbake/template/global/ext/debian/ezbake.service.erb | 1 - .../lein-ezbake/template/global/ext/redhat/ezbake.service.erb | 1 - 2 files changed, 2 deletions(-) diff --git a/resources/puppetlabs/lein-ezbake/template/global/ext/debian/ezbake.service.erb b/resources/puppetlabs/lein-ezbake/template/global/ext/debian/ezbake.service.erb index ef465127..22878e46 100644 --- a/resources/puppetlabs/lein-ezbake/template/global/ext/debian/ezbake.service.erb +++ b/resources/puppetlabs/lein-ezbake/template/global/ext/debian/ezbake.service.erb @@ -66,7 +66,6 @@ KillMode=process <% EZBake::Config[:debian][:post_start_action].each do |action| -%> ExecStartPost=-<%= action %> <% end -%> -ExecReload=kill -HUP $MAINPID SuccessExitStatus=143 [Install] diff --git a/resources/puppetlabs/lein-ezbake/template/global/ext/redhat/ezbake.service.erb b/resources/puppetlabs/lein-ezbake/template/global/ext/redhat/ezbake.service.erb index 8df9c6ac..5ab6a91f 100644 --- a/resources/puppetlabs/lein-ezbake/template/global/ext/redhat/ezbake.service.erb +++ b/resources/puppetlabs/lein-ezbake/template/global/ext/redhat/ezbake.service.erb @@ -66,7 +66,6 @@ KillMode=process <% EZBake::Config[:redhat][:post_start_action].each do |action| -%> ExecStartPost=-<%= action %> <% end -%> -ExecReload=kill -HUP $MAINPID SuccessExitStatus=143 [Install]