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

Show device information #484

Open
deanfourie1 opened this issue Apr 25, 2024 · 4 comments · May be fixed by #573
Open

Show device information #484

deanfourie1 opened this issue Apr 25, 2024 · 4 comments · May be fixed by #573
Assignees
Labels
status: accepted This issue has been accepted for implementation type: feature Request for new feature or change to existing feature

Comments

@deanfourie1
Copy link

NetBox version

v3.7.4

Topology Views version

3.9.1

Feature type

New functionality

Proposed functionality

The ability to show the IP address underneath the labels on the map

Use case

ability to easily obtain the IP addresses of devices, and also find free ip addresses.

External dependencies

No response

@deanfourie1 deanfourie1 added the type: feature Request for new feature or change to existing feature label Apr 25, 2024
@dreng dreng added the status: under review Further discussion is needed to determine this issue's scope and/or implementation label Apr 25, 2024
@dreng
Copy link
Collaborator

dreng commented Apr 25, 2024

Hi,

NTV displays devices, not Interfaces or IP addresses. We've had some proposals like this before. The problem is that IP addresses (or MAC addresses) don't belong to devices. So, technically speaking, there are no "IP addresses of devices". There are IP addresses of interfaces.

A device can easily have 50 interfaces. Interfaces can have theoretically an unlimited number of IP addresses. NetBox models this situation exactly like it is in the real world.

Imagine a router or L3 switch with a bunch of IP addresses for interfaces and virtual interfaces (for vlans). Which IP address are we supposed to display? Displaying all of them would end in a complete chaos.

@Azmodeszer
Copy link

Azmodeszer commented Apr 27, 2024

NetBox allows the assignment of a single primary IPv4 and/or IPv6 directly to a device, though (and a single OOB IP). Having a toggle for that/those might be handy.

@dreng
Copy link
Collaborator

dreng commented Apr 27, 2024

NetBox allows the assignment of a single primary IPv4 and/or IPv6 directly to a device, though (and a single OOB IP).

You are right. These ones are indeed device propoerties. I havn't had that in mind.

Having a toggle for that/those might be handy.

Why not offer three checkboxes or a multiple choice combobox to allow any combination?

We could possibly extend this to all possible device properties, both in the label and in the popup. That way, every user can set what is important to them.
Unfortunately, you can only format the labels as a whole and not individual parts. However, line breaks are possible. The popup is HTML, so anything goes.

@deanfourie1
Copy link
Author

So you think we can expect this in an upcoming update.

Would be nice if I could print out a diagram like this, which included the ip addresses for reference :)

@dreng dreng added status: accepted This issue has been accepted for implementation and removed status: under review Further discussion is needed to determine this issue's scope and/or implementation labels May 1, 2024
@dreng dreng changed the title Show IP addresses Show device information May 1, 2024
@dreng dreng added the status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation label May 1, 2024
@dreng dreng self-assigned this Sep 28, 2024
@dreng dreng removed the status: needs owner This issue is tentatively accepted pending a volunteer committed to its implementation label Sep 28, 2024
@dreng dreng linked a pull request Sep 30, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: accepted This issue has been accepted for implementation type: feature Request for new feature or change to existing feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants