openstack/puppet-cinder
Code Issues Proposed changes

Added support for enable_force_upload option in cinder class

Browse Source 
Change-Id: I2dda087b6a391e465b8530d6ef812947e4bc1ee9
This commit is contained in:
Sam Morrison 2020-11-06 14:21:39 +11:00
parent 75a506e1a9
commit 4b1085370e
3 changed files with 23 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
manifests/init.pp
View File

@ -245,6 +245,12 @@
# (optional) Backend override of host value.
# Defaults to $::os_service_default.
#
# [*enable_force_upload*]
# (optional) Enables the Force option on upload_to_image. This
# enables running upload_volume on in-use volumes for backends that
# support it.
# Defaults to $::os_service_default.
#
# DEPRECATED PARAMETERS
#
# [*database_min_pool_size*]
@ -332,6 +338,7 @@ class cinder (
$enable_new_services = $::os_service_default,
$purge_config = false,
$backend_host = $::os_service_default,
$enable_force_upload = $::os_service_default,
# DEPRECATED PARAMETERS
$database_min_pool_size = undef,
$database_connection = undef,
@ -461,10 +468,10 @@ removed in a future realse. Use cinder::db::database_max_overflow instead')
'DEFAULT/image_conversion_dir': value => $image_conversion_dir;
'DEFAULT/host': value => $host;
'DEFAULT/enable_new_services': value => $enable_new_services;
'DEFAULT/enable_force_upload': value => $enable_force_upload;
'key_manager/backend': value => $keymgr_backend_real;
'barbican/barbican_endpoint': value => $keymgr_encryption_api_url_real;
'barbican/auth_endpoint': value => $keymgr_encryption_auth_url_real;
# NOTE(abishop): $backend_host is not written here because it is not a valid
# DEFAULT option. It is only recognized in the backend sections. Instead,
# for backward compatibility, backends.pp references this parameter.

4
releasenotes/notes/add-enable_force_upload-option-0c91071ed62d1a70.yaml Normal file
View File

@ -0,0 +1,4 @@
---
features:
- |
Added ability to set ``enable_force_upload`` option in the ``cinder`` class

11
spec/classes/cinder_init_spec.rb
View File

@ -276,6 +276,17 @@ describe 'cinder' do
is_expected.to contain_cinder_config('barbican/auth_endpoint').with_value('https://localhost:5000/v3')
end
end
context 'with volume api paramaters' do
let :params do
req_params.merge!({
:enable_force_upload => true,
})
end
it 'should set volume api parameters' do
is_expected.to contain_cinder_config('DEFAULT/enable_force_upload').with_value(true)
end
end
end
on_supported_os({