netbox-bgp icon indicating copy to clipboard operation
netbox-bgp copied to clipboard

Implement MP-BGP

Open pl0xym0r opened this issue 1 year ago • 5 comments

MP-BGP was initially planned but not implemented

models session: afi_safi = None # for future use

  • choices: class AFISAFIChoices(ChoiceSet):

Describe the solution you'd like Implement MG BGP on the plugin

Describe alternatives you've considered What is done on nautobot https://github.com/nautobot/nautobot-app-bgp-models/blob/develop/nautobot_bgp_models/models.py but not sure it could cover all needs on MP-BGP

@k01ek what was in your mind for this feature at first?

pl0xym0r avatar Mar 17 '24 14:03 pl0xym0r

Hi @pl0xym0r,

I too would like an AFI-SAFI/address families field on session. Can I suggest that the field is also included on peer group.

movedempackets avatar Aug 22 '24 16:08 movedempackets

I agree there needs to be AFI-SAFI support for tracking things like redistribution, network statements, aggregate routes, multipath, add path, etc. Also need support for VRFs.

ajackson79 avatar Sep 30 '24 19:09 ajackson79

@ajackson79 For VRF support, please consider #127

pl0xym0r avatar Oct 01 '24 05:10 pl0xym0r

@ajackson79 For VRF support, please consider #127

You need the vrf for a neighbor, but you also need it for the AFI/SAFI so you can track things like redistribution, network statements, aggregate routes, etc. Just adding vrf support to a neighbor is not enough to document everything.

ajackson79 avatar Oct 01 '24 05:10 ajackson79

I think AFI/SAFI should be just a choice field and it should include in unique_together, it will allow to create session with same src/dst/as/ip but with different afi/safi. As for VRF, my thought is same, we don't need to create new field to link VRF, becasue local IP address has this link.

k01ek avatar Feb 07 '25 06:02 k01ek