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

Solve the bugs for SONiC CLI Auto-generation #3222

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

Conversation

oplklum
Copy link

@oplklum oplklum commented Mar 19, 2024

Solve the bugs for SONiC CLI Auto-generation referenced in #3221

  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.

What I did

  1. Fixed the error description for config update
  2. Support config and db-name query in yang_parser.py
  3. Change config.py.j2 to filter config false table
  4. Extend show.py.j2 to support query of non confg db.
  5. Compatible with previous programs and tests.

How I did it

How to verify it

Previous command output (if the output of a command-line utility has changed)

New command output (if the output of a command-line utility has changed)

image

Copy link

linux-foundation-easycla bot commented Mar 19, 2024

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: oplklum / name: MaoLu (8dd25a9)

@oplklum oplklum changed the title Solve the bugs for SONiC CLI Auto-generation (#3221) Solve the bugs for SONiC CLI Auto-generation Mar 19, 2024
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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant