-
Notifications
You must be signed in to change notification settings - Fork 347
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
T5595 : PIM multicast bfd feature #2411
Conversation
<help>BFD support</help> | ||
<completionHelp> | ||
<list>enable disable</list> | ||
</completionHelp> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we have bfd does it mean we want to enable it? Let’s avoid enable/disable
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please see BGP neighbors, there is a simple bfd node that is valueless
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All nodes with values like true/false, enable/disable that still remain in the system are legacy and should be converted to valueless as soon as it's feasible. The policy not to make any new ones has been in place since Vyatta times, in fact. ;)
<help>BFD support</help> | ||
<completionHelp> | ||
<list>enable disable</list> | ||
</completionHelp> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please see BGP neighbors, there is a simple bfd node that is valueless
Will be implemented together with https://vyos.dev/T5733 |
add bfd for fast-convergence in multicast , also , I've created the smoke- test that were missing.
Change Summary
Types of changes
Related Task(s)
Related PR(s)
Component(s) name
Proposed changes
How to test
vyos-cli :
frr :
Smoketest result
Checklist: