Skip to content

Conversation

ton31337
Copy link
Member

Closes #17142

@frrbot frrbot bot added bgp tests Topotests, make check, etc labels Oct 17, 2024
@ton31337 ton31337 force-pushed the fix/send_link_bw_as_non-transitive_to_ebgp branch from bb4f8f7 to 765a085 Compare October 17, 2024 08:45
@frrbot frrbot bot added the bugfix label Oct 17, 2024
Copy link

This pull request has conflicts, please resolve those before we can evaluate the pull request.

If we received a non-transitive extended community (in this case it was
extended link-bandwidth), we treated it as unknown because we didn't check for
the transitive flag correctly.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
…ceived correctly

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
Extended communities can be transitive or non-transitive.

Like other attributes (e.g., MED) non-transitive extended communities SHOULD
be sent to the direct peer, but not forward them to eBGP peers next.

Before this patch, we never send non-transitive extended attributes to the
direct peers at all.

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
…HECK_FLAG()

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
… peers

Fixes: 765a085 ("bgpd: Rework extended community transitiviness")

Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
@ton31337 ton31337 force-pushed the fix/send_link_bw_as_non-transitive_to_ebgp branch from aefea72 to 60eff2e Compare October 22, 2024 06:05
@riw777 riw777 self-requested a review October 22, 2024 15:02
Copy link
Member

@riw777 riw777 left a comment

Choose a reason for hiding this comment

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

looks good

@riw777 riw777 merged commit 56944a5 into FRRouting:master Oct 22, 2024
11 checks passed
@ton31337 ton31337 deleted the fix/send_link_bw_as_non-transitive_to_ebgp branch October 22, 2024 15:26
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.

non-transitive Bandwidth ext-community setting on outbound route-map to eBGP peers
2 participants