Merge pull request #20 from diranged/fix_bad_file_resource

Fix bad file resource
jdowning · Mar 14, 2015 · b70bf4c · b70bf4c
2 parents 249efc6 + 8ead840
commit b70bf4c
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 8 deletions.
diff --git a/manifests/config.pp b/manifests/config.pp
@@ -9,40 +9,42 @@
   file { '/etc/statsd':
     ensure => directory,
     mode   => '0755',
+    owner  => 'root',
+    group  => 'root',
   }->
   file { $configfile:
     content => template('statsd/localConfig.js.erb'),
     mode    => '0644',
+    owner  => 'root',
+    group  => 'root',
   }
 
   file { $statsd::init_location:
     source => $statsd::init_script,
     mode   => $statsd::init_mode,
+    owner  => 'root',
+    group  => 'root',
   }
 
   file {  '/etc/default/statsd':
     content => template('statsd/statsd-defaults.erb'),
+    owner  => 'root',
+    group  => 'root',
     mode    => '0755',
   }
 
   file { '/var/log/statsd':
     ensure => directory,
-    owner  => 'nobody',
     mode   => '0755',
+    owner  => 'nobody',
+    group  => 'root',
   }
 
   file { '/usr/local/sbin/statsd':
     source => 'puppet:///modules/statsd/statsd-wrapper',
     mode   => '0755',
-  }
-
-  File {
     owner  => 'root',
     group  => 'root',
   }
 
-  if $statsd::manage_service == true {
-    File <| |> ~> Service['statsd']
-  }
-
 }
diff --git a/manifests/init.pp b/manifests/init.pp
@@ -98,6 +98,7 @@
       enable    => $service_enable,
       hasstatus => true,
       provider  => $init_provider,
+      subscribe => Class['statsd::config'],
       require   => [ Package['statsd'], File['/var/log/statsd'] ],
     }
   }