Skip to content

[sonic-yang-models]: Add TC_ACTION to sonic-acl#28193

Open
AnantKishorSharma wants to merge 1 commit into
sonic-net:masterfrom
AnantKishorSharma:acl-set-tc-yang
Open

[sonic-yang-models]: Add TC_ACTION to sonic-acl#28193
AnantKishorSharma wants to merge 1 commit into
sonic-net:masterfrom
AnantKishorSharma:acl-set-tc-yang

Conversation

@AnantKishorSharma

@AnantKishorSharma AnantKishorSharma commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Why I did it

Model the new TC_ACTION ACL rule action (set traffic class) in the ACL YANG model so CONFIG_DB validation accepts it. Companion to the orchagent change that maps TC_ACTION to SAI_ACL_ENTRY_ATTR_ACTION_SET_TC.

How I did it

  • Add leaf TC_ACTION { type uint8; } to sonic-acl.yang.j2.
  • Add positive (0/7/255) and negative (out-of-range 256) YANG model tests.

How to verify it

  • pytest on sonic-yang-models (yang_model_tests) passes; a valid TC_ACTION renders and an out-of-range value is rejected (Bounds).

Related PRs:

Signed-off-by: Anant Kishor Sharma <anant.kishor-sharma@hpe.com>
@mssonicbld

Copy link
Copy Markdown
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants