Skip to content

[trim]: Add Packet Trimming YANG model#22248

Merged
qiluo-msft merged 1 commit intosonic-net:masterfrom
nazariig:master-trim
May 2, 2025
Merged

[trim]: Add Packet Trimming YANG model#22248
qiluo-msft merged 1 commit intosonic-net:masterfrom
nazariig:master-trim

Conversation

@nazariig
Copy link
Collaborator

@nazariig nazariig commented Apr 6, 2025

Signed-off-by: Nazarii Hnydyn [email protected]

HLD: sonic-net/SONiC#1898

Why I did it

  • Added YANG model as part of Packet Trimming feature development

Work item tracking

  • N/A

How I did it

  • Added YANG model

How to verify it

  1. Run Packet Trimming UTs

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211
  • 202305

Tested branch (Please provide the tested image version)

  • master
  • 202412

Description for the changelog

  • N/A

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

      .---.        .-----------
     /     \  __  /    ------
    / /     \(  )/    -----
   //////   ' \/ `   ---
  //// / // :    : ---
 // /   /  /`    '--
//          //..\\
       ====UU====UU====
           '//||\\`
             ''``

@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Contributor

@wen587 wen587 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@liat-grozovik
Copy link
Collaborator

@nazariig can you please resolve conflict so i can then merge it?

qiluo-msft
qiluo-msft previously approved these changes Apr 29, 2025
@mssonicbld
Copy link
Collaborator

/azp run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@nazariig
Copy link
Collaborator Author

nazariig commented May 2, 2025

@nazariig can you please resolve conflict so i can then merge it?

@liat-grozovik done

@nazariig
Copy link
Collaborator Author

nazariig commented May 2, 2025

@qiluo-msft can you please help to merge?

@qiluo-msft qiluo-msft merged commit 720824e into sonic-net:master May 2, 2025
19 checks passed
@yejianquan
Copy link
Contributor

Hi @nazariig please fix the conflict and create PR to 202505

@yejianquan
Copy link
Contributor

202505 branched out around 5/16, this commit is included when created, removing request labels.
and please ignore the conflict notification

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.

9 participants