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.

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

[yang] Update device_metadata to add dhcp_server #10382

Merged
merged 2 commits into from
Apr 1, 2022

Conversation

ganglyu
Copy link
Contributor

@ganglyu ganglyu commented Mar 29, 2022

Signed-off-by: Gang Lv ganglv@microsoft.com

Why I did it

dhcp_server is introduced, and need to update yang model.

How I did it

Update yang models and add unit test.

How to verify it

Run unit test for sonic-yang-models.

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111

Description for the changelog

Fix #10365

Link to config_db schema for YANG module changes

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

Signed-off-by: Gang Lv ganglv@microsoft.com
@ganglyu ganglyu added the YANG YANG model related changes label Mar 29, 2022
@ganglyu ganglyu requested a review from qiluo-msft as a code owner March 29, 2022 06:07
@ganglyu
Copy link
Contributor Author

ganglyu commented Mar 29, 2022

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ganglyu
Copy link
Contributor Author

ganglyu commented Mar 30, 2022

/azpw run Azure.sonic-buildimage

@mssonicbld
Copy link
Collaborator

/AzurePipelines run Azure.sonic-buildimage

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@ganglyu
Copy link
Contributor Author

ganglyu commented Mar 31, 2022

@praveen-li @rathnasabapathyv @venkatmahalingam @dgsudharsan
Would you please review this PR?

@ganglyu ganglyu merged commit a8f72e5 into sonic-net:master Apr 1, 2022
@ganglyu ganglyu added the Request for 202111 Branch For PRs being requested for 202111 branch label Apr 14, 2022
@judyjoseph
Copy link
Contributor

There is merge conflict in 202111, please raise a separate PR

@judyjoseph judyjoseph removed the Request for 202111 Branch For PRs being requested for 202111 branch label Apr 18, 2022
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.

[yang] device metadata validation failed
5 participants