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

Add remote device for BGP Sessions #178

Open
pumathingy opened this issue Jan 23, 2024 · 6 comments
Open

Add remote device for BGP Sessions #178

pumathingy opened this issue Jan 23, 2024 · 6 comments
Assignees
Labels
enhancement New feature or request will-not-implement A feature that won't be implemented

Comments

@pumathingy
Copy link

Is your feature request related to a problem? Please describe.
Currently, to attach BGP sessions to a device, one would have to have 2 separate sessions defined, one for each end of the BGP session.

Describe the solution you'd like
Updating the device field to local device and adding a new field for remote device to link the BGP session to both devices inside of netbox.

Describe alternatives you've considered
The alternative that I can see is to add 2 unique sessions one for each device. It seems excessive to add 2 sessions when 90% of the information is duplicated between the 2 sessions.

Additional context
The scenario I am modeling is 2 routers who talk BGP to each other. I do not see a use case where BGP is only defined in a single router and not at the remote end.

@pumathingy pumathingy added the enhancement New feature or request label Jan 23, 2024
@pl0xym0r
Copy link
Contributor

Have you tried to add custom fields, especially remote_device --> dcim.devices ? Just a workaround suggestion

@pumathingy
Copy link
Author

That would be a workaround. Also seems easy enough to add directly. It would be very rare you are not attaching 2 devices together with BGP, so including the second device field natively just makes sense to me.

@pl0xym0r
Copy link
Contributor

Well, I'm torn between the network BGP session object itself and a view from the device that have the session object. You could use listen range and from the device perspective, the representation in the plugin works, but you won't list all dynamic sessions into the plugin.
There are too much parameters in BGP (password, redistribute, ecmp, next hop options, multi hop, allow as, bfd ...) that cannot be listed and implemented in the plugin for ease of use reasons (I think) and custom fields helps to add what's missing in each use case.
I don't know if the aim of the plugin is to represent a core object of a BGP session or the view of a device that owns a BGP session.

@cruse1977
Copy link
Member

That would be a workaround. Also seems easy enough to add directly. It would be very rare you are not attaching 2 devices together with BGP, so including the second device field natively just makes sense to me.

Hey, your use case assumes sessions are all on devices you have in NetBox. This wouldn't be the case for service providers whereas you might have IX / Transit peering, NNI peering, L3VPN peering whereas you wouldn't add a 3rd parties devices into NetBox as they're not directly owned/managed/configured etc by you.

@cruse1977 cruse1977 added the will-not-implement A feature that won't be implemented label May 16, 2024
@cruse1977 cruse1977 self-assigned this May 16, 2024
@cruse1977
Copy link
Member

Considering I wouldn't do this in a session, This does however open a discussion about possible documentary objects on things like 'Resiliency Groups' - whereas a customer may have 2 interconnects (and hence sessions) and it may be nice to link these together in some form of representation

@elipsion
Copy link

elipsion commented Aug 5, 2024

That would be a workaround. Also seems easy enough to add directly. It would be very rare you are not attaching 2 devices together with BGP, so including the second device field natively just makes sense to me.

Hey, your use case assumes sessions are all on devices you have in NetBox. This wouldn't be the case for service providers whereas you might have IX / Transit peering, NNI peering, L3VPN peering whereas you wouldn't add a 3rd parties devices into NetBox as they're not directly owned/managed/configured etc by you.

Then let the parties be either a Device or an IP Address, akin to how either endpoint for a Circuit can be a Device or a Provider Network.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request will-not-implement A feature that won't be implemented
Projects
None yet
Development

No branches or pull requests

4 participants