openstack/puppet-cinder
Code Issues Proposed changes

Expose executor_thread_pool_size

Browse Source 
This option has been supported by puppet-oslo but has not been
configurable.

Change-Id: I54aa9068d40e1da01b72ee186e02ebfd3364f426
This commit is contained in:
Takashi Kajinami 2023-03-06 13:50:50 +09:00
parent d1c198ced9
commit 11f96e6c12
3 changed files with 17 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
manifests/init.pp
View File

@ -22,6 +22,10 @@
# (Optional) # (Optional)
# Defaults to $facts['os_service_default'] # Defaults to $facts['os_service_default']
# #
# [*executor_thread_pool_size*]
# (Optional) Size of executor thread pool when executor is threading or eventlet.
# Defaults to $facts['os_service_default'].
#
# [*notification_transport_url*] # [*notification_transport_url*]
# (Optional) A URL representing the messaging driver to use for notifications # (Optional) A URL representing the messaging driver to use for notifications
# and its full configuration. Transport URLs take the form: # and its full configuration. Transport URLs take the form:
@ -258,6 +262,7 @@ class cinder (
$default_transport_url = $facts['os_service_default'], $default_transport_url = $facts['os_service_default'],
$rpc_response_timeout = $facts['os_service_default'], $rpc_response_timeout = $facts['os_service_default'],
$control_exchange = $facts['os_service_default'], $control_exchange = $facts['os_service_default'],
$executor_thread_pool_size = $facts['os_service_default'],
$notification_transport_url = $facts['os_service_default'], $notification_transport_url = $facts['os_service_default'],
$notification_driver = $facts['os_service_default'], $notification_driver = $facts['os_service_default'],
$notification_topics = $facts['os_service_default'], $notification_topics = $facts['os_service_default'],
@ -358,6 +363,7 @@ class cinder (
} }
oslo::messaging::default { 'cinder_config': oslo::messaging::default { 'cinder_config':
executor_thread_pool_size => $executor_thread_pool_size,
transport_url => $default_transport_url, transport_url => $default_transport_url,
rpc_response_timeout => $rpc_response_timeout, rpc_response_timeout => $rpc_response_timeout,
control_exchange => $control_exchange, control_exchange => $control_exchange,

4
releasenotes/notes/executor_thread_pool_size-5daabf2eb43dfdd4.yaml Normal file
View File

@ -0,0 +1,4 @@
---
features:
- |
The new ``cinder::executor_thread_pool_size`` parameter has been added.

1
spec/classes/cinder_init_spec.rb
View File

@ -19,6 +19,7 @@ describe 'cinder' do
it { it {
is_expected.to contain_oslo__messaging__default('cinder_config').with( is_expected.to contain_oslo__messaging__default('cinder_config').with(
:executor_thread_pool_size => '<SERVICE DEFAULT>',
:transport_url => '<SERVICE DEFAULT>', :transport_url => '<SERVICE DEFAULT>',
:rpc_response_timeout => '<SERVICE DEFAULT>', :rpc_response_timeout => '<SERVICE DEFAULT>',
:control_exchange => '<SERVICE DEFAULT>' :control_exchange => '<SERVICE DEFAULT>'