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: release manual vpn label on instance deletion #18121

Merged
merged 1 commit into from
Feb 13, 2025

Conversation

louis-6wind
Copy link
Contributor

When a BGP instance with a manually assigned VPN label is deleted, the label is not released from the Zebra label registry. As a result, reapplying a configuration with the same manual label leads to VPN prefix export failures.

For example, with the following configuration:

router bgp 65000 vrf BLUE
address-family ipv4 unicast
label vpn export

Release zebra label registry on unconfiguration.

Fixes: d162d5f ("bgpd: fix hardset l3vpn label available in mpls pool")

When a BGP instance with a manually assigned VPN label is deleted, the
label is not released from the Zebra label registry. As a result,
reapplying a configuration with the same manual label leads to VPN
prefix export failures.

For example, with the following configuration:

> router bgp 65000 vrf BLUE
>  address-family ipv4 unicast
>   label vpn export <int>

Release zebra label registry on unconfiguration.

Fixes: d162d5f ("bgpd: fix hardset l3vpn label available in mpls pool")
Signed-off-by: Louis Scalbert <louis.scalbert@6wind.com>
@ton31337
Copy link
Member

@Mergifyio backport dev/10.3 stable/10.2 stable/10.1 stable/10.0

Copy link

mergify bot commented Feb 12, 2025

backport dev/10.3 stable/10.2 stable/10.1 stable/10.0

✅ Backports have been created

@ton31337 ton31337 merged commit cadfc56 into FRRouting:master Feb 13, 2025
18 checks passed
donaldsharp added a commit that referenced this pull request Feb 13, 2025
bgpd: release manual vpn label on instance deletion (backport #18121)
riw777 added a commit that referenced this pull request Feb 18, 2025
bgpd: release manual vpn label on instance deletion (backport #18121)
riw777 added a commit that referenced this pull request Feb 18, 2025
bgpd: release manual vpn label on instance deletion (backport #18121)
riw777 added a commit that referenced this pull request Feb 18, 2025
bgpd: release manual vpn label on instance deletion (backport #18121)
Jafaral added a commit that referenced this pull request Feb 19, 2025
…le/10.2/pr-18121

Revert "bgpd: release manual vpn label on instance deletion (backport #18121)"
Jafaral added a commit that referenced this pull request Feb 19, 2025
…le/10.0/pr-18121

Revert "bgpd: release manual vpn label on instance deletion (backport #18121)"
Jafaral added a commit that referenced this pull request Feb 19, 2025
…le/10.1/pr-18121

Revert "bgpd: release manual vpn label on instance deletion (backport #18121)"
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