-
Notifications
You must be signed in to change notification settings - Fork 667
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
[DRAFT] sonic-utilities: Create SONiC ISIS show commands #2685
Open
cchoate54
wants to merge
10
commits into
sonic-net:master
Choose a base branch
from
cchoate54:sonic_isis_base_show_command
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
[DRAFT] sonic-utilities: Create SONiC ISIS show commands #2685
cchoate54
wants to merge
10
commits into
sonic-net:master
from
cchoate54:sonic_isis_base_show_command
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
cchoate54
force-pushed
the
sonic_isis_base_show_command
branch
2 times, most recently
from
February 17, 2023 21:43
2da799a
to
b473dd5
Compare
* Update show/main.py Include "isis" as a cli group option for show commands * Create show/isis_frr.py Add "isis" as a cli group option for show commands and add support for "show isis neighbors", "show isis database", "show isis hostname", and "show isis interface" Signed-off-by: cchoate54@gmail.com
cchoate54
force-pushed
the
sonic_isis_base_show_command
branch
from
February 22, 2023 23:11
b473dd5
to
7497c3f
Compare
Signed-off-by: ideleon@microsoft.com
Signed-off-by: ideleon@microsoft.com
Signed-off-by: ideleon@microsoft.com
[show/tests] Add support for "show run isis"
Signed-off-by: cchoate <cchoate@microsoft.com>
…s_summary_no_json
…_json [show/tests] Add support for "show isis summary"
cchoate54
force-pushed
the
sonic_isis_base_show_command
branch
9 times, most recently
from
April 24, 2023 18:53
fc89b29
to
2d5d1d7
Compare
is there a design doc for is-is feature in sonic? we need to review the design doc before proceeding with any PR related. |
cchoate54
force-pushed
the
sonic_isis_base_show_command
branch
from
April 24, 2023 19:18
2d5d1d7
to
e2a8c58
Compare
[show/tests] Add support for "show isis topology"
cchoate54
force-pushed
the
sonic_isis_base_show_command
branch
from
April 24, 2023 20:12
e2a8c58
to
59ed4cb
Compare
Hi Guohan,
Yes, there is a design doc for this, and I'll be reviewing it in the weekly
meeting tomorrow.
Design doc: Isis configuration support by cchoate54 · Pull Request #1234 ·
sonic-net/SONiC (github.com) <sonic-net/SONiC#1234>
Thanks,
Caitlin
…On Mon, Apr 24, 2023 at 12:08 PM Guohan Lu ***@***.***> wrote:
is there a design doc for is-is feature in sonic? we need to review the
design doc before proceeding with any PR related.
—
Reply to this email directly, view it on GitHub
<#2685 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/A3KP5NFCOLTZSPMRRFO2DEDXC3FUPANCNFSM6AAAAAAU62VWX4>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Signed-off-by: cchoate54@gmail.com
What I did
Implemented new show isis commands.
How I did it
Added a new file for isis show commands called show/isis_frr.py and added the new file as an import to show/main/py
How to verify it
Included unittests for each show command and the different show command argument options.
Tested the code on a VS image.
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)