Skip to content

[action] [PR:3078] [acl] Add IN_PORTS qualifier for L3 table (#3078)#3086

Merged
mssonicbld merged 1 commit intosonic-net:202305from
mssonicbld:cherry/202305/3078
Mar 22, 2024
Merged

[action] [PR:3078] [acl] Add IN_PORTS qualifier for L3 table (#3078)#3086
mssonicbld merged 1 commit intosonic-net:202305from
mssonicbld:cherry/202305/3078

Conversation

@mssonicbld
Copy link
Copy Markdown
Collaborator

  • Apply IN_PORTS qualifiier for L3 table

Why I did it
IN_PORTS qualifier was allowed for L3 table in 202012 release and below. Changes in #1982 removed that support leading to regression in some of our testcases. The following error was observed
ERR swss#orchagent: :- validateAclRuleMatch: Match SAI_ACL_ENTRY_ATTR_FIELD_IN_PORTS in rule RULE_1 is not supported by table DATAACL

* Apply IN_PORTS qualifiier for L3 table

Why I did it
IN_PORTS qualifier was allowed for L3 table in 202012 release and below. Changes in sonic-net#1982 removed that support leading to regression in some of our testcases. The following error was observed
ERR swss#orchagent: :- validateAclRuleMatch: Match SAI_ACL_ENTRY_ATTR_FIELD_IN_PORTS in rule RULE_1 is not supported by table DATAACL
@mssonicbld
Copy link
Copy Markdown
Collaborator Author

Original PR: #3078

@mssonicbld mssonicbld merged commit d33b854 into sonic-net:202305 Mar 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants