stackforge/puppet-openstack-cloud
Code Issues Proposed changes

cinder/rbd: configure volume_tmp_dir to /tmp

Browse Source 
Set default temporary volume directory in rbd driver to /tmp, otherwise
we can hit some bugs in Cinder where it would be not possible to copy a
volume to an image.

https://bugs.launchpad.net/cinder/+bug/1288240

Signed-off-by: Emilien Macchi <emilien.macchi@enovance.com>
This commit is contained in:
Emilien Macchi 2014-05-22 23:54:18 +02:00
parent b7203bbaef
commit 3cefe2e529
2 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
manifests/volume/backend/rbd.pp
View File

@ -58,7 +58,6 @@ define cloud::volume::backend::rbd (
$rbd_ceph_conf = '/etc/ceph/ceph.conf',
$rbd_flatten_volume_from_snapshot = false,
$rbd_secret_uuid = false,
$volume_tmp_dir = false,
$rbd_max_clone_depth = '5',
) {
@ -69,6 +68,7 @@ define cloud::volume::backend::rbd (
rbd_ceph_conf => $rbd_ceph_conf,
rbd_flatten_volume_from_snapshot => $rbd_flatten_volume_from_snapshot,
rbd_max_clone_depth => $rbd_max_clone_depth,
volume_tmp_dir => '/tmp'
}
# Configure Ceph keyring

3
spec/classes/cloud_volume_storage_spec.rb
View File

@ -100,6 +100,7 @@ describe 'cloud::volume::storage' do
should contain_cinder_config('lowcost/rbd_pool').with_value('ceph_cinder')
should contain_cinder_config('lowcost/rbd_user').with_value('cinder')
should contain_cinder_config('lowcost/rbd_secret_uuid').with_value('secret')
should contain_cinder_config('lowcost/volume_tmp_dir').with_value('/tmp')
should contain_cinder__type('lowcost').with(
:set_key => 'volume_backend_name',
:set_value => 'lowcost',
@ -156,6 +157,7 @@ describe 'cloud::volume::storage' do
should contain_cinder_config('lowcost/rbd_pool').with_value('low')
should contain_cinder_config('lowcost/rbd_user').with_value('cinder')
should contain_cinder_config('lowcost/rbd_secret_uuid').with_value('secret')
should contain_cinder_config('lowcost/volume_tmp_dir').with_value('/tmp')
should contain_cinder__type('lowcost').with(
:set_key => 'volume_backend_name',
:set_value => 'lowcost',
@ -168,6 +170,7 @@ describe 'cloud::volume::storage' do
should contain_cinder_config('normal/rbd_pool').with_value('normal')
should contain_cinder_config('normal/rbd_user').with_value('cinder')
should contain_cinder_config('normal/rbd_secret_uuid').with_value('secret')
should contain_cinder_config('normal/volume_tmp_dir').with_value('/tmp')
should contain_cinder__type('normal').with(
:set_key => 'volume_backend_name',
:set_value => 'normal',