openstack/puppet-cloudkitty
Code Issues Proposed changes

Merge "Import base parameter types for ::policy"

Browse Source
This commit is contained in:
Zuul
2025-09-29 15:20:32 +00:00
committed by Gerrit Code Review
parent eb2c26940a e3fcfed4e9
commit 8279174b2c
2 changed files with 7 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
manifests/policy.pp
View File

@@ -46,13 +46,13 @@
# Defaults to false.
#
class cloudkitty::policy (
$enforce_scope = $facts['os_service_default'],
$enforce_new_defaults = $facts['os_service_default'],
Hash $policies = {},
$policy_path = '/etc/cloudkitty/policy.yaml',
$policy_default_rule = $facts['os_service_default'],
$policy_dirs = $facts['os_service_default'],
Boolean $purge_config = false,
$enforce_scope = $facts['os_service_default'],
$enforce_new_defaults = $facts['os_service_default'],
Openstacklib::Policies $policies = {},
Stdlib::Absolutepath $policy_path = '/etc/cloudkitty/policy.yaml',
$policy_default_rule = $facts['os_service_default'],
$policy_dirs = $facts['os_service_default'],
Boolean $purge_config = false,
) {
include cloudkitty::deps
include cloudkitty::params
@@ -64,7 +64,6 @@ class cloudkitty::policy (
file_group => $cloudkitty::params::group,
file_format => 'yaml',
purge_config => $purge_config,
tag => 'cloudkitty',
}
create_resources('openstacklib::policy', { $policy_path => $policy_parameters })

2
spec/classes/cloudkitty_policy_spec.rb
View File

@@ -33,7 +33,6 @@ describe 'cloudkitty::policy' do
:file_group => 'cloudkitty',
:file_format => 'yaml',
:purge_config => false,
:tag => 'cloudkitty',
)
is_expected.to contain_oslo__policy('cloudkitty_config').with(
:enforce_scope => false,
@@ -64,7 +63,6 @@ describe 'cloudkitty::policy' do
:file_group => 'cloudkitty',
:file_format => 'yaml',
:purge_config => true,
:tag => 'cloudkitty',
)
is_expected.to contain_oslo__policy('cloudkitty_config').with(
:enforce_scope => false,