search

netbox-community / netbox

The premier source of truth powering network automation. Open source under Apache 2. Public demo: https://demo.netbox.dev
http://netboxlabs.com/oss/netbox/
Apache License 2.0
15.44k stars 2.52k forks source link

Support CDP Neighbors #2187

Closed orgito closed 6 years ago

orgito commented 6 years ago

Issue type

[x] Feature request [ ] Bug report [ ] Documentation

Environment

Description

Add the option to list CDP neighbors beside LLDP neighbors.

The use case for that are networks that do not have LLDP enable. We have some legacy Cisco based network where enabling LLDP is not an option.

No need for extra libraries as napalm can be used for that.

orgito commented 6 years ago

I'd be happy to work on this if it is considered useful.

dBitech commented 6 years ago

I would be supportive of this feature as I too have legacy CDP

On Thu, Jun 21, 2018 at 6:52 AM Renato Orgito notifications@github.com wrote:

I'd be happy to work on this if it is considered useful.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/digitalocean/netbox/issues/2187#issuecomment-399110618, or mute the thread https://github.com/notifications/unsubscribe-auth/AJ1FTuhmOzfTlIffKLF69GxNke2LJy5yks5t-6TggaJpZM4UyFRy .

jeremystretch commented 6 years ago

NetBox doesn't interact with devices directly; live data such as LLDP neighbors is retrieved using the NAPALM library, which does not support CDP. It's unlikely that NAPALM will introduce support for CDP as it's an obsolete proprietary protocol.

orgito commented 6 years ago

Napalm allows running arbitrary code. This can be solved by running show cdp neighbors against the device and parsing the output. CDP is a proprietary an obsolete protocol but unfortunately still being used in the real world. I added support for CDP to the ios_interface ansible module and would like to work on adding it to NetBox, if there is a chance of the PR being accepted.

jwhitelan commented 5 years ago

I am guessing that this idea never took off? I would use it as many of our Cisco core routers do not support LLDP.

orgito commented 4 years ago

Apparently, the solution is to get back to https://github.com/napalm-automation/napalm/pull/601

If the functionality is available in napalm it would be simple to add it to netbox.