Skip to content

Commit ce8e6f6

Browse files
author
John Griffith
committed
Add ability to specify cinder lvm_type option
Cinder has had an lvm_type option for quite a while that allows the LVM driver to be configured to use thin provisioning. This required an additional PPA in Precise, but is available in the default Trusty packages. This patch adds the lvm_type option, and we'll use it to do gate testing against the lvm_thin configuration. Change-Id: I99c7fea131f3d79747ec75052adf8b24f41ba483
1 parent c5caff6 commit ce8e6f6

2 files changed

Lines changed: 5 additions & 0 deletions

File tree

lib/cinder

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,10 @@ CINDER_SERVICE_PORT=${CINDER_SERVICE_PORT:-8776}
6565
CINDER_SERVICE_PORT_INT=${CINDER_SERVICE_PORT_INT:-18776}
6666
CINDER_SERVICE_PROTOCOL=${CINDER_SERVICE_PROTOCOL:-$SERVICE_PROTOCOL}
6767

68+
# What type of LVM device should Cinder use for LVM backend
69+
# Defaults to default, which is thick, the other valid choice
70+
# is thin, which as the name implies utilizes lvm thin provisioning.
71+
CINDER_LVM_TYPE=${CINDER_LVM_TYPE:-default}
6872

6973
# Default backends
7074
# The backend format is type:name where type is one of the supported backend

lib/cinder_backends/lvm

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,7 @@ function configure_cinder_backend_lvm {
5151
iniset $CINDER_CONF $be_name volume_driver "cinder.volume.drivers.lvm.LVMVolumeDriver"
5252
iniset $CINDER_CONF $be_name volume_group $VOLUME_GROUP_NAME-$be_name
5353
iniset $CINDER_CONF $be_name iscsi_helper "$CINDER_ISCSI_HELPER"
54+
iniset $CINDER_CONF $be_name lvm_type "$CINDER_LVM_TYPE"
5455

5556
if [[ "$CINDER_SECURE_DELETE" == "False" ]]; then
5657
iniset $CINDER_CONF $be_name volume_clear none

0 commit comments

Comments
 (0)