-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
bfd session is not created in multihop bgp when update-source is interface name #17396
Comments
BFD session is started only once the BGP session is established. Or the main issue for you is that update-source does not work with interface name? |
Yes BFD session is created if BGP session is established. But as you see, when I use ip addr in update-source, BFD session can be created even bgp is down. |
Could you try with the newer versions? I can't replicate this with master branch. |
I have similar issue with master branch:
If you save the configuraion to frr.conf and restart frr, the BFD peers are empty. But it works by resetting bgp bfd via vtysh. |
Could you enable all "debug bgp bfd" stuff and show the logs? |
There is no detail info from the startup log
Using vtysh to del/add bfd again, got below log:
|
By the way, I found the commit 348ad00 that fix the bug report by @weisuofayubg , it is not in 10.0 and below yet. So only 10.1 has the fix |
@tufeigunchu could you create a separate issue for your case? It's a bit confusing since we talk about two issues in a single thread. |
OK |
Related to #17408. |
Description
I define an ebgp multihop neighbor, and enable bfd. But bfd session is not created if bgp session is not up.
Version
How to reproduce
define ebgp neighbor, enable multihop and bfd, set update-source with interface name instead of ip addr.
Expected behavior
I expect to see bfd session even it is Down.
Actual behavior
BFD session is empty
Additional context
When I use ip address in update-source, it works.
Why can't it resolve interface IP if I use interface name in update-source?
Checklist
The text was updated successfully, but these errors were encountered: