openstack/puppet-heat
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: Ic07c62593d63190b523efcc3f020cba12b5001f3
This commit is contained in:
ZhongShengping 2016-11-08 10:05:42 +08:00
parent a6a860106b
commit d727cf872f
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
manifests/params.pp
View File

@ -5,6 +5,8 @@
class heat::params {
include ::openstacklib::defaults
$client_package_name = 'python-heatclient'
case $::osfamily {
'RedHat': {
# package names
@ -12,7 +14,6 @@ class heat::params {
$api_cloudwatch_package_name = 'openstack-heat-api-cloudwatch'
$api_cfn_package_name = 'openstack-heat-api-cfn'
$engine_package_name = 'openstack-heat-engine'
$client_package_name = 'python-heatclient'
$common_package_name = 'openstack-heat-common'
# service names
$api_service_name = 'openstack-heat-api'
@ -26,7 +27,6 @@ class heat::params {
$api_cloudwatch_package_name = 'heat-api-cloudwatch'
$api_cfn_package_name = 'heat-api-cfn'
$engine_package_name = 'heat-engine'
$client_package_name = 'python-heatclient'
$common_package_name = 'heat-common'
# service names
$api_service_name = 'heat-api'