-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
chassis: 'show interface portchannel' shows internal portchannels included #14459
Comments
In `show interface portchannel` and `show ip route`, backend port channels and routes were being displayed. This is due to changes in sonic-net#13660. Fix these issues by switching to reading from PORTCHANNEL_MEMBERS table instead. Fixes sonic-net#14459. Signed-off-by: Saikrishna Arcot <[email protected]>
There is a bug in sonic_py_common/multi_asic.py is_port_channel_internal function. The fix is to use PORTCHANNEL_MEMBER table. I am testing a fix. We can mark this issue dup of #14436 |
realized, there is a PR already in process for this. That should fix #14436 as well. |
* Fix backend port channels and routes being displayed In `show interface portchannel` and `show ip route`, backend port channels and routes were being displayed. This is due to changes in #13660. Fix these issues by switching to reading from PORTCHANNEL_MEMBERS table instead. Fixes #14459. * Replace table name with constant Signed-off-by: Saikrishna Arcot <[email protected]>
* Fix backend port channels and routes being displayed In `show interface portchannel` and `show ip route`, backend port channels and routes were being displayed. This is due to changes in sonic-net#13660. Fix these issues by switching to reading from PORTCHANNEL_MEMBERS table instead. Fixes sonic-net#14459. * Replace table name with constant Signed-off-by: Saikrishna Arcot <[email protected]>
Description
'show interface portchannel' is supposed to display only external portchannels.
'-d all' option is used to display all.
But looks like after #13660 change,
the CLI shows all port channels including the back plane ones without '-d all' option.
Steps to reproduce the issue:
Describe the results you received:
command included internal port channels with back port members.
Describe the results you expected:
command should not included internal port channels with back port members.
Output of
show version
:Output of
show techsupport
:Additional information you deem important (e.g. issue happens only occasionally):
The text was updated successfully, but these errors were encountered: