bgpd: show json output changes to optimize various show commands #17431
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
ton31337
requested changes
Nov 14, 2024
krishna-samy
force-pushed
the
bgpd_json_commits
branch
3 times, most recently
from
c32f90a
to
b2af790
Compare
Introduced JSON support of "show bgp router" command
VTY:
mlx-3700-19# show bgp router
BGP started gracefully at Tue Nov 14 21:18:34 2023
Graceful restart completed at Tue Nov 14 21:18:37 2023
Number of BGP instances (including default): 1
JSON:
mlx-3700-19# show bgp router json
{
"bgpStartedAt":"Tue Nov 14 21:18:34 2023\n",
"bgpStartedGracefully":"Yes",
"grComplete":"Yes",
"grCompletedAt":"Tue Nov 14 21:18:37 2023\n",
"bgpInMaintenanceMode":"No",
"bgpInstanceCount":1
}
Issue:3624937
Signed-off-by: Pooja Jagadeesh Doijode <[email protected]>
Json object could lead to out-of-memory in scaled bgp l2vpn evpn route setup. Changes: - not use pretty print and stringify smaller json objects to reduce memory usage - free memory after ouput of json_rd - minor formatting of json output Commands supported with this Json stringify: show bgp l2vpn evpn route detail json show bgp l2vpn evpn route detail type 2 json show bgp l2vpn evpn route detail type 2 self-originate json show bgp l2vpn evpn route detail self-originate json show bgp l2vpn evpn route json show bgp l2vpn evpn route type 2 json show bgp l2vpn evpn route type 2 self-originate json show bgp l2vpn evpn route self-originate json Ticket:#3513249 Issue:3513249 Signed-off-by: Ashwini Reddy <[email protected]> Signed-off-by: Sindhu Parvathi Gopinathan's <[email protected]>
krishna-samy
force-pushed
the
bgpd_json_commits
branch
5 times, most recently
from
cc3769b
to
819db39
Compare
|
Please fix frrbot yet. |
ton31337
requested changes
Nov 16, 2024
JSON object could lead to out-of-memory in scaled bgp neighbors received-routes/advertised-routes setup. Changes: not use pretty print and stringify smaller json objects to reduce memory usage free memory after ouput of json object Commands supported with this Json stringify: ``` show bgp vrf <vrf-id> neighbors <nbr-id> advertised-routes json show bgp vrf <vrf-id> neighbors <nbr-id> advertised-routes detail json show bgp vrf <vrf-id> neighbors <nbr-id> advertised-routes <prefix> json show bgp vrf <vrf-id> neighbors <nbr-id> received-routes json show bgp vrf <vrf-id> neighbors <nbr-id> received-routes detail json show bgp vrf <vrf-id> neighbors <nbr-id> received-routes <prefix> json ``` Ticket:#3513253, #3513254 Issue:3513253, 3513254 Signed-off-by: Sindhu Parvathi Gopinathan's <[email protected]>
Issue: 3576989 Ticket# 3576989 Signed-off-by: Sindhu Parvathi Gopinathan's <[email protected]> Signed-off-by: Ashwini Reddy's <[email protected]>
krishna-samy
force-pushed
the
bgpd_json_commits
branch
from
819db39
to
6c0726e
Compare
This commit has changes to validate show bgp router json command Signed-off-by: Krishnasamy R <[email protected]>
Signed-off-by: Krishnasamy R <[email protected]>
krishna-samy
force-pushed
the
bgpd_json_commits
branch
from
6c0726e
to
631ba8f
Compare
Done |
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.
This PR includes multiple commits that optimize the show json output for various BGP show commands in scaled setup.