frr icon indicating copy to clipboard operation
frr copied to clipboard

bgpd: fix do not use import|export vpn when import vrf command used

Open pguibert6WIND opened this issue 5 months ago • 1 comments

The following error happens:

red1(config)# router bgp 500 vrf RED_B red1(config-router)# address-family ipv4 unicast red1(config-router-af)# import vrf default red1(config-router-af)# import vpn red1(config-router-af)# import vrf afz % error: Please unconfigure vpn to vrf commands before using import vrf commands

Do not use the import|export vpn command while vrf command is used.

pguibert6WIND avatar Jun 16 '25 08:06 pguibert6WIND

And vice versa, it's okay?

red1(config-router-af)# import vrf default
red1(config-router-af)# import vpn

ton31337 avatar Jun 16 '25 08:06 ton31337

And vice versa, it's okay?

red1(config-router-af)# import vrf default
red1(config-router-af)# import vpn
red2(config-router-af)# import vrf default 
red2(config-router-af)# import vpn 
% error: Please unconfigure import vrf commands before using vpn commands

pguibert6WIND avatar Jun 16 '25 20:06 pguibert6WIND

Sorry, I meant:

red1(config-router-af)# import vpn
red1(config-router-af)# import vrf default

ton31337 avatar Jun 17 '25 05:06 ton31337

im

Sorry, I meant:

red1(config-router-af)# import vpn
red1(config-router-af)# import vrf default
red2(config-router-af)# import vpn 
red2(config-router-af)# import vrf default 
% error: Please unconfigure vpn to vrf commands before using import vrf commands
red2(config-router-af)# 

pguibert6WIND avatar Jun 17 '25 06:06 pguibert6WIND

can we add a topotest that shows that this works properly now?

donaldsharp avatar Jun 17 '25 13:06 donaldsharp

LGTM, waiting for a topotest?

yes, a topotest. I don't have too much time to do it now. so you will have to wait a bit..

pguibert6WIND avatar Jun 20 '25 06:06 pguibert6WIND