-
Notifications
You must be signed in to change notification settings - Fork 477
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
Add DF Mode, nexthopgroup id and splithorizon list assignment #1965
base: master
Are you sure you want to change the base?
Conversation
before making this PR for review please fix all the errors and sign commits |
{ | ||
SAI_VLAN_DF_MODE_IGNORE, | ||
|
||
SAI_VLAN_DF_MODE_BLOCKED, |
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.
can you plesae add more details on what exactly these modes are..
/** | ||
* @brief Split Horizon List ID | ||
* | ||
* Valid only for Ethernet Segment Ports - EVPN Multihoming feature |
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.
what is the PORT attribute to define whether the port is Ethernet segment PORT or not...?
* @default SAI_NULL_OBJECT_ID | ||
* @objects SAI_OBJECT_TYPE_NEXT_HOP_GROUP | ||
*/ | ||
SAI_FDB_ENTRY_ATTR_NEXT_HOP_GROUP_ID, |
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.
Does it need any changes in nexthopgroup types..
Creating a Draft PR for HLD review of EVPN Multihoming feature