starred/netbox
1
0
Fork 0
mirror of https://github.com/netbox-community/netbox.git synced 2026-01-11 21:10:29 +01:00
Code Issues 273 Packages Projects Releases 10 Wiki Activity

Support CDP Neighbors #1804

New Issue
Closed
opened 2025-12-29 17:19:16 +01:00 by adam · 6 comments
adam commented 2025-12-29 17:19:16 +01:00
Owner
Copy Link

Originally created by @orgito on GitHub (Jun 21, 2018).

Issue type

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

Environment

  • Python version: 3.4.8
  • NetBox version: 2.3.4

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.

Originally created by @orgito on GitHub (Jun 21, 2018). ### Issue type [x] Feature request <!-- An enhancement of existing functionality --> [ ] Bug report <!-- Unexpected or erroneous behavior --> [ ] Documentation <!-- A modification to the documentation --> ### Environment * Python version: 3.4.8 * NetBox version: 2.3.4 <!-- BUG REPORTS must include: * A list of the steps needed for someone else to reproduce the bug * A description of the expected and observed behavior * Any relevant error messages (screenshots may also help) FEATURE REQUESTS must include: * A detailed description of the proposed functionality * A use case for the new feature * A rough description of any necessary changes to the database schema * Any relevant third-party libraries which would be needed --> ### 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.
adam closed this issue 2025-12-29 17:19:16 +01:00
adam commented 2025-12-29 17:19:17 +01:00
Author
Owner
Copy Link

@orgito commented on GitHub (Jun 21, 2018):

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

@orgito commented on GitHub (Jun 21, 2018): I'd be happy to work on this if it is considered useful.
adam commented 2025-12-29 17:19:17 +01:00
Author
Owner
Copy Link

@dBitech commented on GitHub (Jun 21, 2018):

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
.

@dBitech commented on GitHub (Jun 21, 2018): 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> > . >
adam commented 2025-12-29 17:19:17 +01:00
Author
Owner
Copy Link

@jeremystretch commented on GitHub (Jun 21, 2018):

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.

@jeremystretch commented on GitHub (Jun 21, 2018): NetBox doesn't interact with devices directly; live data such as LLDP neighbors is retrieved using the [NAPALM library](https://github.com/napalm-automation/napalm/), which does not support CDP. It's unlikely that NAPALM will introduce support for CDP as it's an obsolete proprietary protocol.
adam commented 2025-12-29 17:19:17 +01:00
Author
Owner
Copy Link

@orgito commented on GitHub (Jun 21, 2018):

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.

@orgito commented on GitHub (Jun 21, 2018): 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.
adam commented 2025-12-29 17:19:17 +01:00
Author
Owner
Copy Link

@jwhitelan commented on GitHub (Oct 1, 2018):

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

@jwhitelan commented on GitHub (Oct 1, 2018): I am guessing that this idea never took off? I would use it as many of our Cisco core routers do not support LLDP.
adam commented 2025-12-29 17:19:17 +01:00
Author
Owner
Copy Link

@orgito commented on GitHub (Aug 26, 2019):

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.

@orgito commented on GitHub (Aug 26, 2019): 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.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
beta
breaking change
complexity: high
complexity: low
complexity: medium
needs milestone
netbox
pending closure
plugin candidate
pull-request
Mirrored from GitHub Pull Request
 severity: high
severity: low
severity: medium
status: accepted
status: backlog
status: blocked
status: duplicate
status: needs owner
status: needs triage
status: revisions needed
status: under review
topic: GraphQL
topic: Internationalization
topic: OpenAPI
topic: UI/UX
topic: cabling
topic: event rules
topic: htmx navigation
topic: industrialization
topic: migrations
topic: plugins
topic: scripts
topic: templating
topic: testing
type: bug
type: deprecation
type: documentation
type: feature
type: housekeeping
type: translation
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
adam
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/netbox#1804
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?