openstack/puppet-cinder
Code Issues Proposed changes

correct package name for cinder backup

Browse Source 
the default backup_driver was mistyped and could not be loaded

Change-Id: I54eceeb366a7ccd7e78b382fcb8830106f9153d3
Closes-Bug: 1380716
This commit is contained in:
Oliver Boettcher 2014-10-14 11:46:38 +02:00
parent c9271e02d0
commit 2a09e402ee
2 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
manifests/backup/ceph.pp
View File

@ -21,6 +21,10 @@
#
# === Parameters
#
# [*backup_driver*]
# (optional) Which cinder backup driver to use
# Defaults to 'cinder.backup.drivers.ceph'
#
# [*backup_ceph_conf*]
# (optional) Ceph config file to use.
# Should be a valid ceph configuration file
@ -54,7 +58,7 @@
#
class cinder::backup::ceph (
$backup_driver = 'cinder.backup.driver.ceph',
$backup_driver = 'cinder.backup.drivers.ceph',
$backup_ceph_conf = '/etc/ceph/ceph.conf',
$backup_ceph_user = 'cinder',
$backup_ceph_chunk_size = '134217728',

2
spec/classes/cinder_backup_ceph_spec.rb
View File

@ -41,7 +41,7 @@ describe 'cinder::backup::ceph' do
end
it 'configures cinder.conf' do
should contain_cinder_config('DEFAULT/backup_driver').with_value('cinder.backup.driver.ceph')
should contain_cinder_config('DEFAULT/backup_driver').with_value('cinder.backup.drivers.ceph')
should contain_cinder_config('DEFAULT/backup_ceph_conf').with_value(p[:backup_ceph_conf])
should contain_cinder_config('DEFAULT/backup_ceph_user').with_value(p[:backup_ceph_user])
should contain_cinder_config('DEFAULT/backup_ceph_chunk_size').with_value(p[:backup_ceph_chunk_size])