-
Notifications
You must be signed in to change notification settings - Fork 691
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
Added auto-generated CLIs from Fine-Grained Hashing Yang model #3740
base: master
Are you sure you want to change the base?
Conversation
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
b43af96
to
df096dc
Compare
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
|
||
|
||
# Load sonic-cfggen from source since /usr/local/bin/sonic-cfggen does not have .py extension. | ||
sonic_cfggen = general.load_module_from_source('sonic_cfggen', '/usr/local/bin/sonic-cfggen') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pls add test case for the new function into https://github.com/sonic-net/sonic-utilities/tree/df096dcbea47ed96fcfe2444855625e78707bf04/tests
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@FengPan-Frank I've added test cases now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@FengPan-Frank I've added unit tests for the CLI.
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
b4de958
to
1498c6d
Compare
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
4d5be4e
to
c6d9993
Compare
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
1dd24c7
to
f006034
Compare
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
What I did
Generated config and show CLIs from Fine-Grained ECMP Yang model.
PR for Yang Model changes:
sonic-net/sonic-buildimage#21545
How I did it
sonic-cli-gen generate config sonic-fine-grained-ecmp
sonic-cli-gen generate show sonic-fine-grained-ecmp
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)