Don't assume every role has default_route_networks
This broken after new jinja release.
NOTE: This included nic config updates as well.
Co-Authored-By: Alex Schultz <aschultz@redhat.com>
Closes-Bug: #1928145
Change-Id: I9cbdcf289bae96f8ea7867e1cec365a9a07019ca
(cherry picked from commit 4af0c6ccac
)
This commit is contained in:
parent
59d1947baf
commit
1311f8a521
|
@ -59,7 +59,7 @@ parameters:
|
|||
guaranteed to pass through the data path of the segments in the
|
||||
{{network.name}} network.
|
||||
type: number
|
||||
{%- if network.name in role.default_route_networks %}
|
||||
{%- if role.default_route_networks is defined and network.name in role.default_route_networks %}
|
||||
{{network.name}}InterfaceDefaultRoute:
|
||||
default: ''
|
||||
description: default route for the {{network.name_lower}} network
|
||||
|
|
|
@ -57,7 +57,7 @@ parameters:
|
|||
guaranteed to pass through the data path of the segments in the
|
||||
{{network.name}} network.
|
||||
type: number
|
||||
{%- if network.name in role.default_route_networks %}
|
||||
{%- if role.default_route_networks is defined and network.name in role.default_route_networks %}
|
||||
{{network.name}}InterfaceDefaultRoute:
|
||||
default: ''
|
||||
description: default route for the {{network.name_lower}} network
|
||||
|
|
|
@ -59,7 +59,7 @@ parameters:
|
|||
guaranteed to pass through the data path of the segments in the
|
||||
{{network.name}} network.
|
||||
type: number
|
||||
{%- if network.name in role.default_route_networks %}
|
||||
{%- if role.default_route_networks is defined and network.name in role.default_route_networks %}
|
||||
{{network.name}}InterfaceDefaultRoute:
|
||||
default: ''
|
||||
description: default route for the {{network.name_lower}} network
|
||||
|
|
|
@ -59,7 +59,7 @@ parameters:
|
|||
guaranteed to pass through the data path of the segments in the
|
||||
{{network.name}} network.
|
||||
type: number
|
||||
{%- if network.name in role.default_route_networks %}
|
||||
{%- if role.default_route_networks is defined and network.name in role.default_route_networks %}
|
||||
{{network.name}}InterfaceDefaultRoute:
|
||||
default: ''
|
||||
description: default route for the {{network.name_lower}} network
|
||||
|
|
|
@ -57,7 +57,7 @@ parameters:
|
|||
guaranteed to pass through the data path of the segments in the
|
||||
{{network.name}} network.
|
||||
type: number
|
||||
{%- if network.name in role.default_route_networks %}
|
||||
{%- if role.default_route_networks is defined and network.name in role.default_route_networks %}
|
||||
{{network.name}}InterfaceDefaultRoute:
|
||||
default: ''
|
||||
description: default route for the {{network.name_lower}} network
|
||||
|
|
|
@ -57,7 +57,7 @@ parameters:
|
|||
guaranteed to pass through the data path of the segments in the
|
||||
{{network.name}} network.
|
||||
type: number
|
||||
{%- if network.name in role.default_route_networks %}
|
||||
{%- if role.default_route_networks is defined and network.name in role.default_route_networks %}
|
||||
{{network.name}}InterfaceDefaultRoute:
|
||||
default: ''
|
||||
description: default route for the {{network.name_lower}} network
|
||||
|
|
|
@ -1031,7 +1031,7 @@ resources:
|
|||
{{network.name_lower}}_host_routes:
|
||||
list_concat_unique:
|
||||
- {get_attr: [Networks, net_attributes_map, {{network.name_lower}}, subnets, {{_role_net_subnet}}, host_routes]}
|
||||
{%- if network.name in role.default_route_networks %}
|
||||
{%- if role.default_route_networks is defined and network.name in role.default_route_networks %}
|
||||
- - default: true
|
||||
next_hop: {get_attr: [Networks, net_attributes_map, {{network.name_lower}}, subnets, {{_role_net_subnet}}, gateway_ip]}
|
||||
{%- endif %}
|
||||
|
@ -1075,7 +1075,7 @@ resources:
|
|||
service_metadata_settings: {get_attr: [{{role.name}}ServiceChainRoleData, value, service_metadata_settings]}
|
||||
tripleo_network_config_template: {get_param: {{role.name}}NetworkConfigTemplate}
|
||||
tripleo_network_config_with_ansible: {get_param: NetworkConfigWithAnsible}
|
||||
default_route_networks: {{role.default_route_networks}}
|
||||
default_route_networks: {{role.default_route_networks|default(['ControlPlane'])}}
|
||||
networks_skip_config: {{ role.networks_skip_config|default([]) }}
|
||||
role_tags: {{role.tags}}
|
||||
|
||||
|
@ -1108,7 +1108,7 @@ resources:
|
|||
{{network.name}}InterfaceRoutes:
|
||||
list_concat_unique:
|
||||
- {get_attr: [Networks, net_attributes_map, {{network.name_lower}}, subnets, {{_role_net_subnet}}, host_routes]}
|
||||
{%- if network.name in role.default_route_networks %}
|
||||
{%- if role.default_route_networks is defined and network.name in role.default_route_networks %}
|
||||
- - default: true
|
||||
next_hop: {get_attr: [Networks, net_attributes_map, {{network.name_lower}}, subnets, {{_role_net_subnet}}, gateway_ip]}
|
||||
{%- endif %}
|
||||
|
@ -1120,7 +1120,7 @@ resources:
|
|||
not $.data.where($.startsWith('tripleo_vlan_id')).len() => 1,
|
||||
true => int($.data.where($.startsWith('tripleo_vlan_id')).first().split('=').last()))
|
||||
data: {get_attr: [Networks, net_attributes_map, {{network.name_lower}}, subnets, {{_role_net_subnet}}, tags]}
|
||||
{%- if network.name in role.default_route_networks %}
|
||||
{%- if role.default_route_networks is defined and network.name in role.default_route_networks %}
|
||||
{{network.name}}InterfaceDefaultRoute: {get_attr: [Networks, net_attributes_map, {{network.name_lower}}, subnets, {{_role_net_subnet}}, gateway_ip]}
|
||||
{%- endif %}
|
||||
{%- endfor %}
|
||||
|
|
Loading…
Reference in New Issue