Skip to content

Commit

Permalink
[frr/bgpd] fix frr VXLAN EVPN configuration for unified config
Browse files Browse the repository at this point in the history
During debugging of VXLAN EVPN using unified vs split configuration, the
same BGP configuration is made with one exception, the setting of
```
no fpm use-next-hop-groups
```
Is forcibly set for all FRR instances.

This change was introduced in PR #12852 when switching to the new
fpm dataplane plugin.

When running `vtysh -c "config" -c "no fpm use-next-hop-groups"`
it can be seen the VTEP immediately comes online and traffic
flows as expected.

This adds the option as is present in dockers/docker-fpm-frr/frr/zebra/zebra.conf.j2
controlled by the DEVICE_METADATA nexthop_group option.

Signed-off-by: Brad House (@bradh352)
  • Loading branch information
bradh352 committed Dec 5, 2024
1 parent 8e0f1c6 commit 1b7d6ee
Showing 1 changed file with 9 additions and 0 deletions.
9 changes: 9 additions & 0 deletions src/sonic-frr-mgmt-framework/templates/frr/frr.conf.j2
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,15 @@
!
agentx
!
{% if ( ('localhost' in DEVICE_METADATA) and ('nexthop_group' in DEVICE_METADATA['localhost']) and
(DEVICE_METADATA['localhost']['nexthop_group'] == 'enabled') ) %}
! enable next hop group support
fpm use-next-hop-groups
{% else %}
! Uses the old known FPM behavior of including next hop information in the route (e.g. RTM_NEWROUTE) messages
no fpm use-next-hop-groups
{% endif %}
!
!Add fpm address for zebra
fpm address 127.0.0.1
Expand Down

0 comments on commit 1b7d6ee

Please sign in to comment.