Skip to content

bgpd: adjust display format#19141

Merged
ton31337 merged 1 commit intoFRRouting:masterfrom
anlancs:bgpd/fix-space-display
Jul 8, 2025
Merged

bgpd: adjust display format#19141
ton31337 merged 1 commit intoFRRouting:masterfrom
anlancs:bgpd/fix-space-display

Conversation

@anlancs
Copy link
Contributor

@anlancs anlancs commented Jul 7, 2025

Just adjust display format for show bgp ipv6 unicast, add two white spaces.

Before:

     Network          Next Hop            Metric LocPrf Weight Path
 *>i 3001::/64        fe80::2e53:4aff:fe30:820
                                             0    100      0 i

After:

     Network          Next Hop            Metric LocPrf Weight Path
 *>i 3001::/64        fe80::2e53:4aff:fe30:820
                                               0    100      0 i

Just adjust display format for `show bgp ipv6 unicast`, add two white spaces.

Before:
```
     Network          Next Hop            Metric LocPrf Weight Path
 *>i 3001::/64        fe80::2e53:4aff:fe30:820
                                             0    100      0 i
```

After
```
     Network          Next Hop            Metric LocPrf Weight Path
 *>i 3001::/64        fe80::2e53:4aff:fe30:820
                                               0    100      0 i
```

Signed-off-by: anlan_cs <anlan_cs@126.com>
@anlancs anlancs force-pushed the bgpd/fix-space-display branch from 9ae6eaf to 1332311 Compare July 7, 2025 03:06
@ton31337
Copy link
Member

ton31337 commented Jul 7, 2025

Could you check if it's not broken for other AFIs/SAFIs, including the wide option? (Not saying this commit breaks something, but since you are here, it's worth asking this)

@anlancs
Copy link
Contributor Author

anlancs commented Jul 7, 2025

@ton31337
Yes, i have tested ipv6/ipv4, and wide for them. ipv4 is short string, so it can't enter this scene which requires new line.
But the left cases are strange to me, so i didn't test them....
As to the code, the left cases should have the same procedure.

@ton31337 ton31337 merged commit 974eb29 into FRRouting:master Jul 8, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants