Support new os-brick options

Add support for the new options added in os-brick 6.9.0 .

Depends-on: https://review.opendev.org/930364
Change-Id: I1763188b3f0f6722aac5b826d7fd657f7f5f1853
This commit is contained in:
Takashi Kajinami 2024-09-25 02:17:02 +09:00
parent c716870fc9
commit 204a7f21d7
3 changed files with 32 additions and 5 deletions

View File

@ -8,11 +8,24 @@
# (Optional) Directory to use for os-brick lock files.
# Defaults to $facts['os_service_default']
#
# [*wait_mpath_device_attempts*]
# (Optional) Number of attempts for the multipath device to be ready for I/O
# after it was created.
# Defaults to $facts['os_service_default']
#
# [*wait_mpath_device_interval*]
# (Optional) Interval value to wait for multipath device to be ready for I/O.
# Defaults to $facts['os_service_default']
#
class cinder::os_brick(
$lock_path = $facts['os_service_default'],
$lock_path = $facts['os_service_default'],
$wait_mpath_device_attempts = $facts['os_service_default'],
$wait_mpath_device_interval = $facts['os_service_default'],
) {
oslo::os_brick { 'cinder_config':
lock_path => $lock_path
lock_path => $lock_path,
wait_mpath_device_attempts => $wait_mpath_device_attempts,
wait_mpath_device_interval => $wait_mpath_device_interval,
}
}

View File

@ -0,0 +1,8 @@
---
features:
- |
The following parameters have been added to the ``cinder::os_brick``
class.
- ``wait_mpath_device_attempts``
- ``wait_mpath_device_interval``

View File

@ -7,7 +7,9 @@ describe 'cinder::os_brick' do
context 'with defaults' do
it 'configures the default values' do
is_expected.to contain_oslo__os_brick('cinder_config').with(
:lock_path => '<SERVICE DEFAULT>',
:lock_path => '<SERVICE DEFAULT>',
:wait_mpath_device_attempts => '<SERVICE DEFAULT>',
:wait_mpath_device_interval => '<SERVICE DEFAULT>',
)
end
end
@ -15,13 +17,17 @@ describe 'cinder::os_brick' do
context 'with parameters overridden' do
let :params do
{
:lock_path => '/var/lib/openstack/lock'
:lock_path => '/var/lib/openstack/lock',
:wait_mpath_device_attempts => 4,
:wait_mpath_device_interval => 1,
}
end
it 'configures the overridden values' do
is_expected.to contain_oslo__os_brick('cinder_config').with(
:lock_path => '/var/lib/openstack/lock',
:lock_path => '/var/lib/openstack/lock',
:wait_mpath_device_attempts => 4,
:wait_mpath_device_interval => 1,
)
end
end