From 9a2a82b4b081bfd0c61f4900f831e4006e2f8ecd Mon Sep 17 00:00:00 2001 From: Thomas Goirand Date: Thu, 1 Apr 2021 23:13:30 +0200 Subject: [PATCH] Allow to configure policy_dirs This patch makes it possible to override the current service default, which is /etc//policy.d . Change-Id: Ideeb71d8fbdcffba89c1c98eebd9ca21f2403eee --- manifests/policy.pp | 8 +++++++- releasenotes/notes/policy-dirs-3354de166c13f99b.yaml | 5 +++++ spec/classes/murano_policy_spec.rb | 2 ++ 3 files changed, 14 insertions(+), 1 deletion(-) create mode 100644 releasenotes/notes/policy-dirs-3354de166c13f99b.yaml diff --git a/manifests/policy.pp b/manifests/policy.pp index 192e57d..9d27e6c 100644 --- a/manifests/policy.pp +++ b/manifests/policy.pp @@ -32,11 +32,16 @@ # (Optional) Path to the murano policy.yaml file # Defaults to /etc/murano/policy.yaml # +# [*policy_dirs*] +# (Optional) Path to the murano policy folder +# Defaults to $::os_service_default +# class murano::policy ( $enforce_scope = $::os_service_default, $enforce_new_defaults = $::os_service_default, $policies = {}, $policy_path = '/etc/murano/policy.yaml', + $policy_dirs = $::os_service_default, ) { include murano::deps @@ -56,7 +61,8 @@ class murano::policy ( oslo::policy { 'murano_config': enforce_scope => $enforce_scope, enforce_new_defaults => $enforce_new_defaults, - policy_file => $policy_path + policy_file => $policy_path, + policy_dirs => $policy_dirs, } } diff --git a/releasenotes/notes/policy-dirs-3354de166c13f99b.yaml b/releasenotes/notes/policy-dirs-3354de166c13f99b.yaml new file mode 100644 index 0000000..e87d70f --- /dev/null +++ b/releasenotes/notes/policy-dirs-3354de166c13f99b.yaml @@ -0,0 +1,5 @@ +--- +features: + - | + There is now a new policy_dirs parameter in the murano::policy class, + so one can set a custom path. diff --git a/spec/classes/murano_policy_spec.rb b/spec/classes/murano_policy_spec.rb index 972303e..c702332 100644 --- a/spec/classes/murano_policy_spec.rb +++ b/spec/classes/murano_policy_spec.rb @@ -7,6 +7,7 @@ describe 'murano::policy' do :enforce_scope => false, :enforce_new_defaults => false, :policy_path => '/etc/murano/policy.yaml', + :policy_dirs => '/etc/murano/policy.d', :policies => { 'context_is_admin' => { 'key' => 'context_is_admin', @@ -28,6 +29,7 @@ describe 'murano::policy' do :enforce_scope => false, :enforce_new_defaults => false, :policy_file => '/etc/murano/policy.yaml', + :policy_dirs => '/etc/murano/policy.d', ) end end