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

[CLI][show bgp] On chassis don't show internal BGP sessions by default #1927

Merged
merged 5 commits into from
Dec 6, 2021

Conversation

smaheshm
Copy link
Contributor

@smaheshm smaheshm commented Nov 15, 2021

What I did

Similar to multi ASIC platform, on chassis don't show internal BGP sessions by default

Unit tests rely on output of 'show ip bgp summary' to check BGP status. Since the original output included internal BGP sessions BGP status check fails as internal sessions are never brought down.

How I did it

perform internal BGP session on chassis

Requires sonic-net/sonic-buildimage#9264

How to verify it

admin@str2-chassis-lc3-1:~$ show ip bgp summary

IPv4 Unicast Summary:
BGP router identifier 192.0.0.3, local AS number 65100 vrf-id 0
BGP table version 8576
RIB entries 17069, using 3277248 bytes of memory
Peers 21, using 458136 KiB of memory
Peer groups 2, using 128 bytes of memory


Neighbhor      V     AS    MsgRcvd    MsgSent    TblVer    InQ    OutQ  Up/Down    State/PfxRcd    NeighborName
-----------  ---  -----  ---------  ---------  --------  -----  ------  ---------  --------------  --------------
10.0.0.1       4  65200       1190       1190         0      0       0  00:45:57   8513            ARISTA01T3
10.0.0.9       4  65200          0          0         0      0       0  never      Active          ARISTA05T3
10.0.0.13      4  65200          0          0         0      0       0  never      Active          ARISTA07T3
10.0.0.17      4  65200          0          0         0      0       0  never      Active          ARISTA09T3
10.0.0.21      4  65200          0          0         0      0       0  never      Active          ARISTA11T3
10.0.0.25      4  65200          0          0         0      0       0  never      Active          ARISTA13T3
.
.


admin@str2-chassis-lc3-1:~$ show ip bgp summary -d all

IPv4 Unicast Summary:
BGP router identifier 10.3.147.15, local AS number 65100 vrf-id 0
BGP table version 8623
RIB entries 17107, using 3284544 bytes of memory
Peers 23, using 501768 KiB of memory
Peer groups 3, using 192 bytes of memory


Neighbhor      V     AS    MsgRcvd    MsgSent    TblVer    InQ    OutQ  Up/Down    State/PfxRcd    NeighborName
-----------  ---  -----  ---------  ---------  --------  -----  ------  ---------  --------------  ---------------
3.3.3.6        4  65100          0          0         0      0       0  never      Active          str2-chassis-lc6-1
3.3.3.7        4  65100         44       8570         0      0       0  00:30:54   20              str2-chassis-lc7-1
10.0.0.1       4  65200       4879       4880         0      0       0  00:30:52   8514            ARISTA01T3
10.0.0.9       4  65202          0          0         0      0       0  never      Active          ARISTA05T3
10.0.0.13      4  65203          0          0         0      0       0  never      Connect         ARISTA07T3
10.0.0.17      4  65204          0          0         0      0       0  never      Connect         ARISTA09T3


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


admin@str2-chassis-lc3-1:~$ show ip bgp summary

IPv4 Unicast Summary:
BGP router identifier 10.3.147.15, local AS number 65100 vrf-id 0
BGP table version 8623
RIB entries 17107, using 3284544 bytes of memory
Peers 23, using 501768 KiB of memory
Peer groups 3, using 192 bytes of memory


Neighbhor      V     AS    MsgRcvd    MsgSent    TblVer    InQ    OutQ  Up/Down    State/PfxRcd    NeighborName
-----------  ---  -----  ---------  ---------  --------  -----  ------  ---------  --------------  ---------------
3.3.3.6        4  65100          0          0         0      0       0  never      Active          str2-chassis-lc6-1
3.3.3.7        4  65100         44       8570         0      0       0  00:30:54   20              str2-chassis-lc7-1
10.0.0.1       4  65200       4879       4880         0      0       0  00:30:52   8514            ARISTA01T3
10.0.0.9       4  65202          0          0         0      0       0  never      Active          ARISTA05T3
10.0.0.13      4  65203          0          0         0      0       0  never      Connect         ARISTA07T3
10.0.0.17      4  65204          0          0         0      0       0  never      Connect         ARISTA09T3


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

@smaheshm
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@smaheshm
Copy link
Contributor Author

needs sonic-net/sonic-buildimage#9290 to pass the builds.

@smaheshm
Copy link
Contributor Author

smaheshm commented Dec 2, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@arlakshm
Copy link
Contributor

arlakshm commented Dec 3, 2021

/Azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@arlakshm
Copy link
Contributor

arlakshm commented Dec 3, 2021

@ysmanman, @vganesan-nokia for viz

Copy link
Contributor

@arlakshm arlakshm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@smaheshm
Copy link
Contributor Author

smaheshm commented Dec 3, 2021

/Azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@smaheshm
Copy link
Contributor Author

smaheshm commented Dec 3, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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.

2 participants