Skip to content

Commit

Permalink
vxlan-controller: support dualstack (#1057)
Browse files Browse the repository at this point in the history
In Azure we need to run Dualstack AKS at some point because that is the way to provision a dualstack NLB for AKS. There is no "bring your NLB" for AKS right now.

This skips ipv6 addresses during initialization, because autoscaler does not need ipv6, and the following lines do not work with ipv6 resulting in an error.

neondatabase/cloud#13924
  • Loading branch information
chaporgin authored Oct 7, 2024
1 parent ecdb9ed commit 72267ba
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions neonvm/tools/vxlan/controller/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -203,6 +203,11 @@ func updateFDB(vxlanName string, nodeIPs []string, ownIP string) error {

for _, ip := range nodeIPs {
if ip != ownIP {
if net.ParseIP(ip).To4() == nil {
log.Printf("not adding IPv6 addr %q to FDB broadcast entry, no support for it", ip)
continue
}

broadcastFdbEntry := netlink.Neigh{
LinkIndex: link.Attrs().Index,
Family: syscall.AF_BRIDGE,
Expand Down

0 comments on commit 72267ba

Please sign in to comment.