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

Feature Request: Deprecate ASN model in favor of NetBox 3.1.0 ASN model

Open ryanmerolle opened this issue 3 years ago • 1 comments

Is your feature request related to a problem? Please describe. NetBox 3.1.0 added the new ASN model allowing you to specify multiple sites to an ASN. It would be ideal to leverage said model, and extend it in the same way you have within this BGP plugin.

Describe the solution you'd like Deprecate the custom plugin model for ASNs and leverage the core NetBox model. The only difference between the two is that the core model allows for:

  • multiple sites per ASN
  • tenant group assignments for ASNs
  • An RIR association

IE that means no current attribute of the netbox-plugin model would be missing.

Sessions would just be linked to the NetBox core model of ASN.

Describe alternatives you've considered Keep the two separate and continue confusion within the API and UI.

Additional context N/A

ryanmerolle avatar Feb 19 '22 04:02 ryanmerolle

There's also a conflict between the native ASNTable and the plugins' ASNTable interface configuration settings. The thing is that a native one has the 'asn' field that is representing the number of AS, while the plugin has the 'number' field for that purpose. Each time I'm changing the table display setting for each table it rewrites the settings for another one.

image image

dteknet avatar Apr 26 '22 09:04 dteknet

Hi @k01ek first of all cool work! Im currently thinking on using your plugin too but i was wondering if i should wait till this change is implemented. Do you have any eta?

peterbaumert avatar Sep 09 '22 14:09 peterbaumert

Hi @peterbaumert ! Thanks for interest! I am gonna to make it this week.

k01ek avatar Sep 13 '22 11:09 k01ek

Nice will try first thing monday :)

peterbaumert avatar Sep 17 '22 09:09 peterbaumert