Skip to content

[sonic-yang]: introduce yang template for cvl yang #8880

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 2 commits into from
Nov 6, 2021

Conversation

lguohan
Copy link
Collaborator

@lguohan lguohan commented Sep 30, 2021

…models

python3 setup.py build
python3 setup.py sdist

Signed-off-by: Guohan Lu [email protected]

Why I did it

How I did it

How to verify it

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

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106

Description for the changelog

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

@lguohan lguohan added the YANG YANG model related changes label Sep 30, 2021
@lguohan
Copy link
Collaborator Author

lguohan commented Oct 7, 2021

related #6861

Copy link
Contributor

@anand-kumar-subramanian anand-kumar-subramanian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yang model type should be CVL and PV(python validator) instead of int and ext.

@zhangyanzhao
Copy link

Yang model type should be CVL and PV(python validator) instead of int and ext.

@lguohan would you please double check this and get the build issue fixed? Thanks

@maheshwari-mayank
Copy link
Contributor

Is Python validator going to validate external yangs? As external yangs will have specific extensions which Python validator does not understand.

@lguohan
Copy link
Collaborator Author

lguohan commented Nov 4, 2021

/azp run

@azure-pipelines
Copy link

Pull request contains merge conflicts.

@lguohan lguohan force-pushed the yangtmpl2 branch 2 times, most recently from 446c0bf to 5988c7a Compare November 4, 2021 16:43
…models

python3 setup.py build
python3 setup.py sdist

Signed-off-by: Guohan Lu <[email protected]>
@lguohan lguohan changed the title [sonic-yang]: introduce yang template for internal and external yang … [sonic-yang]: introduce yang template for cvl yang Nov 6, 2021
@lguohan lguohan marked this pull request as ready for review November 6, 2021 07:42
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.

4 participants