bgpd: fix for the validity and the presence of prefixes in the BGP VPN table. #17370
Conversation
This file contains hidden or 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
106e889 to
fdd3c21
Compare
fdd3c21 to
ad5959e
Compare
ad5959e to
0a4f3e1
Compare
b4c2e9b to
dc3a481
Compare
|
ci:rerun |
1 similar comment
|
ci:rerun |
ton31337
reviewed
Nov 26, 2024
|
ci:rerun |
dc3a481 to
c821483
Compare
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
|
conflicts need to be resolved and @ton31337 's comment addressed before we can push ... |
c821483 to
7712a51
Compare
|
Just waiting on @ton31337 's comments now ... |
7712a51 to
d2ae894
Compare
|
Just rebased to make sure everything is still OK. Waiting on @ton31337 comments |
|
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
|
@Mergifyio rebase |
☑️ Nothing to do
|
|
need to fix the conflicts ... |
They are the same value. Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>
Mark a nexthop as invalid if the origin VRF is unusable, either because it does not exist or its interface is down. Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>
When leak_update() rechecks an existing path, it considers nothing to update if the attributes and labels are not changed. However, it does not take into account the nexthop validity. Perform a leak update if the nexthop validity has changed. Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>
Do not leak if origin vrf is not usable Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>
Update route_leak_basic tests. The routes with an unusable nexthop VRF are no more present in the RIB. Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>
The bgp_bmp test is failing because r2 lacks the vrf1 VRF, which prevents it from exporting VPN prefixes from the configured vrf1 BGP instance. Previous versions allowed the export of static BGP prefixes from a non-existent VRF, so the test passed under those conditions. Add a vrf1 VRF on r2. Fixes: d748544 ("topotests: add basic bgp bmp test") Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>
Check bgp vpn table in bgp_vpnv4_route_leak_basic Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>
In bgp_vpnv4_route_leak_basic, remove and add back the static prefix 172.16.3.0/24 on VRF DONNA. Before the previous fixes, the 172.16.3.0/24 prefix re-appeared when it was added back in the VPN table but it was marked as invalid. Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>
d2ae894 to
fcae7a4
Compare
2 tasks
|
Quickly checked 10.3 stable, apparently this fix is not part of the release. Any chance this will be introduced 10.3.1? Thanks |
|
I'd say 3-4 weeks to bundle more bugfixes. |
2 tasks
|
@Mergifyio backport stable/10.3 |
✅ Backports have been created
|
Jafaral
added a commit
that referenced
this pull request
Jun 5, 2025
bgpd: fix for the validity and the presence of prefixes in the BGP VPN table. (backport #17370)
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.
Fix for the validity and the presence of prefixes in the BGP VPN table.
See individial commit logs