openstack/openstack-ansible-os_horizon
Code Issues Proposed changes

Fix wrong neutron_ml2_drivers_type

Browse Source 
neutron_ml2_drivers_type is a string, not a list, while default
value for it is a list. Local settings also assume that it is a list.

So we ensure that the falue is a string by default now and
treat it as string in the code.

Change-Id: Ida72c712153dcda4cd06e0959f98ade4fee8dfbd
This commit is contained in:
Dmitriy Rabotyagov 2023-07-19 14:18:53 +02:00 committed by Dmitriy Rabotyagov
parent 1ebeab7d26
commit 5d635c469a
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
defaults/main.yml
View File

@ -208,7 +208,7 @@ horizon_enable_ha_router: False
horizon_default_dns_nameservers: []
# Provide list of network types that are available for creation
horizon_network_provider_types: "{{ neutron_ml2_drivers_type | default(['geneve', 'vlan', 'flat']) }}"
horizon_network_provider_types: "{{ neutron_ml2_drivers_type | default('geneve,vlan,flat') }}"
# DISALLOW_IFRAME_EMBED can be used to prevent Horizon from being embedded
# within an iframe. Legacy browsers are still vulnerable to a Cross-Frame

2
templates/horizon_local_settings.py.j2
View File

@ -340,7 +340,7 @@ OPENSTACK_NEUTRON_NETWORK = {
# Set which provider network types are supported. Only the network types
# in this list will be available to choose from when creating a network.
# Network types include local, flat, vlan, gre, and vxlan.
'supported_provider_types': {{ horizon_network_provider_types | to_json }},
'supported_provider_types': {{ horizon_network_provider_types.split(',') | to_json }},
# Set which VNIC types are supported for port binding. Only the VNIC
# types in this list will be available to choose from when creating a