Skip to content

SONiC Yang for Warm Restart #7698

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.

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

Merged
merged 11 commits into from
Sep 28, 2021
Merged

Conversation

ArthiSivanantham
Copy link
Contributor

Signed-off-by: Arthi Sivanantham [email protected]

Why I did it

SONiC YANG model support for warm restart.

How I did it

Defined warm restart YANG containers and lists based on config-DB schema.

How to verify it

Successful build of the following packages:
make target/python-wheels/sonic_yang_models-1.0-py3-none-any.whl
make target/python-wheels/sonic_yang_mgmt-1.0-py3-none-any.whl

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

  • 201811
  • 201911
  • 202006
  • 202012

Description for the changelog

SONiC YANG model support for warm restart.

Signed-off-by: Arthi Sivanantham <[email protected]>
@ArthiSivanantham ArthiSivanantham requested a review from lguohan as a code owner May 25, 2021 01:45
Signed-off-by: Arthi Sivanantham <[email protected]>
Signed-off-by: Arthi Sivanantham <[email protected]>
@ArthiSivanantham
Copy link
Contributor Author

@lguohan - Could you please add reviewers for this PR ?

@ArthiSivanantham
Copy link
Contributor Author

@anshuv-mfst - Can you please assign "labels" and "project" for this PR ?

@ArthiSivanantham
Copy link
Contributor Author

@lguohan - Could you please add reviewers for this PR ?

@anshuv-mfst anshuv-mfst added the YANG YANG model related changes label Aug 22, 2021
@anshuv-mfst
Copy link

Sure, done @ArthiSivanantham .

@venkatmahalingam
Copy link
Collaborator

@ArthiSivanantham Please resolve the conflict.

@ArthiSivanantham
Copy link
Contributor Author

@ArthiSivanantham Please resolve the conflict.

Done

@rathnasabapathyv
Copy link
Collaborator

@zhangyanzhao & @lguohan : Conflicts are resolved in this PR & we have enough approvals. Once build is completed, we can merge this PR. Thanks.

@lguohan
Copy link
Collaborator

lguohan commented Sep 2, 2021

@yxieca , can you review?

@zhangyanzhao
Copy link

@yxieca please help to review.

@zhangyanzhao zhangyanzhao requested a review from yxieca September 2, 2021 17:09
@venkatmahalingam
Copy link
Collaborator

@yxieca Please look at the reply for your comment, if no more comments, please merge this PR.

@zhangyanzhao
Copy link

@venkatmahalingam will handle Ying's comment today.

Signed-off-by: Arthi Sivanantham <[email protected]>
@ArthiSivanantham
Copy link
Contributor Author

@yxieca - Review comments have been addressed. Can you please review ?

@zhangyanzhao
Copy link

@venkatmahalingam will fix the conflict by this week, then we can merge.

@yxieca
Copy link
Contributor

yxieca commented Sep 27, 2021

/anp run

@yxieca yxieca removed the request for review from yingxie2021 September 27, 2021 16:02
@venkatmahalingam
Copy link
Collaborator

@yxieca Can we please merge this PR?

@yxieca yxieca merged commit ada8043 into sonic-net:master Sep 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
YANG YANG model related changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants