Skip to content

bgpd: Ignore transitiveness flag when checking type for link bandwidth (backport #20607)#20750

Merged
donaldsharp merged 2 commits intostable/10.5from
mergify/bp/stable/10.5/pr-20607
Feb 10, 2026
Merged

bgpd: Ignore transitiveness flag when checking type for link bandwidth (backport #20607)#20750
donaldsharp merged 2 commits intostable/10.5from
mergify/bp/stable/10.5/pr-20607

Conversation

@mergify
Copy link

@mergify mergify bot commented Feb 10, 2026


This is an automatic backport of pull request #20607 done by [Mergify](https://mergify.com).

…h extcomm

If the community comes as non-transitive, we incorrectly take it and never consider
using for weighted load balancing.

draft-ietf-idr-link-bandwidth-24 defines:

If a BGP speaker receives a route with more than one Link Bandwidth Extended
Communities and uses the route to compute weighted load balancing, it SHOULD
use the extended community with the lowest bandwidth value (including zero),
ignoring the transitivity. Implementations MAY provide configuration to change
the above preference.

The ordering is already done at the ingestion ecommunity_parse() when we receive
a new community/value. So we always take the lowest one.

Fixes: 60eff2e ("bgpd: Handle non-transitive opaque extended communities also for eBGP peers")

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
(cherry picked from commit f14b771)
Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
(cherry picked from commit f1483e6)
@greptile-apps
Copy link

greptile-apps bot commented Feb 10, 2026

Target branch is not in the allowed branches list.

@frrbot frrbot bot added bgp bugfix tests Topotests, make check, etc labels Feb 10, 2026
@donaldsharp donaldsharp merged commit 9d05f27 into stable/10.5 Feb 10, 2026
20 of 25 checks passed
@mergify mergify bot deleted the mergify/bp/stable/10.5/pr-20607 branch February 10, 2026 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants