openstack/puppet-trove
Code Issues Proposed changes

Fix common options position in params.pp

Browse Source 
These parameters are common for both systems,
they could be moved out from 'case'.

Change-Id: I3b1c2243dc3d0c8f7cebad6a231f2bdda6b4bdb6
This commit is contained in:
ZhongShengping 2016-11-08 11:40:07 +08:00
parent 1032a9e41e
commit 9ed939888b
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
manifests/params.pp
View File

@ -1,11 +1,14 @@
# == Class: trove::params
#
# Parameters for puppet-trove
#
class trove::params {
include ::openstacklib::defaults
$client_package_name = 'python-troveclient'
case $::osfamily {
'RedHat': {
$client_package_name = 'python-troveclient'
$common_package_name = 'openstack-trove'
$conductor_package_name = 'openstack-trove-conductor'
$conductor_service_name = 'openstack-trove-conductor'
@ -17,7 +20,6 @@ class trove::params {
$taskmanager_service_name = 'openstack-trove-taskmanager'
}
'Debian': {
$client_package_name = 'python-troveclient'
$common_package_name = 'trove-common'
$conductor_package_name = 'trove-conductor'
$conductor_service_name = 'trove-conductor'