Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[yang]Add FG-ECMP yang model #14003

Merged
merged 16 commits into from
Mar 2, 2023
Merged

[yang]Add FG-ECMP yang model #14003

merged 16 commits into from
Mar 2, 2023

Conversation

lmingz
Copy link
Contributor

@lmingz lmingz commented Feb 27, 2023

Why I did it
Add yang model definition for FG_NHG_MEMBER, FG_NHG and FG_NHG_PREFIX from request: #12457

How I did it
Add yang model refereeing to description https://github.com/sonic-net/SONiC/blob/master/doc/ecmp/fine_grained_next_hop_hld.md

How to verify it
pyang tests

Which release branch to backport (provide reason below if selected)
201811
201911
202006
202012
202106
202111
202205
[x ] 202211
Description for the changelog
Add yang model for FG_NHG_MEMBER, FG_NHG and FG_NHG_PREFIX

Link to config_db schema for YANG module changes

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

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Feb 27, 2023 •
edited
Loading

CLA Signed

The committers listed above are authorized under a signed CLA.

@lmingz lmingz marked this pull request as ready for review February 28, 2023 17:37
@lmingz lmingz requested a review from qiluo-msft as a code owner February 28, 2023 17:37
@lmingz lmingz marked this pull request as draft February 28, 2023 17:40
@lmingz lmingz marked this pull request as ready for review February 28, 2023 17:40
@lmingz
Copy link
Contributor Author

lmingz commented Mar 1, 2023

Hi @ganglyu and @anish-n, please review this pr.

@ganglyu ganglyu self-requested a review March 1, 2023 12:05
@lmingz lmingz changed the title add fine-grained-ecmp model [yang]Add FG-ECMP yang model Mar 2, 2023
@lmingz
Copy link
Contributor Author

lmingz commented Mar 2, 2023

@zhangyanzhao please review this new pr about FG_ECMP yang model.

@qiluo-msft qiluo-msft added the YANG YANG model related changes label Mar 2, 2023
@zhangyanzhao
Copy link
Collaborator

@lguohan @yxieca can you please help to merge this PR? Thanks.

@qiluo-msft qiluo-msft merged commit eca5599 into sonic-net:master Mar 2, 2023
xumia pushed a commit to xumia/sonic-buildimage-1 that referenced this pull request Mar 10, 2023
@lmingz @xumia
[yang]Add FG-ECMP yang model (sonic-net#14003)
52bcde0 
Why I did it
Add yang model definition for FG_NHG_MEMBER, FG_NHG and FG_NHG_PREFIX from request: sonic-net#12457

How I did it
Add yang model refereeing to description https://github.com/sonic-net/SONiC/blob/master/doc/ecmp/fine_grained_next_hop_hld.md

How to verify it
pyang tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ganglyu ganglyu ganglyu approved these changes

@anish-n anish-n anish-n approved these changes

@qiluo-msft qiluo-msft qiluo-msft approved these changes

Assignees
No one assigned
Labels
YANG YANG model related changes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@lmingz @zhangyanzhao @qiluo-msft @anish-n @ganglyu @MingzhiLeng