-
Notifications
You must be signed in to change notification settings - Fork 626
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
FRR - ADD BFD feature on web interface #2050
Comments
Did you check If bfd is compatible on the FreeBSD Version of FRR? Most of it only works with linux |
Hi, thanks for your answer. I can activate BFD via the cli |
Can you leave a füll sample Here so I can have a look |
In the file frr_daemons="zebra ospfd bfdd bgpd" And My frr cli configuration looks like:
|
And after this BFD really works? I mean, it eats the commands, but regarding FreeBSD and FRR this means nothing. |
Yes I can see BFD packets get out from my interface using tcpdump. |
I had a look at the docs, they are using peer syntax in bfdd.conf, is there a reason why you not configuring it? I will try to add a bfd submenu |
Firstly I can't use it because the BFD configuration that I use is in the file /usr/local/etc/frr/ospfd.conf generated by opnsense and Secondly the file /etc/rc.conf.d/frr where we can specify to frr to start BFD daemon is also generated by opnsense. Thanks for your answers. |
They only get overwritten when you hit save in the UI. You can setup everything from UI, then switch to CLI, edit rc.conf.d file and also add a bfdd.conf in etc folder and restart via |
But it'll be overwritten after a reboot and each time after a save in the UI. |
Indeed, the only thing I need from you is an "OK", I've tested it with just config files and CLI, it runs fine, peers are detected. (a show run and show bfd XXX would be nice) I can't count the times I added a feature which in the end wasn't supported the way, the issuer thought it should work, dont take it as an offence. |
Oh no worries, I'm grateful that feature could be added. It's all fine to me if the configuration I pasted before can be configured in UI or at least not reset when I save in the UI |
BFD would be great ... it should also speed up the convergence if used with the "Enable CARP Failover" option |
When you follow the thread you'll see that this needs more testers before to get integrated. |
Oh I see, I misunderstood you previous post. IMHO it works fine; I've added bfdd as daemon and
|
Can you do a write in vtysh and post the contents of bfdd.conf, please? |
if I add the BFD only under the interface the file is:
if I add it in the main configuration
|
The vtysh commands was 1st case:
2nd case:
PS: In my operational tests I used only the interface config as with the vyos I use just this one |
@fprina ok thx, and can you also check this: #2050 (comment) |
vtysh
bfdd.conf
After a quick test it have the same behaviour of
|
What do you mean by this needs more testers before to get integrated ? Should I show you more tests results and configurations ? |
Not anymore, initially I needed a confirmation of #2050 (comment) cause it changes they way to introduce it. The first test would be a checkbox in OSPF, the second one is a new submenu where you configure peers outside of the protocol (which is more generic) |
Thanks for quick response. Since it is the same behavior, the second seems to me indeed more suitable. |
hai @mimugmail |
I am just setting up an opnsense router in a BGP environement and want to use BFD as well. The other routers in my test lab use BFD, so I am happy to help testing this. |
Sorry for the delay guys, I added a PR for BFD |
@MarkStitson @fprina @LensHunnel can anyone of you guys test the new devel package and check if it works for you? |
Which is the right command to use to patch the code ? |
Being new to OpnSense, it took me a few attempts to get the update, but basically I switched to Development in System->Firmware->Settings->Type, then ran an update rebooted and ran an update again. It seems to do the right thing for me and bfd is running: Hello, this is FRRouting (version 7.4). OPNsense.local.xxx.com# show bfd peers brief OPNsense.local.xxx.com# A useful addition would be to add BFD to the diagnostics under Routing->Diagnostics->BFD with 2 tabs like BGP:
Should I raise a separate feature request for that? |
Oops. I spoke too soon. I can't enabled BFD in Routing->BGP-Neighbors when adding a peer to create the line: router bgp 65509 |
@MarkStitson Do you really need this line in bgp when you already have a peer in BFD. By the way you don't need to switch to Development branch, you can also just install via CLI: |
Thanks for the hint how to install the package. With regard to if the line is required in the BGP section, the BGP daemon needs to be told which peer uses BFD and which peer does not use BFD, the FRR docs also say that http://docs.frrouting.org/en/latest/bfd.html BFD is a helper for the actual routing protocol, alone it doesn't do anything useful. It needs to be linked to an actual routing protocol and tells the routing protocol if a peer with BGP (or interface with OSPF) is up or down. If I configure with the extra line, then the vtysh BFD: Type: single hop If I don't have the extra line the BFD status is missing from BGP, to enable it we need a tickbox in the BGP Neigbour config. I believe you have to do something similar for OSPF, but there it is per interface, not per peer, see above |
So this means you only need a checkpoint in neighbor grid to enable/disable BFD correct? |
Correct, there has to be a checkpoint in the BGP Neighbour and the OSPF Interface so BFD can be enabled or disabled for each neighbout/interface. There should probably also be a checkbox in the BFD Neighbour settings, so you can enable multihop per neighbour, but I don't use that and I am not sure how to test it. This would generate |
I try to add it tomorrow |
That's great. I would offer to help more, but I'm not that familiar with the OpenSense system yet. |
@MarkStitson no worries .. these were the required changes for adding the checkbox: |
I've now checked the change and it works as expected. I've also created the missing diagnostics and made a pull request for that: #2399 I've got the code for the OSPF setup for BFD here as well, but I need someone who can test this for me. |
Thx 😍 |
Hi @MarkStitson, Sorry to reopen such an old issue, but I'm in a situation where I want to use BFD over OSPF. Would you still be willing to contribute the code if I run the tests for you? |
As far as I recall I added the code for BGP and OSPF. I just couldn't test it.
The thing that I never finished was the diagnostics.
Happy to get back onto that and test with you.
I won't have time to look at it until next week though.
Mark
Sent from Outlook for Android<https://aka.ms/AAb9ysg>
…________________________________
From: Jeremy Berkleef ***@***.***>
Sent: Tuesday, June 20, 2023 7:42:43 PM
To: opnsense/plugins ***@***.***>
Cc: Mark Stitson ***@***.***>; Mention ***@***.***>
Subject: Re: [opnsense/plugins] FRR - ADD BFD feature on web interface (#2050)
Hi @MarkStitson<https://github.com/MarkStitson>,
Sorry to reopen such an old issue, but I'm in a situation where I want to use BFD over OSPF. Would you still be willing to contribute the code if I run the tests for you?
—
Reply to this email directly, view it on GitHub<#2050 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AGFP3ILJJOVGEQ7CBSIVOWDXMHVKHANCNFSM4SA5JZ7Q>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
Hi Mark, Thanks for getting back to this. As far as I can tell, there's no UI to enable BFD for my OSPF interfaces. |
If you are confident in trying it out on your box, here is my branch commit for BFD on OSPF: |
Is your feature request related to a problem? Please describe.
always frustrated when I configure the routing on Opnsense because I'm using bfd on my OSPF configuration and I can't start the bfdd process because on the template /usr/local/opnsense/service/templates/OPNsense/Quagga/frr it only configure ospf rip and BGP and not BFD.
Describe the solution you'd like
I would like two checkboxes fields:
the first one to activate bfd globally and
the second one in the
ui/quagga/ospf/
, in the interface configuration view, which would addip ospf bfd
in frr configuration under interface.The text was updated successfully, but these errors were encountered: