openstack/puppet-cinder
Code Issues Proposed changes

Merge "Make scheduler_driver option can be clean up"

Browse Source
This commit is contained in:
Jenkins
2015-03-26 11:01:10 +00:00
committed by Gerrit Code Review
parent a8552ca2ba 54d77c27ae
commit 77e1c46656
2 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
manifests/scheduler.pp
View File

@@ -38,6 +38,10 @@ class cinder::scheduler (
cinder_config {
'DEFAULT/scheduler_driver': value => $scheduler_driver;
}
} else {
cinder_config {
'DEFAULT/scheduler_driver': ensure => absent;
}
}
if $::cinder::params::scheduler_package {

3
spec/classes/cinder_scheduler_spec.rb
View File

@@ -11,11 +11,12 @@ describe 'cinder::scheduler' do
describe 'with default parameters' do
it { is_expected.to contain_class('cinder::params') }
it { is_expected.to contain_cinder_config('DEFAULT/scheduler_driver').with_ensure('absent') }
it { is_expected.to contain_package('cinder-scheduler').with(
:name => 'cinder-scheduler',
:ensure => 'present',
:before => 'Service[cinder-scheduler]'
:before => ['Cinder_config[DEFAULT/scheduler_driver]','Service[cinder-scheduler]'],
) }
it { is_expected.to contain_service('cinder-scheduler').with(