openstack/openstack-ansible-os_neutron
Code Issues Proposed changes

Check length of network_mappings

Browse Source 
The OVS bridge creation logic for OVN deployments may fail
when the provider bridge has not been defined. This patch uses
logic that exists in the OVS deployment scenario to check the
length of neutron_provider_networks.network_mappings to ensure
a value has been set before attempting to create the bridge.

Change-Id: I34256e4ad22169ae6907a3c40270cb714cf33466
This commit is contained in:
James Denton 2023-09-06 08:53:08 -05:00
parent 4abf3e9383
commit 7c0ac931b0
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
tasks/providers/setup_ovs_ovn.yml
View File

@ -71,7 +71,7 @@
loop_var: bridge_mapping
when:
- neutron_services['neutron-ovn-controller']['group'] in group_names
- neutron_provider_networks.network_mappings is defined
- neutron_provider_networks.network_mappings is defined and (neutron_provider_networks.network_mappings|length > 0)
- name: Add ports to Network Provider Bridges
openvswitch_port:
@ -89,7 +89,7 @@
command: "ovs-vsctl set open . external-ids:ovn-bridge-mappings={{ neutron_provider_networks.network_mappings }}"
changed_when: false
when:
- neutron_provider_networks.network_mappings is defined
- neutron_provider_networks.network_mappings is defined and (neutron_provider_networks.network_mappings|length > 0)
- neutron_services['neutron-ovn-controller']['group'] in group_names
- name: Including ovn_cluster_setup tasks