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

Bugs for SONiC CLI Auto-generation #3221

Open
oplklum opened this issue Mar 19, 2024 · 0 comments
Open

Bugs for SONiC CLI Auto-generation #3221

oplklum opened this issue Mar 19, 2024 · 0 comments

Comments

@oplklum
Copy link

oplklum commented Mar 19, 2024 •
edited
Loading

Description

Currently I'm working on a CLI application with SONiC application extension, and also using SONiC CLI Auto-generation tools.
The CLI will be automatically generated by SONiC yang model. But there are some issues listed below when I test the CLI.

  1. Command description error for config update.
image

It should be 'update' instead of 'add'.

  1. Still displaying the command of not configurable table that defined config false in the yang model.
image image
  1. Cannot show other db (eg. state db) table data except config db.
image image

ATTENUATOR table is in config db, and ATTENUATOR_TABLE table is in state db. It can only display ATTENUATOR table elements with attenuator command, but ATTENUATOR_TABLE is empty by attenuator-table command.

Steps to reproduce the issue

  1. Please use the attached yang model file to test the CLI

Describe the results you received

Describe the results you expected

Additional information you deem important (e.g. issue happens only occasionally)

Output of show version

(paste your output here)

sonic-optical-attenuator.zip
oplklum added a commit to oplklum/sonic-utilities that referenced this issue Mar 19, 2024
@oplklum
Solve the bugs for SONiC CLI Auto-generation (sonic-net#3221)
d6c86bc 
1. Fixed the error description for config update
2. Not displaying the command of not configurable table that defined config false in the yang model.
3. Support showing other db table data defined in sonic yang model, not just config db.
@oplklum oplklum mentioned this issue Mar 19, 2024
Open
oplklum added a commit to oplklum/sonic-utilities that referenced this issue Mar 19, 2024
@oplklum
Solve the bugs for SONiC CLI Auto-generation (sonic-net#3221)
8dd25a9 
1. Fixed the error description for config update
2. Not displaying the command of not configurable table that defined config false in the yang model.
3. Support showing other db table data defined in sonic yang model, not just config db.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@oplklum