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

[sonic-cfggen]: Update test minigraph to add deployment_id #9593

Merged
merged 2 commits into from
Dec 21, 2021

Conversation

ganglyu
Copy link
Contributor

@ganglyu ganglyu commented Dec 20, 2021

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

Why I did it

Config db schema generated from test minigraph can't pass yang validation.

How I did it

Update minigraph xml to add DeploymentId.

How to verify it

Build sonic-config-engine.
Run command 'sonic-cfggen -m xxx.xml --print-data', and check deployment_id field.

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

Description for the changelog

Fix #9550

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

Signed-off-by: Gang Lv ganglv@microsoft.com
Signed-off-by: Gang Lv ganglv@microsoft.com
@ganglyu ganglyu merged commit 67e40b5 into sonic-net:master Dec 21, 2021
@ganglyu ganglyu added the Request for 202111 Branch For PRs being requested for 202111 branch label Jan 6, 2022
judyjoseph pushed a commit that referenced this pull request Jan 9, 2022
Why I did it
Config db schema generated from test minigraph can't pass yang validation.

How I did it
Update minigraph xml to add DeploymentId.

How to verify it
Build sonic-config-engine.
Run command 'sonic-cfggen -m xxx.xml --print-data', and check deployment_id field.

Signed-off-by: Gang Lv ganglv@microsoft.com
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Included in 202111 Branch Request for 202111 Branch For PRs being requested for 202111 branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[yang-models] device_metadata validation issue
3 participants