openstack/puppet-cinder
Code Issues Proposed changes

Default setting san_thin_provision should be true

Browse Source 
Currently cinder eqlx_san_thin_provision option is false to
disable creation of thin-provisioned volumes.
The default should be true to enable thin provisioning which is the recommended setting.

There is a Openstack launchpad bug to fix documentation:
https://bugs.launchpad.net/openstack-manuals/+bug/1384927

Change-Id: I4252da3e60004998c3cd15921ac403bd8969ee57
This commit is contained in:
rajinir
2014-10-27 13:28:58 -05:00
parent 987c51bc09
commit 6d90b9c5f3
2 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
manifests/backend/eqlx.pp
View File

@@ -15,7 +15,7 @@
# #
# [*san_thin_provision*] # [*san_thin_provision*]
# (optional) Whether or not to use thin provisioning for volumes. # (optional) Whether or not to use thin provisioning for volumes.
# Defaults to false # Defaults to true
# #
# [*volume_backend_name*] # [*volume_backend_name*]
# (optional) The backend name. # (optional) The backend name.
@@ -53,7 +53,7 @@ define cinder::backend::eqlx (
$san_ip, $san_ip,
$san_login, $san_login,
$san_password, $san_password,
$san_thin_provision = false, $san_thin_provision = true,
$volume_backend_name = $name, $volume_backend_name = $name,
$eqlx_group_name = 'group-0', $eqlx_group_name = 'group-0',
$eqlx_pool = 'default', $eqlx_pool = 'default',

4
manifests/volume/eqlx.pp
View File

@@ -15,7 +15,7 @@
# #
# [*san_thin_provision*] # [*san_thin_provision*]
# (optional) Whether or not to use thin provisioning for volumes. # (optional) Whether or not to use thin provisioning for volumes.
# Defaults to false # Defaults to true
# #
# [*eqlx_group_name*] # [*eqlx_group_name*]
# (optional) The CLI prompt message without '>'. # (optional) The CLI prompt message without '>'.
@@ -49,7 +49,7 @@ class cinder::volume::eqlx (
$san_ip, $san_ip,
$san_login, $san_login,
$san_password, $san_password,
$san_thin_provision = false, $san_thin_provision = true,
$eqlx_group_name = 'group-0', $eqlx_group_name = 'group-0',
$eqlx_pool = 'default', $eqlx_pool = 'default',
$eqlx_use_chap = false, $eqlx_use_chap = false,