Multi-ASIC support for show ip(v6) route (201911 branch)#1283
Merged
gechiang merged 3 commits intosonic-net:201911from Dec 8, 2020
gechiang:201911
Merged
Multi-ASIC support for show ip(v6) route (201911 branch)#1283gechiang merged 3 commits intosonic-net:201911from gechiang:201911
gechiang merged 3 commits intosonic-net:201911from
gechiang:201911
Conversation
|
This pull request introduces 2 alerts when merging 65c5c18 into 28399bf - view on LGTM.com new alerts:
|
stepanblyschak
pushed a commit
to stepanblyschak/sonic-utilities
that referenced
this pull request
Apr 28, 2022
ccb5245 (HEAD -> 201911, origin/201911) [fast-reboot] Fix fast-reboot when NDP entries are present (sonic-net#1295) d09667b Multi-ASIC support for show ip(v6) route (201911 branch) (sonic-net#1283) 28399bf [201911-Mellanox] SKU creator Tool (sonic-net#1163) (sonic-net#1250) Signed-off-by: Abhishek Dosi <[email protected]>
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Add Multi-ASIC support to handle "show ip/ipv6 route" on multi-ASIC devices
Porting (#1216) to 201911 branch
Signed-off-by: Gen-Hwa Chiang [email protected]
- What I did
Add support for multi ASIC CLI options for "show ip/ipv6 route"
2 new options have added
[-n, --namespace] to allow user to display the information for given namespaces (ASIC)
If this option is not present the information from all the namespaces will be displayed
[-d, --display] to allow user to display ip routes related with nexthop that are going through both internal and external interfaces
If this option is not present only ip routes with external interfaces as its nexthop will be display
On single ASIC platform, this options are not valid, so the behavior remains unchanged
- How I did it
- How to verify it
Help menu
show ip route for all routes (include routes that uses internal interface as nexthop) from ASIC0 in multi ASIC device
show ip route (exclude routes that uses internal interface as nexthop) from ASIC0 in multi ASIC device
show ip route (exclude routes that uses internal interface as nexthop) from ASIC5 in multi ASIC device in json format
show ip route (exclude routes that uses internal interface as nexthop) from ASIC5 in multi ASIC device
show ip route for all routes (exclude routes that uses internal interface as nexthop) from ALL ASICs in multi ASIC device
show ip route for all routes (include routes that uses internal interface as nexthop) from ALL ASICs in multi ASIC device
show ipv6 route for specific route from all ASICs (include routes that uses internal interface as nexthop) in multi ASIC device
show ip route for specific route from all ASICs and how that is combined when only front-end interface filtering applied
- 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)
** Please Note **
This is the same PR that I raised previously "#1089" which I closed due to design was changed to handle IPV6 as well as the new "merge routes" criteria.