Add ensure value for user and group resources

'ensure' property for user and group type resource has no default value;
if we are managing ceilometer user and group we should set ensure to
'present' so that the resources get created.

Also make user depend on group resource being created first.

Change-Id: Ie5b2704b3ea9a7727c6ff4f06efcbea201e6314a

diff --git a/manifests/init.pp b/manifests/init.pp
index c209f99fa76b5bfa913bbfb3da259b40086464ed..a73a6b015c50b11297b0424a7eec79b55f21b3e1 100644 (file)
--- a/manifests/init.pp
+++ b/manifests/init.pp
@@ -361,11 +361,13 @@ please use memcache_servers instead.")
   }
 
   group { 'ceilometer':
+    ensure  => present,
     name    => 'ceilometer',
     require => Anchor['ceilometer::install::end'],
   }
 
   user { 'ceilometer':
+    ensure  => present,
     name    => 'ceilometer',
     gid     => 'ceilometer',
     system  => true,

diff --git a/spec/classes/ceilometer_init_spec.rb b/spec/classes/ceilometer_init_spec.rb
index a7e9f651ffa8242dade0de5217fef92f5d1d3534..02e8650cc2fff1976439e55dc73451d4ba86e2f0 100644 (file)
--- a/spec/classes/ceilometer_init_spec.rb
+++ b/spec/classes/ceilometer_init_spec.rb
@@ -84,6 +84,7 @@ describe 'ceilometer' do
 
     it 'configures ceilometer group' do
       is_expected.to contain_group('ceilometer').with(
+        :ensure  => 'present',
         :name    => 'ceilometer',
         :require => 'Anchor[ceilometer::install::end]'
       )
@@ -91,6 +92,7 @@ describe 'ceilometer' do
 
     it 'configures ceilometer user' do
       is_expected.to contain_user('ceilometer').with(
+        :ensure  => 'present',
         :name    => 'ceilometer',
         :gid     => 'ceilometer',
         :system  => true,