Use implicit dependencies

Puppet has implicit dependencies:
- Users and Groups have an implicit dependency
- Implicitly orders users and groups
- Implicitly recognizes file hierarchy
- Implicitly creates relationship between file ownership and user resources

diff --git a/manifests/init.pp b/manifests/init.pp
index 4e19fb4fc5d2dc5cbc55d822334977441a900683..df8591931397caeb3fdba2e62fe2ca59a3d0bb35 100644 (file)
--- a/manifests/init.pp
+++ b/manifests/init.pp
@@ -23,10 +23,7 @@ class ceilometer (
     gid     => $::ceilometer::params::groupname,
     groups  => ['nova'],
     system  => true,
-    require => [
-      Group['ceilometer'],
-      Package['ceilometer-common']
-    ],
+    require => Package['ceilometer-common'],
   }
 
   file { '/etc/ceilometer/':
@@ -34,7 +31,7 @@ class ceilometer (
     owner   => 'ceilometer',
     group   => 'ceilometer',
     mode    => '0750',
-    require => [Package['ceilometer-common'], User['ceilometer']],
+    require => Package['ceilometer-common'],
   }
 
   file { '/etc/ceilometer/ceilometer.conf':
@@ -42,7 +39,6 @@ class ceilometer (
     owner   => 'ceilometer',
     group   => 'ceilometer',
     mode    => '0640',
-    require => [File['/etc/ceilometer'], User['ceilometer']],
   }
 
   package { 'ceilometer-common':

diff --git a/spec/classes/ceilometer_init_spec.rb b/spec/classes/ceilometer_init_spec.rb
index eb63c69a18bce271b5284c35199894862e83b50a..4968266860e031bfafcc8ed458830ac8f9b13655 100644 (file)
--- a/spec/classes/ceilometer_init_spec.rb
+++ b/spec/classes/ceilometer_init_spec.rb
@@ -32,7 +32,7 @@ describe 'ceilometer' do
         :gid     => 'ceilometer',
         :groups  => ['nova'],
         :system  => true,
-        :require => ['Group[ceilometer]', 'Package[ceilometer-common]']
+        :require => 'Package[ceilometer-common]'
       )
     end
 
@@ -42,7 +42,7 @@ describe 'ceilometer' do
         :owner   => 'ceilometer',
         :group   => 'ceilometer',
         :mode    => '0750',
-        :require => ['Package[ceilometer-common]','User[ceilometer]']
+        :require => 'Package[ceilometer-common]'
       )
     end
 
@@ -51,8 +51,7 @@ describe 'ceilometer' do
         :ensure  => 'file',
         :owner   => 'ceilometer',
         :group   => 'ceilometer',
-        :mode    => '0640',
-        :require => ['File[/etc/ceilometer]', 'User[ceilometer]']
+        :mode    => '0640'
       )
     end