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

[202205] Vnet table yang models #15500

Open
wants to merge 1 commit into
base: 202205
Choose a base branch
from

Conversation

siqbal1986
Copy link
Contributor

Created Yang Modle for VNET table.
Orignal PR : #14534

  • Microsoft ADO (number only): 18215579
    18215579

Why I did it

Work item tracking
  • Microsoft ADO (number only):

How I did it

How to verify it

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111
  • 202205
  • 202211

Tested branch (Please provide the tested image version)

Description for the changelog

Link to config_db schema for YANG module changes

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

Created Yang Modle for VNET table.
sonic-net#14534

- Microsoft ADO **(number only)**:
18215579
@siqbal1986 siqbal1986 requested a review from qiluo-msft as a code owner June 15, 2023 23:49
@siqbal1986 siqbal1986 requested a review from prsunny June 15, 2023 23:49
@prsunny
Copy link
Contributor

prsunny commented Jun 16, 2023

/azp run

@azure-pipelines
Copy link

You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list.

@siqbal1986
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Commenter does not have sufficient privileges for PR 15500 in repo sonic-net/sonic-buildimage

@siqbal1986
Copy link
Contributor Author

/azpw run

@mssonicbld
Copy link
Collaborator

/AzurePipelines run

@azure-pipelines
Copy link

You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list.

@siqbal1986
Copy link
Contributor Author

/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).

@prsunny
Copy link
Contributor

prsunny commented Jun 26, 2023

@yxieca for viz and merge

@siqbal1986
Copy link
Contributor Author

/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).

@yxieca
Copy link
Contributor

yxieca commented Jun 27, 2023

@siqbal1986 looks like the build failures were caused by this change, please fix the unit tests.

@siqbal1986
Copy link
Contributor Author

@yxieca the failure is because there is no yang model available for vxlan. the port to 202205 is missing. the master PR was https://github.com/sonic-net/sonic-buildimage/pull/7294/files. I have requested that change for 202205 now.

@zhangyanzhao
Copy link
Collaborator

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

@yxieca
Copy link
Contributor

yxieca commented Nov 10, 2023

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

@siqbal1986 @zhangyanzhao the buildimage dependency PR hasn't been cherry-picked yet due to conflicts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants