Skip to content

bgp: Introduce --with-attrs option to bgp/routes#45015

Merged
YutaroHayakawa merged 2 commits intomainfrom
yutaro/bgp-print-attrs
Mar 31, 2026
Merged

bgp: Introduce --with-attrs option to bgp/routes#45015
YutaroHayakawa merged 2 commits intomainfrom
yutaro/bgp-print-attrs

Conversation

@YutaroHayakawa
Copy link
Copy Markdown
Member

Introduce a new option --with-attrs to bgp/routes which displays path attribute. In addition to the normal output.

> bgp/routes loc ipv4 unicast --with-attrs
Instance   Prefix           NextHop    Best   Age      Attrs
rack0      100.64.0.1/32    10.0.0.1   true   32m27s   [{Origin: i} {AsPath: 65200 65000 65100}]
           100.64.1.11/32   0.0.0.0    true   32m29s   [{Origin: i}]

This is still optional as in some cases, printing path attribute makes the output horizontally very long.

bgp: Introduce --with-attrs option to bgp/routes

@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Mar 27, 2026
@YutaroHayakawa YutaroHayakawa added the release-note/misc This PR makes changes that have no direct user impact. label Mar 27, 2026
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Mar 27, 2026
@YutaroHayakawa
Copy link
Copy Markdown
Member Author

/test

@YutaroHayakawa YutaroHayakawa marked this pull request as ready for review March 27, 2026 04:19
@YutaroHayakawa YutaroHayakawa requested a review from a team as a code owner March 27, 2026 04:19
We mistakenly documented the peer name argument in bgp/routes despite it
is not yet implemented. Remove it.

Signed-off-by: Yutaro Hayakawa <yutaro.hayakawa@isovalent.com>
Introduce a new option --with-attrs to bgp/routes which displays path
attribute. In addition to the normal output.

```
> bgp/routes loc ipv4 unicast --with-attrs
Instance   Prefix           NextHop    Best   Age      Attrs
rack0      100.64.0.1/32    10.0.0.1   true   32m27s   [{Origin: i} {AsPath: 65200 65000 65100}]
           100.64.1.11/32   0.0.0.0    true   32m29s   [{Origin: i}]
```

This is still optional as in some cases, printing path attribute makes
the output horizontally very long.

Signed-off-by: Yutaro Hayakawa <yutaro.hayakawa@isovalent.com>
@YutaroHayakawa YutaroHayakawa force-pushed the yutaro/bgp-print-attrs branch from d897e38 to 2a814db Compare March 30, 2026 18:55
@YutaroHayakawa
Copy link
Copy Markdown
Member Author

/test

1 similar comment
@YutaroHayakawa
Copy link
Copy Markdown
Member Author

/test

@YutaroHayakawa
Copy link
Copy Markdown
Member Author

@YutaroHayakawa YutaroHayakawa added this pull request to the merge queue Mar 31, 2026
Merged via the queue into main with commit a4be772 Mar 31, 2026
720 of 770 checks passed
@YutaroHayakawa YutaroHayakawa deleted the yutaro/bgp-print-attrs branch March 31, 2026 21:39
@YutaroHayakawa YutaroHayakawa added backport/author The backport will be carried out by the author of the PR. needs-backport/1.19 This PR / issue needs backporting to the v1.19 branch and removed backport/author The backport will be carried out by the author of the PR. labels Mar 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-backport/1.19 This PR / issue needs backporting to the v1.19 branch release-note/misc This PR makes changes that have no direct user impact.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants