Skip to content

Conversation

@sougatahitcs
Copy link

This request includes two BFD-related commits that I would like to contribute upstream to FRR from Cumulus.

Log:
root@r1:/tmp/topotests/bgp_bfd_session.test_bgp_bfd_session/r1# vtysh -c "show run"
Building configuration...

Current configuration:
!
frr version 10.6-dev-MyOwnFRRVersion
frr defaults traditional
hostname r1
domainname localdomain
log commands
service integrated-vtysh-config
!
ip route 20.0.0.0/24 10.0.0.2 bfd profile pf1
!
interface r1-eth0
ip address 10.0.0.1/24
exit
!
router bgp 65000
neighbor PG peer-group
neighbor PG remote-as auto
neighbor PG bfd
neighbor PG bfd profile pf1
neighbor PG ebgp-multihop 15
neighbor PG update-source 10.0.0.1
neighbor PG2 peer-group
neighbor PG2 remote-as auto
neighbor PG2 bfd
neighbor PG2 bfd profile pf1
neighbor PG2 ebgp-multihop 25
neighbor 192.168.1.4 peer-group PG
neighbor 192.168.1.5 peer-group PG2
neighbor 192.168.1.5 update-source 10.0.0.1
neighbor 192.168.1.2 remote-as auto
neighbor 192.168.1.2 bfd
neighbor 192.168.1.2 bfd profile pf1
neighbor 192.168.1.2 ebgp-multihop 10
neighbor 192.168.1.2 update-source 10.0.0.1
neighbor 192.168.1.3 remote-as auto
neighbor 192.168.1.3 bfd
neighbor 192.168.1.3 bfd profile pf1
neighbor 192.168.1.3 ebgp-multihop 20
neighbor 192.168.1.3 update-source r1-eth0
exit
!
bfd
profile pf1
exit
!
exit
!
end
root@r1:/tmp/topotests/bgp_bfd_session.test_bgp_bfd_session/r1# vtysh -c "show bfd peers" | grep "profile|Profile"
Profile: pf1
Profile: pf1
Profile: pf1
Profile: pf1
Profile: pf1
root@r1:/tmp/topotests/bgp_bfd_session.test_bgp_bfd_session/r1# vtysh -c "show bfd peers json" | grep "profile|Profile"
"profile":"pf1",
"profile":"pf1",
"profile":"pf1",
"profile":"pf1",
"profile":"pf1",
root@r1:/tmp/topotests/bgp_bfd_session.test_bgp_bfd_session/r1# vtysh -c "show bfd static route"
Showing BFD monitored static routes:

Next hops:
VRF default IPv4 Unicast:
20.0.0.0/24 peer 10.0.0.2 (status: uninstalled)

VRF default IPv4 Multicast:

VRF default IPv6 Unicast:

root@r1:/tmp/topotests/bgp_bfd_session.test_bgp_bfd_session/r1#

@frrbot frrbot bot added the bfd label Dec 24, 2025
@sougatahitcs sougatahitcs force-pushed the sougatab/_cl-frr-to-upstream-frr-bfd-commits branch from cb4fed1 to d2b669f Compare December 24, 2025 10:15
@github-actions github-actions bot added size/S and removed size/M labels Dec 24, 2025
…config for single hop static route bfd

 root@mid0:mgmt:/media/node/sougatab/debs/frr-upgrade# vtysh -c "conf t" -c "ip route 4.4.4.4/32 34.0.0.2 bfd profile pf1 source 34.0.0.1 vrf vrf1"
 root@mid0:mgmt:/media/node/sougatab/debs/frr-upgrade# vtysh -c "show run" | grep 4.4.4.4
 ip route 4.4.4.4/32 34.0.0.2 bfd profile pf1

Ticket: # 4402283

Signed-off-by: Sougata Barik <[email protected]>
@sougatahitcs sougatahitcs force-pushed the sougatab/_cl-frr-to-upstream-frr-bfd-commits branch from d2b669f to 00c5bdd Compare December 24, 2025 10:29
@sougatahitcs
Copy link
Author

ci:rerun

1 similar comment
@sougatahitcs
Copy link
Author

ci:rerun

@github-actions github-actions bot added size/M and removed size/S labels Dec 25, 2025
BFD profile information is essential, particularly in scenarios where multiple BFD clients use different profiles for the same session. In such cases, BFD ultimately applies the most recently updated profile. Currently, there is no way to determine which profile is configured and which one is actively in use.

Ticket: #4653777

Signed-off-by: Sougata Barik <[email protected]>
@sougatahitcs sougatahitcs force-pushed the sougatab/_cl-frr-to-upstream-frr-bfd-commits branch from 3a1e59b to dc8b2d6 Compare December 25, 2025 20:05
@sougatahitcs
Copy link
Author

ci:rerun

{
char addr_buf[INET6_ADDRSTRLEN];

if (bs->profile_name)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doesn't this change break the table format?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants