openstack
/
puppet-congress
Code Issues Proposed changes

updating service name 

red hat systems refer to the congress service as openstack-congress
updating the params to reflect this.

Change-Id: I300be98d6c6782d882018827aed7aeebc142e617
Signed-off-by: Dan Radez <dradez@redhat.com>
This commit is contained in:
Dan Radez 2016-12-21 21:51:56 -05:00
parent 2f6ba01b76
commit 90cf1892e9
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