Horizon service cleanups for hiera json hook
This patch resolves a few issues I noticed when porting our Horizon service to support the new heat hiera agent hook (which uses Json instead of Yaml). -we only need to set django_debug if the string is non-empty. This should match previous behavior. -remove the duplicated NeutronMechanismDrivers setting. This is already managed in the neutron services and shouldn't be set here. Change-Id: I473e110bb9b14cb8f57d41c4fc398871548726b0 Partial-bug: #1596373
This commit is contained in:
parent
b7cfa7f7b8
commit
133edad130
|
@ -24,7 +24,7 @@ parameters:
|
|||
type: json
|
||||
HorizonAllowedHosts:
|
||||
default: '*'
|
||||
description: A list of IP/Hostname for the server Horizonis running on.
|
||||
description: A list of IP/Hostname for the server Horizon is running on.
|
||||
Used for header checks.
|
||||
type: comma_delimited_list
|
||||
HorizonSecret:
|
||||
|
@ -32,11 +32,6 @@ parameters:
|
|||
type: string
|
||||
hidden: true
|
||||
default: ''
|
||||
NeutronMechanismDrivers:
|
||||
default: 'openvswitch'
|
||||
description: |
|
||||
The mechanism drivers for the Neutron tenant network.
|
||||
type: comma_delimited_list
|
||||
MemcachedIPv6:
|
||||
default: false
|
||||
description: Enable IPv6 features in Memcached.
|
||||
|
@ -45,6 +40,10 @@ parameters:
|
|||
default: 'overcloud-horizon'
|
||||
type: string
|
||||
|
||||
conditions:
|
||||
|
||||
debug_empty: {equals : [{get_param: Debug}, '']}
|
||||
|
||||
outputs:
|
||||
role_data:
|
||||
description: Role data for the Horizon role.
|
||||
|
@ -52,34 +51,34 @@ outputs:
|
|||
service_name: horizon
|
||||
monitoring_subscription: {get_param: MonitoringSubscriptionHorizon}
|
||||
config_settings:
|
||||
horizon::allowed_hosts: {get_param: HorizonAllowedHosts}
|
||||
neutron::plugins::ml2::mechanism_drivers:
|
||||
str_replace:
|
||||
template: MECHANISMS
|
||||
params:
|
||||
MECHANISMS: {get_param: NeutronMechanismDrivers}
|
||||
tripleo.horizon.firewall_rules:
|
||||
'126 horizon':
|
||||
dport:
|
||||
- 80
|
||||
- 443
|
||||
horizon::disable_password_reveal: true
|
||||
horizon::cache_backend: django.core.cache.backends.memcached.MemcachedCache
|
||||
horizon::django_session_engine: 'django.contrib.sessions.backends.cache'
|
||||
horizon::vhost_extra_params:
|
||||
add_listen: false
|
||||
priority: 10
|
||||
access_log_format: '%a %l %u %t \"%r\" %>s %b \"%%{}{Referer}i\" \"%%{}{User-Agent}i\"'
|
||||
horizon::bind_address: {get_param: [ServiceNetMap, HorizonNetwork]}
|
||||
horizon::django_debug: {get_param: Debug}
|
||||
horizon::keystone_url: {get_param: [EndpointMap, KeystoneInternal, uri]}
|
||||
horizon::secret_key:
|
||||
yaql:
|
||||
expression: $.data.passwords.where($ != '').first()
|
||||
data:
|
||||
passwords:
|
||||
- {get_param: HorizonSecret}
|
||||
- {get_param: [DefaultPasswords, horizon_secret]}
|
||||
memcached_ipv6: {get_param: MemcachedIPv6}
|
||||
map_merge:
|
||||
- horizon::allowed_hosts: {get_param: HorizonAllowedHosts}
|
||||
tripleo.horizon.firewall_rules:
|
||||
'126 horizon':
|
||||
dport:
|
||||
- 80
|
||||
- 443
|
||||
horizon::disable_password_reveal: true
|
||||
horizon::cache_backend: django.core.cache.backends.memcached.MemcachedCache
|
||||
horizon::django_session_engine: 'django.contrib.sessions.backends.cache'
|
||||
horizon::vhost_extra_params:
|
||||
add_listen: false
|
||||
priority: 10
|
||||
access_log_format: '%a %l %u %t \"%r\" %>s %b \"%%{}{Referer}i\" \"%%{}{User-Agent}i\"'
|
||||
horizon::bind_address: {get_param: [ServiceNetMap, HorizonNetwork]}
|
||||
horizon::keystone_url: {get_param: [EndpointMap, KeystoneInternal, uri]}
|
||||
horizon::secret_key:
|
||||
yaql:
|
||||
expression: $.data.passwords.where($ != '').first()
|
||||
data:
|
||||
passwords:
|
||||
- {get_param: HorizonSecret}
|
||||
- {get_param: [DefaultPasswords, horizon_secret]}
|
||||
memcached_ipv6: {get_param: MemcachedIPv6}
|
||||
-
|
||||
if:
|
||||
- debug_empty
|
||||
- {}
|
||||
- horizon::django_debug: {get_param: Debug}
|
||||
step_config: |
|
||||
include ::tripleo::profile::base::horizon
|
||||
|
|
Loading…
Reference in New Issue