openstack
/
puppet-congress
Code Issues Proposed changes

Merge "updating service name"

This commit is contained in:
Jenkins 2016-12-22 15:04:14 +00:00 committed by Gerrit Code Review
parent cb05afc9fe 90cf1892e9
commit 74f2c9dec1
2 changed files with 11 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
manifests/params.pp
View File

@ -15,7 +15,7 @@ class congress::params {
case $::osfamily {
'RedHat': {
$package_name = 'openstack-congress'
$service_name = 'congress-server'
$service_name = 'openstack-congress'
}
'Debian': {
$package_name = 'congress-server'

11
spec/classes/congress_server_spec.rb
View File

@ -37,7 +37,7 @@ describe 'congress::server' do
it 'configures congress-server service' do
is_expected.to contain_service('congress-server').with(
:ensure => (params[:manage_service] && params[:enabled]) ? 'running' : 'stopped',
:name => 'congress-server',
:name => platform_params[:congress_service],
:enable => params[:enabled],
:tag => 'congress-service',
)
@ -55,6 +55,15 @@ describe 'congress::server' do
facts.merge!(OSDefaults.get_facts())
end
let(:platform_params) do
case facts[:osfamily]
when 'Debian'
{ :congress_service => 'congress-server' }
when 'RedHat'
{ :congress_service => 'openstack-congress' }
end
end
it_configures 'congress::server'
end
end