Skip to content

doc/acl: document the PACKET_COLOR_ACTION ACL rule action#2429

Open
AnantKishorSharma wants to merge 1 commit into
sonic-net:masterfrom
AnantKishorSharma:hld-acl-packet-color-action
Open

doc/acl: document the PACKET_COLOR_ACTION ACL rule action#2429
AnantKishorSharma wants to merge 1 commit into
sonic-net:masterfrom
AnantKishorSharma:hld-acl-packet-color-action

Conversation

@AnantKishorSharma

Copy link
Copy Markdown

What I did

Document the PACKET_COLOR_ACTION ACL rule action in the ACL High Level Design, matching how the existing ACL rule actions (packet_action, mirror_action, inner_src_mac_rewrite_action) are documented.

PACKET_COLOR_ACTION (GREEN/YELLOW/RED) maps to SAI_ACL_ENTRY_ATTR_ACTION_SET_PACKET_COLOR in the generic CONFIG_DB AclOrch pipeline. It is capability-gated per ACL stage (ingress and/or egress) and is a no-op on platforms whose SAI does not advertise the action.

Details

  • Rev 1.1 -> 1.2 + revision-history row
  • Design paragraph: AclRulePacket, the GREEN/YELLOW/RED -> SAI_PACKET_COLOR_* mapping, and the per-stage capability gating (ACL_STAGE_CAPABILITY_TABLE)
  • Appendix keyword-table row for packet_color_action

Companion to the implementation in sonic-net/sonic-swss#4717 (YANG model in sonic-net/sonic-buildimage#28151).

Note: this overlaps with the in-flight #2423 (policer HLD) on the revision number and tables; whichever merges first, I'll rebase the other (bump to 1.3).

Document PACKET_COLOR_ACTION (GREEN/YELLOW/RED) in the ACL High Level Design:
it maps to SAI_ACL_ENTRY_ATTR_ACTION_SET_PACKET_COLOR in the generic CONFIG_DB
AclOrch pipeline, is capability-gated per ACL stage (ingress and/or egress),
and is a no-op on platforms whose SAI does not advertise the action.

Companion to sonic-net/sonic-swss#4717.

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

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
No pipelines are associated with this pull request.

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