openstack/tripleo-heat-templates
Code Issues Proposed changes

Use /32 netmask for VIPs

Browse Source 
Prior to commit c712355e4bae4ef2fc1b83e5603c0364dbd50a78
KeepaliveD created the VIP addresses. KeepaliveD created
the VIPs with /32 netmask, when moving the VIPs to the
DeployedServerPortMap and adding them to the br-ctlplane
interface the netmask of the ctlplane subnet was used
(typically /24). The result is a routing table that
potentially uses the incorrect device for traffic when
the public VIP is not on in the ctlplane subnet.

This change hard-codes the netmask for the VIP addresses
to /32.

blueprint replace-keepalived-undercloud
Closes-Bug: #1878101
Change-Id: I873e925d2250677f25b9ae51ed0b87bd1b8e6b32
This commit is contained in:
Harald Jensås 2020-05-14 08:33:16 +02:00
parent 0f7cca3e1b
commit 1ebf115f85
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
net-config-standalone.j2.yaml
View File

@ -135,7 +135,7 @@ resources:
list_join:
- /
- - {get_param: [DeployedServerPortMap, 'control_virtual_ip', fixed_ips, 0, ip_address]}
- {str_split: ['/', {get_param: [DeployedServerPortMap, 'control_virtual_ip', subnets, 0, cidr]}, 1]}
- '32'
-
if:
- public_virtual_ip_unset
@ -144,7 +144,7 @@ resources:
list_join:
- /
- - {get_param: [DeployedServerPortMap, 'public_virtual_ip', fixed_ips, 0, ip_address]}
- {str_split: ['/', {get_param: [DeployedServerPortMap, 'public_virtual_ip', subnets, 0, cidr]}, 1]}
- '32'
routes: {get_param: ControlPlaneStaticRoutes}
members:
- type: interface

4
net-config-undercloud.j2.yaml
View File

@ -130,7 +130,7 @@ resources:
list_join:
- /
- - {get_param: [DeployedServerPortMap, 'control_virtual_ip', fixed_ips, 0, ip_address]}
- {str_split: ['/', {get_param: [DeployedServerPortMap, 'control_virtual_ip', subnets, 0, cidr]}, 1]}
- '32'
-
if:
- public_virtual_ip_unset
@ -139,7 +139,7 @@ resources:
list_join:
- /
- - {get_param: [DeployedServerPortMap, 'public_virtual_ip', fixed_ips, 0, ip_address]}
- {str_split: ['/', {get_param: [DeployedServerPortMap, 'public_virtual_ip', subnets, 0, cidr]}, 1]}
- '32'
routes: {get_param: ControlPlaneStaticRoutes}
members:
- type: interface