templates should not notifiy statsd service to restart if service att…

recipes/configure.rb

@@ -19,6 +19,15 @@
 
 require 'chef/mixin/deep_merge'
 
+service_status = node['statsd']['service'].map do |a, s|
+  case a.to_s
+  when 'enable'
+    s == false ? :disable : :enable
+  when 'start'
+    s == false ? :stop : :start
+  end
+end
+
 backends = []
 
 if node['statsd']['graphite_enabled']
@@ -73,7 +82,7 @@
 
   Chef::Mixin::DeepMerge.deep_merge!(node['statsd']['extra_config'], config_hash)
   variables config_hash: config_hash
-  notifies :restart, 'service[statsd]', :delayed
+  notifies :restart, 'service[statsd]', :delayed unless service_status.any? { |x| [:disable, :stop].include?(x) }
 end
 
 file node['statsd']['log_file'] do

recipes/systemd_service.rb

@@ -17,9 +17,19 @@
 # limitations under the License.
 #
 
+service_status = node['statsd']['service'].map do |a, s|
+  case a.to_s
+  when 'enable'
+    s == false ? :disable : :enable
+  when 'start'
+    s == false ? :stop : :start
+  end
+end
+
 template '/etc/systemd/system/statsd.service' do
   source 'systemd.service.erb'
   action :create
+  notifies :restart, 'service[statsd]', :delayed unless service_status.any? { |x| [:disable, :stop].include?(x) }
 end
 
 service 'statsd' do

recipes/upstart_service.rb

@@ -17,11 +17,20 @@
 # limitations under the License.
 #
 
+service_status = node['statsd']['service'].map do |a, s|
+  case a.to_s
+  when 'enable'
+    s == false ? :disable : :enable
+  when 'start'
+    s == false ? :stop : :start
+  end
+end
+
 # Install our init script.
 template '/etc/init/statsd.conf' do
   source 'upstart.conf.erb'
   mode 0644
-  notifies :restart, 'service[statsd]', :delayed
+  notifies :restart, 'service[statsd]', :delayed unless service_status.any? { |x| [:disable, :stop].include?(x) }
 end
 
 # Set up our service.