Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bgpd: Request SRv6 locator after zebra connection (backport #18069) #18115

Merged
merged 1 commit into from
Feb 12, 2025

Conversation

mergify[bot]
Copy link

@mergify mergify bot commented Feb 12, 2025

When SRv6 is enabled and an SRv6 locator is specified in the BGP configuration, BGP may attempt to request SRv6 locator information from zebra before the connection is fully established. If this occurs, the request fails with the following error:

2025/02/06 16:37:32 BGP: [HR66R-TWQYD][EC 100663302] srv6_manager_get_locator: invalid zclient socket

As a result, BGP is unable to obtain the locator information, preventing SRv6 VPN from working.

This PR fixes the issue by ensuring BGP requests SRv6 locator information once the connection with zebra is successfully established.


This PR fixes intermittent failures in BGP SRv6 L3VPN topotests.


This is an automatic backport of pull request #18069 done by Mergify.

When SRv6 is enabled and an SRv6 locator is specified in the BGP
configuration, BGP may attempt to request SRv6 locator information from
zebra before the connection is fully established. If this occurs, the
request fails with the following error:

```
2025/02/06 16:37:32 BGP: [HR66R-TWQYD][EC 100663302] srv6_manager_get_locator: invalid zclient socket
````

As a result, BGP is unable to obtain the locator information,
preventing SRv6 VPN from working.

This commit fixes the issue by ensuring BGP requests SRv6 locator
information once the connection with zebra is successfully established.

Signed-off-by: Carmine Scarpitta <cscarpit@cisco.com>
(cherry picked from commit 16640b6)
@frrbot frrbot bot added the bgp label Feb 12, 2025
@donaldsharp donaldsharp merged commit 60dbf7e into dev/10.3 Feb 12, 2025
14 of 16 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