Skip to content
This repository has been archived by the owner on Jun 25, 2024. It is now read-only.

Commit

Permalink
Merge pull request #670 from Jaganathancse/network_naming_issues
Browse files Browse the repository at this point in the history
Getting network name missing in dictionary issue
  • Loading branch information
openshift-merge-bot[bot] authored Feb 16, 2024
2 parents ff1a79d + f60c981 commit 22de1fe
Showing 1 changed file with 4 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ spec:
mtu: {{ min_viable_mtu }}
# force the MAC address of the bridge to this interface
primary: true
{% for network in role_networks if network not in ["External", "Tenant"] %}
{% for network in role_networks if network not in ["external", "tenant"] %}
- type: vlan
mtu: {{ lookup('vars', networks_lower[network] ~ '_mtu') }}
vlan_id: {{ lookup('vars', networks_lower[network] ~ '_vlan_id') }}
Expand All @@ -85,11 +85,11 @@ spec:
- type: ovs_user_bridge
name: br-link1
use_dhcp: false
ovs_extra: "set port br-link1 tag={{ lookup('vars', networks_lower['Tenant'] ~ '_vlan_id') }}"
ovs_extra: "set port br-link1 tag={{ lookup('vars', networks_lower['tenant'] ~ '_vlan_id') }}"
addresses:
- ip_netmask: {{ lookup('vars', networks_lower['Tenant'] ~ '_ip') }}/{{ lookup('vars', networks_lower['Tenant'] ~ '_cidr') }}
- ip_netmask: {{ lookup('vars', networks_lower['tenant'] ~ '_ip') }}/{{ lookup('vars', networks_lower['tenant'] ~ '_cidr') }}
mtu: {{ lookup('vars', networks_lower['Tenant'] ~ '_mtu') }}
mtu: {{ lookup('vars', networks_lower['tenant'] ~ '_mtu') }}
members:
- type: ovs_dpdk_port
name: dpdk1
Expand Down

0 comments on commit 22de1fe

Please sign in to comment.