search

sonic-net / DASH

Disaggregated APIs for SONiC Hosts
Apache License 2.0
77 stars 88 forks source link

Update dash_acl.p4 Change Optional to Ternary #590

Closed KrisNey-MSFT closed 1 week ago

KrisNey-MSFT commented 3 weeks ago

Change 'optional' to 'ternary' according to last Behavioral Model meeting. Issue 575 Change dash_acl

KrisNey-MSFT commented 3 weeks ago

This work is going to take some time and it also changes the attributes of the ACL table too (ternary match will add a priority attribute, which is not in design), so it will require a larger change to get it addressed. (Why are we not using 'priority' from the beginning)? Libraries to make this work w/bmv2 will need to be updated (template updates). We may also hit some test issues as well.
Kristina closing PR, but will keep Issue open.

KrisNey-MSFT commented 1 week ago

See comments above