openstack/puppet-cinder
Code Issues Proposed changes

nvmeof: Add reserved_percentage

Browse Source 
This defined resource type was overlooked when the reserved_percentage
option was added[1].

[1] 5451b7135b

Change-Id: I602f47cb06eff1c41b47abe36853bc418a122332
This commit is contained in:
Takashi Kajinami 2024-03-17 16:15:51 +09:00
parent af07a1a46b
commit 634b6152c1
3 changed files with 12 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
manifests/backend/nvmeof.pp
View File

@ -50,6 +50,10 @@
# (Optional) Max number of entries allowed in the image volume cache. # (Optional) Max number of entries allowed in the image volume cache.
# Defaults to $facts['os_service_default'], # Defaults to $facts['os_service_default'],
# #
# [*reserved_percentage*]
# (Optional) The percentage of backend capacity is reserved.
# Defaults to $facts['os_service_default'].
#
# [*volume_driver*] # [*volume_driver*]
# (Optional) Driver to use for volume creation # (Optional) Driver to use for volume creation
# Defaults to 'cinder.volume.drivers.lvm.LVMVolumeDriver'. # Defaults to 'cinder.volume.drivers.lvm.LVMVolumeDriver'.
@ -83,6 +87,7 @@ define cinder::backend::nvmeof (
$image_volume_cache_enabled = $facts['os_service_default'], $image_volume_cache_enabled = $facts['os_service_default'],
$image_volume_cache_max_size_gb = $facts['os_service_default'], $image_volume_cache_max_size_gb = $facts['os_service_default'],
$image_volume_cache_max_count = $facts['os_service_default'], $image_volume_cache_max_count = $facts['os_service_default'],
$reserved_percentage = $facts['os_service_default'],
$volume_driver = 'cinder.volume.drivers.lvm.LVMVolumeDriver', $volume_driver = 'cinder.volume.drivers.lvm.LVMVolumeDriver',
$volume_group = $facts['os_service_default'], $volume_group = $facts['os_service_default'],
$nvmeof_conn_info_version = $facts['os_service_default'], $nvmeof_conn_info_version = $facts['os_service_default'],
@ -106,6 +111,7 @@ define cinder::backend::nvmeof (
"${name}/image_volume_cache_enabled": value => $image_volume_cache_enabled; "${name}/image_volume_cache_enabled": value => $image_volume_cache_enabled;
"${name}/image_volume_cache_max_size_gb": value => $image_volume_cache_max_size_gb; "${name}/image_volume_cache_max_size_gb": value => $image_volume_cache_max_size_gb;
"${name}/image_volume_cache_max_count": value => $image_volume_cache_max_count; "${name}/image_volume_cache_max_count": value => $image_volume_cache_max_count;
"${name}/reserved_percentage": value => $reserved_percentage;
"${name}/volume_driver": value => $volume_driver; "${name}/volume_driver": value => $volume_driver;
"${name}/volume_group": value => $volume_group; "${name}/volume_group": value => $volume_group;
"${name}/nvmeof_conn_info_version": value => $nvmeof_conn_info_version; "${name}/nvmeof_conn_info_version": value => $nvmeof_conn_info_version;

5
releasenotes/notes/nvmeof-reserved_percentage-4609d4aae48b5e37.yaml Normal file
View File

@ -0,0 +1,5 @@
---
features:
- |
The ``cinder::backend::nvmeof::reserved_percentage`` parameter has been
added.

1
spec/defines/cinder_backend_nvmeof_spec.rb
View File

@ -29,6 +29,7 @@ describe 'cinder::backend::nvmeof' do
is_expected.to contain_cinder_config('nvme-backend/image_volume_cache_enabled').with_value('<SERVICE DEFAULT>') is_expected.to contain_cinder_config('nvme-backend/image_volume_cache_enabled').with_value('<SERVICE DEFAULT>')
is_expected.to contain_cinder_config('nvme-backend/image_volume_cache_max_size_gb').with_value('<SERVICE DEFAULT>') is_expected.to contain_cinder_config('nvme-backend/image_volume_cache_max_size_gb').with_value('<SERVICE DEFAULT>')
is_expected.to contain_cinder_config('nvme-backend/image_volume_cache_max_count').with_value('<SERVICE DEFAULT>') is_expected.to contain_cinder_config('nvme-backend/image_volume_cache_max_count').with_value('<SERVICE DEFAULT>')
is_expected.to contain_cinder_config('nvme-backend/reserved_percentage').with_value('<SERVICE DEFAULT>')
is_expected.to contain_cinder_config('nvme-backend/volume_driver').with_value('cinder.volume.drivers.lvm.LVMVolumeDriver') is_expected.to contain_cinder_config('nvme-backend/volume_driver').with_value('cinder.volume.drivers.lvm.LVMVolumeDriver')
is_expected.to contain_cinder_config('nvme-backend/nvmeof_conn_info_version').with_value('<SERVICE DEFAULT>') is_expected.to contain_cinder_config('nvme-backend/nvmeof_conn_info_version').with_value('<SERVICE DEFAULT>')
is_expected.to contain_cinder_config('nvme-backend/lvm_share_target').with_value('<SERVICE DEFAULT>') is_expected.to contain_cinder_config('nvme-backend/lvm_share_target').with_value('<SERVICE DEFAULT>')