-
Notifications
You must be signed in to change notification settings - Fork 478
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
Enhancements for MPLS support #1181
Conversation
Signed-off-by: Ann Pokora <[email protected]>
inc/saiswitch.h
Outdated
* @flags READ_ONLY | ||
*/ | ||
SAI_SWITCH_ATTR_AVAILABLE_MPLS_INSEG_ENTRY, | ||
|
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 consider using sai_object_type_get_availability()
This is generic object CRM API, replacing the old approach of specific attribute per object
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.
Is there an example of this that I can see? I don't see any usage of sai_object_type_get_availability() in latest CrmOrch.cpp.
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.
The PR has examples
ef89cb2#diff-1f169362ad7823ade5ade3918ba6a35fe17189e13d7f05a8b9edf2f6aeccc5f8
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.
AFAIK, Sonic hasn't yet ported existing CRM code to the new API, but is using the new API for new objects, such as debug counters
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.
ef89cb2#diff-1f169362ad7823ade5ade3918ba6a35fe17189e13d7f05a8b9edf2f6aeccc5f8
Shows usage examples
Signed-off-by: Ann Pokora <[email protected]>
* | ||
* @type bool | ||
* @flags CREATE_AND_SET | ||
* @default false |
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.
default it to true? just like IPv4 and IPv6?
This PR defines missing support for MPLS as compared to equivalent IPv4/IPv6 support: