-
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
[DASH] Update dash headers to most recent #1832
base: v1.12
Are you sure you want to change the base?
Conversation
@marian-pritsak, @Pterosaur, @chrispsommers please review |
Please reference the exact commit ID of DASH which these headers were generated from, |
commit generated sonic-net/DASH@b6d44c7 |
/azp run |
Azure Pipelines could not run because the pipeline triggers exclude this branch/path. |
* @type sai_uint32_t | ||
* @flags MANDATORY_ON_CREATE | CREATE_ONLY | ||
*/ | ||
SAI_DASH_ACL_RULE_ATTR_DST_TAG, |
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.
ACL tags should be removed from the SAI API. Recently DASH WG decided to remove ACL tags from the SAI requirements
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.
i don't follow exactly what rules are for acl, i just generated headers from DASH repo and just added here
Updates DASH experimental headers to headers generated from https://github.com/sonic-net/DASH repo on origin/main