openstack/openstack-helm-infra
Code Issues Proposed changes

Ingress: add keepalived-router-id for keeplived vip

Browse Source 
When there are multiple keepalived instances in same network space,
equal keepalived router-ids cause conflict (now default router-id number
is 100). So we have to specify keepalived's router_id for VRRP peering.
This commit make keepalived route-id configurable, so that we can
prevent keepalived conflict caused by default keepalived router-id.

Change-Id: Ia92a8b64205ab52ad15237e9fdeaacb61aae6400
This commit is contained in:
SIRI KIM 2019-06-05 05:14:07 +00:00 committed by Pete Birley
parent 843db92744
commit 9022c9237d
2 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ingress/templates/deployment-ingress.yaml
View File

@ -338,6 +338,8 @@ spec:
value: {{ ( .Values.network.vip.addr | split "/" )._0 | quote }} value: {{ ( .Values.network.vip.addr | split "/" )._0 | quote }}
- name: KEEPALIVED_UNICAST_PEERS - name: KEEPALIVED_UNICAST_PEERS
value: null value: null
- name: KEEPALIVED_ROUTER_ID
value: {{ .Values.network.vip.keepalived_router_id | quote }}
{{- end }} {{- end }}
{{- end }} {{- end }}
volumes: volumes:

1
ingress/values.yaml
View File

@ -140,6 +140,7 @@ network:
mode: routed mode: routed
interface: ingress-vip interface: ingress-vip
addr: 172.18.0.1/32 addr: 172.18.0.1/32
keepalived_router_id: 100
ingress: ingress:
annotations: annotations:
#NOTE(portdirect): if left blank this is populated from #NOTE(portdirect): if left blank this is populated from