openstack/puppet-cinder
Code Issues Proposed changes

policy.pp : Fix duplicate ressource issue openstacklib::policy

Browse Source 
Currently, in the policy.pp file the class openstacklib::policy
is called. The same apply for all components resulting in a
Duplicate Ressource issue. Using directly the underlying ressource
openstacklib::policy::base we prevent this issue.

Change-Id: Ida23a774158f3275bc874d1b52a3424afeeb6fb2
This commit is contained in:
Yanis Guenane 2014-11-10 14:27:42 -05:00
parent c79a2a2c3a
commit b5dce39754
2 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
manifests/policy.pp
View File

@ -18,11 +18,12 @@ class cinder::policy (
$policy_path = '/etc/cinder/policy.json',
) {
validate_hash($policies)
Openstacklib::Policy::Base {
file_path => $policy_path,
}
class { 'openstacklib::policy' :
policies => $policies,
}
create_resources('openstacklib::policy::base', $policies)
}

5
spec/classes/cinder_policy_spec.rb
View File

@ -16,8 +16,9 @@ describe 'cinder::policy' do
end
it 'set up the policies' do
should contain_class('openstacklib::policy').with({
:policies => params[:policies]
should contain_openstacklib__policy__base('context_is_admin').with({
:key => 'context_is_admin',
:value => 'foo:bar'
})
end
end