diff --git a/manifests/params.pp b/manifests/params.pp index 3463cc5d..c9634169 100644 --- a/manifests/params.pp +++ b/manifests/params.pp @@ -14,6 +14,7 @@ class gnocchi::params { $metricd_service_name = 'gnocchi-metricd' $statsd_package_name = 'gnocchi-statsd' $statsd_service_name = 'gnocchi-statsd' + $group = 'gnocchi' case $::osfamily { 'RedHat': { diff --git a/manifests/policy.pp b/manifests/policy.pp index fe156652..899c1573 100644 --- a/manifests/policy.pp +++ b/manifests/policy.pp @@ -29,11 +29,14 @@ class gnocchi::policy ( ) { include ::gnocchi::deps + include ::gnocchi::params validate_hash($policies) Openstacklib::Policy::Base { - file_path => $policy_path, + file_path => $policy_path, + file_user => 'root', + file_group => $::gnocchi::params::group, } create_resources('openstacklib::policy::base', $policies) diff --git a/spec/classes/gnocchi_policy_spec.rb b/spec/classes/gnocchi_policy_spec.rb index 409d4973..5d29b13a 100644 --- a/spec/classes/gnocchi_policy_spec.rb +++ b/spec/classes/gnocchi_policy_spec.rb @@ -17,8 +17,10 @@ describe 'gnocchi::policy' do it 'set up the policies' do is_expected.to contain_openstacklib__policy__base('context_is_admin').with({ - :key => 'context_is_admin', - :value => 'foo:bar' + :key => 'context_is_admin', + :value => 'foo:bar', + :file_user => 'root', + :file_group => 'gnocchi', }) is_expected.to contain_oslo__policy('gnocchi_config').with( :policy_file => '/etc/gnocchi/policy.json', @@ -34,8 +36,7 @@ describe 'gnocchi::policy' do facts.merge!(OSDefaults.get_facts()) end - it_behaves_like 'gnocchi policies' + it_configures 'gnocchi policies' end end - end