diff --git a/manifests/api.pp b/manifests/api.pp index 2d3e699c..ab3f621a 100644 --- a/manifests/api.pp +++ b/manifests/api.pp @@ -28,7 +28,7 @@ # # [*keymgr_api_class*] # (optional) Key Manager service class. -# Example of valid value: cinder.keymgr.barbican.BarbicanKeyManager +# Example of valid value: castellan.key_manager.barbican_key_manager.BarbicanKeyManager # Defaults to $::os_service_default # # [*keymgr_encryption_api_url*] @@ -376,9 +376,9 @@ class cinder::api ( } cinder_config { - 'keymgr/api_class': value => $keymgr_api_class; - 'keymgr/encryption_api_url': value => $keymgr_encryption_api_url; - 'keymgr/encryption_auth_url': value => $keymgr_encryption_auth_url; + 'key_manager/api_class': value => $keymgr_api_class; + 'barbican/barbican_endpoint': value => $keymgr_encryption_api_url; + 'barbican/auth_endpoint': value => $keymgr_encryption_auth_url; } if $auth_strategy_real { diff --git a/spec/classes/cinder_api_spec.rb b/spec/classes/cinder_api_spec.rb index efcc68e1..b093bd3d 100644 --- a/spec/classes/cinder_api_spec.rb +++ b/spec/classes/cinder_api_spec.rb @@ -67,9 +67,9 @@ describe 'cinder::api' do is_expected.to contain_cinder_config('DEFAULT/os_privileged_user_password').with_value('') is_expected.to contain_cinder_config('DEFAULT/os_privileged_user_tenant').with_value('') is_expected.to contain_cinder_config('DEFAULT/os_privileged_user_auth_url').with_value('') - is_expected.to contain_cinder_config('keymgr/api_class').with_value('') - is_expected.to contain_cinder_config('keymgr/encryption_api_url').with_value('') - is_expected.to contain_cinder_config('keymgr/encryption_auth_url').with_value('') + is_expected.to contain_cinder_config('key_manager/api_class').with_value('') + is_expected.to contain_cinder_config('barbican/barbican_endpoint').with_value('') + is_expected.to contain_cinder_config('barbican/auth_endpoint').with_value('') is_expected.to contain_cinder_config('oslo_middleware/enable_proxy_headers_parsing').with('value' => '') end end @@ -253,7 +253,7 @@ describe 'cinder::api' do req_params.merge({ :keymgr_encryption_auth_url => 'http://localhost:5000/v3' }) end - it { is_expected.to contain_cinder_config('keymgr/encryption_auth_url').with( + it { is_expected.to contain_cinder_config('barbican/auth_endpoint').with( :value => 'http://localhost:5000/v3' )} end @@ -380,15 +380,15 @@ describe 'cinder::api' do describe 'with barbican parameters' do let :params do req_params.merge!({ - :keymgr_api_class => 'cinder.keymgr.barbican.BarbicanKeyManager', + :keymgr_api_class => 'castellan.key_manager.barbican_key_manager.BarbicanKeyManager', :keymgr_encryption_api_url => 'https://localhost:9311/v1', :keymgr_encryption_auth_url => 'https://localhost:5000/v3', }) end it 'should set keymgr parameters' do - is_expected.to contain_cinder_config('keymgr/api_class').with_value('cinder.keymgr.barbican.BarbicanKeyManager') - is_expected.to contain_cinder_config('keymgr/encryption_api_url').with_value('https://localhost:9311/v1') - is_expected.to contain_cinder_config('keymgr/encryption_auth_url').with_value('https://localhost:5000/v3') + is_expected.to contain_cinder_config('key_manager/api_class').with_value('castellan.key_manager.barbican_key_manager.BarbicanKeyManager') + is_expected.to contain_cinder_config('barbican/barbican_endpoint').with_value('https://localhost:9311/v1') + is_expected.to contain_cinder_config('barbican/auth_endpoint').with_value('https://localhost:5000/v3') end end