Merge "Allow list values for enabled_extensions_admin/v2"

This commit is contained in:
Zuul 2021-07-19 14:48:34 +00:00 committed by Gerrit Code Review
commit f11b16f35a
2 changed files with 18 additions and 4 deletions

View File

@ -146,8 +146,8 @@ class designate::api (
'service:api/default_limit_v2' : value => $default_limit_v2;
'service:api/max_limit_v2' : value => $max_limit_v2;
'service:api/pecan_debug' : value => $pecan_debug;
'service:api/enabled_extensions_v2' : value => $enabled_extensions_v2;
'service:api/enabled_extensions_admin' : value => $enabled_extensions_admin;
'service:api/enabled_extensions_v2' : value => join(any2array($enabled_extensions_v2), ',');
'service:api/enabled_extensions_admin' : value => join(any2array($enabled_extensions_admin), ',');
}
if $auth_strategy == 'keystone' {

View File

@ -72,8 +72,8 @@ describe 'designate::api' do
:default_limit_v2 => '25',
:max_limit_v2 => '1500',
:pecan_debug => true,
:enabled_extensions_v2 => 'experimental',
:enabled_extensions_admin => 'reports,quotas,counts,tenants,target_sync',
:enabled_extensions_v2 => 'ext1,ext2',
:enabled_extensions_admin => 'reports,quotas,zones',
:enable_proxy_headers_parsing => true,
})
end
@ -101,6 +101,20 @@ describe 'designate::api' do
end
end
context 'with enabled_extentions in list' do
before do
params.merge!({
:enabled_extensions_v2 => ['ext1', 'ext2'],
:enabled_extensions_admin => ['reports', 'quotas', 'zones']
})
end
it 'configure service_api' do
is_expected.to contain_designate_config('service:api/enabled_extensions_v2').with_value('ext1,ext2')
is_expected.to contain_designate_config('service:api/enabled_extensions_admin').with_value('reports,quotas,zones')
end
end
context 'with custom package name' do
before do
params.merge!({ :api_package_name => 'designate-api-custom-name' })