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.

Sign up for GitHub

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

Jump to bottom

Need Yang for SYSTEM_PORT table #12458

Closed
ganglyu opened this issue Oct 20, 2022 · 2 comments · Fixed by #12689
Closed

Need Yang for SYSTEM_PORT table #12458

ganglyu opened this issue Oct 20, 2022 · 2 comments · Fixed by #12689
Labels
Help Wanted 🆘 Triaged this issue has been triaged YANG YANG model related changes

Comments

@ganglyu
Copy link
Contributor

ganglyu commented Oct 20, 2022

We don't have Yang model for SYSTEM_PORT table.

Description

Steps to reproduce the issue:

  1. Run build pipeline for below PR:
    [sonic-cfggen] Track missing yang models #10963
  2. Check build log:
    sonic_yang(6):Note: Below table(s) have no YANG models: SYSTEM_PORT, TUNNEL

Describe the results you received:

sonic-config-engine unit test failed.

Describe the results you expected:

sonic-config-engine unit test should pass.

Output of show version:

(paste your output here)

Output of show techsupport:

(paste your output here or download and attach the file here )

Additional information you deem important (e.g. issue happens only occasionally):
@ganglyu ganglyu added the YANG YANG model related changes label Oct 20, 2022
@ganglyu
Copy link
Contributor Author

ganglyu commented Oct 21, 2022

@mlorrillere
Did you introduce SYSTEM_PORT table?

@zhangyanzhao zhangyanzhao added Help Wanted 🆘 Triaged this issue has been triaged labels Oct 27, 2022
@qiluo-msft qiluo-msft mentioned this issue Jan 5, 2023
Merged
7 tasks
@zhangyanzhao
Copy link
Collaborator

#12689 merged and close this issue

qiluo-msft pushed a commit that referenced this issue Feb 2, 2023
@kenneth-arista
[yang-models] Add YANG model for SYSTEM_PORT (#12689)
9d19ac9 
Add YANG model for SYSTEM_PORT.
Resolves #12458

#### Why I did it
YANG model for SYSTEM_PORT in CONFIG_DB was missing. 

#### How I did it
Added new YANG model and associated unit tests.

#### How to verify it
Passing unit tests
@gechiang gechiang mentioned this issue Apr 19, 2024
Merged
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Help Wanted 🆘 Triaged this issue has been triaged YANG YANG model related changes
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[yang-models] Add YANG model for SYSTEM_PORT kenneth-arista/sonic-buildimage
2 participants
@zhangyanzhao @ganglyu