Skip to content

Commit

Permalink
Changing ipv6 loopback addresses on the linecards to be on different …
Browse files Browse the repository at this point in the history
…subnets and adding t2 as a supported topology in veos (#3797)

In bgpd.main.conf.j2 in sonic-buildimage (http://github.com/Azure/sonic-buildimage/blob/master/dockers/docker-fpm-frr/frr/bgpd/bgpd.main.conf.j2#L77),
Ipv6 Loopback0 addresses are using a 64 bit mask. In t2 topology, we have 2 linecards, and we were assigning two different IPv6 addresses:

- fc00:10::1/128
- fc00:10::2/128

However, in bgpd.main.conf.j2 in sonic-buildimage (http://github.com/Azure/sonic-buildimage/blob/master/dockers/docker-fpm-frr/frr/bgpd/bgpd.main.conf.j2#L77),
ipv6 Loopback0 addresses are using a 64 bit mask. Thus, the route to the Loopback0 address of the remote linecard was being masked by the local Loopback0 address,
and not pointing to the inband port.

Fix for this is to use different subnets for each linecard. So, changing the Loopback0 addresses to:
- fc00:10::1/128
- fc00:11::1/128

Also, t2 topology was missing from veos as a supported topology
  • Loading branch information
sanmalho-git authored Jul 21, 2021
1 parent 3403b65 commit 1d66a2b
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
2 changes: 1 addition & 1 deletion ansible/vars/topo_t2-vs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ topology:
- 10.1.0.2/32
ipv6:
- FC00:10::1/128
- FC00:10::2/128
- FC00:11::1/128
vs_chassis:
inband_port:
- 30
Expand Down
4 changes: 2 additions & 2 deletions ansible/vars/topo_t2.yml
Original file line number Diff line number Diff line change
Expand Up @@ -326,8 +326,8 @@ topology:
- 10.1.0.3/32
ipv6:
- FC00:10::1/128
- FC00:10::2/128
- FC00:10::3/128
- FC00:11::1/128
- FC00:12::1/128

configuration_properties:
common:
Expand Down
1 change: 1 addition & 0 deletions ansible/veos
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ all:
- tgen-t0-3
- tgen-t1-3-lag
- mgmttor
- t2
children:
server_1:
server_2:
Expand Down

0 comments on commit 1d66a2b

Please sign in to comment.