Set hasstatus => true for cinder services
Fix bug 1170705 notifies to cinder services on RHEL were not causing a restart of the service Change-Id: I2fa06695acd068d14ff5a13de08538cf992f9005
This commit is contained in:
parent
8720c70956
commit
cde157b822
@ -39,6 +39,7 @@ class cinder::api (
|
|||||||
name => $::cinder::params::api_service,
|
name => $::cinder::params::api_service,
|
||||||
enable => $enabled,
|
enable => $enabled,
|
||||||
ensure => $ensure,
|
ensure => $ensure,
|
||||||
|
hasstatus => true,
|
||||||
require => Package['cinder'],
|
require => Package['cinder'],
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -37,6 +37,7 @@ class cinder::scheduler (
|
|||||||
name => $::cinder::params::scheduler_service,
|
name => $::cinder::params::scheduler_service,
|
||||||
enable => $enabled,
|
enable => $enabled,
|
||||||
ensure => $ensure,
|
ensure => $ensure,
|
||||||
|
hasstatus => true,
|
||||||
require => Package['cinder'],
|
require => Package['cinder'],
|
||||||
subscribe => File[$::cinder::params::cinder_conf],
|
subscribe => File[$::cinder::params::cinder_conf],
|
||||||
}
|
}
|
||||||
|
@ -30,6 +30,7 @@ class cinder::volume (
|
|||||||
name => $::cinder::params::volume_service,
|
name => $::cinder::params::volume_service,
|
||||||
enable => $enabled,
|
enable => $enabled,
|
||||||
ensure => $ensure,
|
ensure => $ensure,
|
||||||
|
hasstatus => true,
|
||||||
require => Package['cinder'],
|
require => Package['cinder'],
|
||||||
subscribe => File[$::cinder::params::cinder_conf],
|
subscribe => File[$::cinder::params::cinder_conf],
|
||||||
}
|
}
|
||||||
|
@ -13,6 +13,11 @@ describe 'cinder::api' do
|
|||||||
let :params do
|
let :params do
|
||||||
req_params
|
req_params
|
||||||
end
|
end
|
||||||
|
|
||||||
|
it { should contain_service('cinder-api').with(
|
||||||
|
'hasstatus' => 'true'
|
||||||
|
)}
|
||||||
|
|
||||||
it 'should configure cinder api correctly' do
|
it 'should configure cinder api correctly' do
|
||||||
should contain_cinder_config('DEFAULT/auth_strategy').with(
|
should contain_cinder_config('DEFAULT/auth_strategy').with(
|
||||||
:value => 'keystone'
|
:value => 'keystone'
|
||||||
|
@ -23,6 +23,7 @@ describe 'cinder::scheduler' do
|
|||||||
:enable => true,
|
:enable => true,
|
||||||
:ensure => 'running',
|
:ensure => 'running',
|
||||||
:require => 'Package[cinder]',
|
:require => 'Package[cinder]',
|
||||||
|
:hasstatus => 'true',
|
||||||
:subscribe => 'File[/etc/cinder/cinder.conf]'
|
:subscribe => 'File[/etc/cinder/cinder.conf]'
|
||||||
) }
|
) }
|
||||||
end
|
end
|
||||||
|
@ -11,5 +11,8 @@ describe 'cinder::volume' do
|
|||||||
end
|
end
|
||||||
|
|
||||||
it { should contain_package('cinder-volume') }
|
it { should contain_package('cinder-volume') }
|
||||||
it { should contain_service('cinder-volume') }
|
it { should contain_service('cinder-volume').with(
|
||||||
|
'hasstatus' => 'true'
|
||||||
|
)}
|
||||||
|
|
||||||
end
|
end
|
||||||
|
Loading…
x
Reference in New Issue
Block a user