Skip to content

[hash] Add ECMP/LAG Hash Algorithm YANG model#17079

Merged
liat-grozovik merged 2 commits intosonic-net:masterfrom
nazariig:master-hash-algorithm
Nov 20, 2023
Merged

[hash] Add ECMP/LAG Hash Algorithm YANG model#17079
liat-grozovik merged 2 commits intosonic-net:masterfrom
nazariig:master-hash-algorithm

Conversation

@nazariig
Copy link
Copy Markdown
Collaborator

@nazariig nazariig commented Nov 2, 2023 •
edited by liat-grozovik
Loading

HLD: sonic-net/SONiC#1501

Why I did it

  • Added YANG model as part of Generic Hash feature development
Work item tracking
  • N/A

How I did it

  • Added YANG model

How to verify it

  1. Add UT
  2. Verified manually with the feature qualification

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

Description for the changelog

  • Added hashing algorithm configuration

Link to config_db schema for YANG module changes

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

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

@nazariig
[hash]: Add ECMP/LAG Hash Algorithm YANG model.
2868c0b 
Signed-off-by: Nazarii Hnydyn <nazariig@nvidia.com>
@nazariig nazariig requested a review from qiluo-msft as a code owner November 2, 2023 17:54
@nazariig
[hash]: Handle review comments.
2d4617d 
Signed-off-by: Nazarii Hnydyn <nazariig@nvidia.com>
@nazariig nazariig mentioned this pull request Nov 2, 2023
Merged
@liat-grozovik
Copy link
Copy Markdown
Collaborator

liat-grozovik commented Nov 19, 2023

@dgsudharsan can you please help to review this one as well? based on your approval and Stepan i will merge this PR

@dgsudharsan dgsudharsan added the YANG YANG model related changes label Nov 20, 2023
@liat-grozovik liat-grozovik merged commit c99ec1f into sonic-net:master Nov 20, 2023
@liat-grozovik liat-grozovik changed the title [hash]: Add ECMP/LAG Hash Algorithm YANG model [hash] Add ECMP/LAG Hash Algorithm YANG model Nov 20, 2023
@mssonicbld
Copy link
Copy Markdown
Collaborator

mssonicbld commented Jan 31, 2024

@nazariig PR conflicts with 202311 branch

@yxieca
Copy link
Copy Markdown
Contributor

yxieca commented Mar 21, 2024

@nazariig can you help create PR for 202311 branch?

@nazariig
Copy link
Copy Markdown
Collaborator Author

nazariig commented Mar 22, 2024

@yxieca already part of 202311

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

Reviewers

@dgsudharsan dgsudharsan dgsudharsan approved these changes

@stepanblyschak stepanblyschak stepanblyschak approved these changes

@liat-grozovik liat-grozovik liat-grozovik approved these changes

@qiluo-msft qiluo-msft Awaiting requested review from qiluo-msft qiluo-msft is a code owner

@volodymyrsamotiy volodymyrsamotiy Awaiting requested review from volodymyrsamotiy

@bingwang-ms bingwang-ms Awaiting requested review from bingwang-ms

@prsunny prsunny Awaiting requested review from prsunny

Assignees

No one assigned

Labels

Approved for 202311 Branch Included in 202311 Branch Request for 202311 Branch YANG YANG model related changes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@nazariig @liat-grozovik @mssonicbld @yxieca @dgsudharsan @stepanblyschak