Treat an exit status of 3 as success for chef-client

When chef-client is terminated with SIGTERM it unhelpfully
returns an exit status of 3 which, by default, is treated as
an unsuccessful exit.

diff --git a/cookbooks/chef/recipes/default.rb b/cookbooks/chef/recipes/default.rb
index 9443d054848af99e118d17837d58a911cd1dda2e..2e2e345a11374886d93acdb60e6294c54d8de91e 100644 (file)
--- a/cookbooks/chef/recipes/default.rb
+++ b/cookbooks/chef/recipes/default.rb
@@ -112,6 +112,7 @@ if node[:lsb][:release].to_f >= 15.10
     description "Chef client"
     after "network.target"
     exec_start "/usr/bin/chef-client -i 1800 -s 20"
+    success_exit_status 3
     restart "on-failure"
   end
 else

diff --git a/cookbooks/systemd/resources/service.rb b/cookbooks/systemd/resources/service.rb
index 2a8f22b2b30547805e47869d2d558f418f58e1a3..664304da8d6e07c51fbac1ee1b01683f12936f94 100644 (file)
--- a/cookbooks/systemd/resources/service.rb
+++ b/cookbooks/systemd/resources/service.rb
@@ -45,6 +45,7 @@ property :standard_output, String,
          :is => %w(inherit null tty journal syslog kmsg journal+console syslog+console kmsg+console socket)
 property :standard_error, String,
          :is => %w(inherit null tty journal syslog kmsg journal+console syslog+console kmsg+console socket)
+property :success_exit_status, [Integer, String, Array]
 property :restart, String,
          :is => %w(on-success on-failure on-abnormal on-watchdog on-abort always)
 property :private_tmp, [TrueClass, FalseClass]

diff --git a/cookbooks/systemd/templates/default/service.erb b/cookbooks/systemd/templates/default/service.erb
index ffc0efb7203e5da5696596f6109abba04c355e86..f7f851ae54160b373cbc40f5d3506099c731bd7e 100644 (file)
--- a/cookbooks/systemd/templates/default/service.erb
+++ b/cookbooks/systemd/templates/default/service.erb
@@ -75,6 +75,9 @@ ProtectHome=<%= @protect_home %>
 <% if @no_new_privileges -%>
 NoNewPrivileges=<%= @no_new_privileges %>
 <% end -%>
+<% if @success_exit_status -%>
+SuccessExitStatus=<%= Array(@success_exit_status).join(" ") %>
+<% end -%>
 <% if @restart -%>
 Restart=<%= @restart %>
 <% end -%>