diff --git a/manifests/backend/multistore/cinder.pp b/manifests/backend/multistore/cinder.pp index 405eae48..621bd40a 100644 --- a/manifests/backend/multistore/cinder.pp +++ b/manifests/backend/multistore/cinder.pp @@ -114,7 +114,7 @@ define glance::backend::multistore::cinder( "${name}/cinder_store_auth_address": value => $cinder_store_auth_address; "${name}/cinder_store_project_name": value => $cinder_store_project_name; "${name}/cinder_store_user_name": value => $cinder_store_user_name; - "${name}/cinder_store_password": value => $cinder_store_password; + "${name}/cinder_store_password": value => $cinder_store_password, secret => true; "${name}/cinder_os_region_name": value => $cinder_os_region_name_real; "${name}/cinder_enforce_multipath": value => $cinder_enforce_multipath; "${name}/cinder_use_multipath": value => $cinder_use_multipath; @@ -131,11 +131,11 @@ define glance::backend::multistore::cinder( "${name}/cinder_store_auth_address": value => $cinder_store_auth_address; "${name}/cinder_store_project_name": value => $cinder_store_project_name; "${name}/cinder_store_user_name": value => $cinder_store_user_name; + "${name}/cinder_store_password": value => $cinder_store_password, secret => true; "${name}/cinder_os_region_name": value => $cinder_os_region_name_real; "${name}/cinder_enforce_multipath": value => $cinder_enforce_multipath; "${name}/cinder_mount_point_base": value => $cinder_mount_point_base; "${name}/cinder_use_multipath": value => $cinder_use_multipath; - "${name}/cinder_store_password": value => $cinder_store_password; } create_resources('glance_api_config', {}) diff --git a/spec/defines/glance_backend_multistore_cinder_spec.rb b/spec/defines/glance_backend_multistore_cinder_spec.rb index 86b07361..e98a44ce 100644 --- a/spec/defines/glance_backend_multistore_cinder_spec.rb +++ b/spec/defines/glance_backend_multistore_cinder_spec.rb @@ -39,7 +39,7 @@ describe 'glance::backend::multistore::cinder' do is_expected.to contain_glance_api_config('cinder/cinder_store_auth_address').with_value('') is_expected.to contain_glance_api_config('cinder/cinder_store_project_name').with_value('') is_expected.to contain_glance_api_config('cinder/cinder_store_user_name').with_value('') - is_expected.to contain_glance_api_config('cinder/cinder_store_password').with_value('') + is_expected.to contain_glance_api_config('cinder/cinder_store_password').with_value('').with_secret(true) is_expected.to contain_glance_api_config('cinder/cinder_os_region_name').with_value('RegionOne') is_expected.to contain_glance_api_config('cinder/cinder_enforce_multipath').with_value('') is_expected.to contain_glance_api_config('cinder/cinder_use_multipath').with_value('') @@ -55,7 +55,7 @@ describe 'glance::backend::multistore::cinder' do is_expected.to contain_glance_cache_config('cinder/cinder_store_auth_address').with_value('') is_expected.to contain_glance_cache_config('cinder/cinder_store_project_name').with_value('') is_expected.to contain_glance_cache_config('cinder/cinder_store_user_name').with_value('') - is_expected.to contain_glance_cache_config('cinder/cinder_store_password').with_value('') + is_expected.to contain_glance_cache_config('cinder/cinder_store_password').with_value('').with_secret(true) is_expected.to contain_glance_cache_config('cinder/cinder_os_region_name').with_value('RegionOne') is_expected.to contain_glance_cache_config('cinder/cinder_enforce_multipath').with_value('') is_expected.to contain_glance_cache_config('cinder/cinder_use_multipath').with_value('') @@ -92,7 +92,7 @@ describe 'glance::backend::multistore::cinder' do is_expected.to contain_glance_api_config('cinder/cinder_store_auth_address').with_value('127.0.0.2:8080/v3/') is_expected.to contain_glance_api_config('cinder/cinder_store_project_name').with_value('services') is_expected.to contain_glance_api_config('cinder/cinder_store_user_name').with_value('glance') - is_expected.to contain_glance_api_config('cinder/cinder_store_password').with_value('glance') + is_expected.to contain_glance_api_config('cinder/cinder_store_password').with_value('glance').with_secret(true) is_expected.to contain_glance_api_config('cinder/cinder_os_region_name').with_value('RegionTwo') is_expected.to contain_glance_api_config('cinder/cinder_enforce_multipath').with_value(true) is_expected.to contain_glance_api_config('cinder/cinder_use_multipath').with_value(true) @@ -108,7 +108,7 @@ describe 'glance::backend::multistore::cinder' do is_expected.to contain_glance_cache_config('cinder/cinder_store_auth_address').with_value('127.0.0.2:8080/v3/') is_expected.to contain_glance_cache_config('cinder/cinder_store_project_name').with_value('services') is_expected.to contain_glance_cache_config('cinder/cinder_store_user_name').with_value('glance') - is_expected.to contain_glance_cache_config('cinder/cinder_store_password').with_value('glance') + is_expected.to contain_glance_cache_config('cinder/cinder_store_password').with_value('glance').with_secret(true) is_expected.to contain_glance_cache_config('cinder/cinder_os_region_name').with_value('RegionTwo') is_expected.to contain_glance_cache_config('cinder/cinder_enforce_multipath').with_value(true) is_expected.to contain_glance_cache_config('cinder/cinder_use_multipath').with_value(true)