From 23b2c0aa9e9d78966deab9f64cd46ca1f7f11c55 Mon Sep 17 00:00:00 2001 From: Clayton O'Neill Date: Thu, 12 Feb 2015 18:19:43 +0000 Subject: [PATCH] Ensure designate user/group exist Without the ensure line, Puppet will only try to manage the attributes on these resources. This issue only occurs if the system packages don't create the users. Change-Id: Id46f06a03c31c1dd29971fec21703715744ab8d5 --- manifests/init.pp | 2 ++ spec/classes/designate_init_spec.rb | 6 ++++-- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/manifests/init.pp b/manifests/init.pp index 40a3ea70..2ccc1ad6 100644 --- a/manifests/init.pp +++ b/manifests/init.pp @@ -60,6 +60,7 @@ class designate( } user { 'designate': + ensure => 'present', name => 'designate', gid => 'designate', groups => ['designate',$::dns::params::group], @@ -68,6 +69,7 @@ class designate( } group { 'designate': + ensure => 'present', name => 'designate', require => Package['designate-common'], } diff --git a/spec/classes/designate_init_spec.rb b/spec/classes/designate_init_spec.rb index e3b164f5..03909b16 100644 --- a/spec/classes/designate_init_spec.rb +++ b/spec/classes/designate_init_spec.rb @@ -39,16 +39,18 @@ describe 'designate' do it 'configures designate group' do should contain_group('designate').with( + :ensure => 'present', :name => 'designate', - :require => 'Package[designate-common]' + :require => 'Package[designate-common]', ) end it 'configures designate user' do should contain_user('designate').with( + :ensure => 'present', :name => 'designate', :gid => 'designate', - :system => true + :system => true, ) end