os-lldpd: Add additional configuration input field #4561
Comments
3 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
{{ message }}
Important notices
Before you add a new report, we ask you kindly to acknowledge the following:
Is your feature request related to a problem? Please describe.
Would it be possible to add a multi-line text input field to the Services -> LLDPs page to input arbitrary lldp configuration commands, one per line?
There are a few lldp commands that I use to make the "show lldp neighbors" output from connected switches a bit easier to interpret. The first two are good candidates for setting as defaults or maybe check boxes. The last would really benefit from the multi-line input field.
This would also be a solution to Add ability to set chassis management ip via LLDP plugin #4418.
Describe the solution you'd like
I would like a multi-line text input field on
/ui/lldpd/general/indexwhich would accept arbitrary lldp configuration commands.Describe alternatives you've considered
This can be done with lldpcli from the shell but I have not found a way to make the changes permanent.
Additional context
The first change I make is "configure lldp portidsubtype ifname" This is needed because the default behavior of lldp is to use the MAC address as port identifier and the interface name as port description, unless the interface has an alias. This is not a problem for most interfaces since you can specify a description on the appropriate page under Interfaces. However, interfaces that are part of a LAG do not have an individual page under Interfaces and so there is no way to set a description. In this case a show lldp neighbors from the switch lists a MAC address for the two interfaces. By issuing "configure lldp portidsubtype ifname" on OPNsense it sets the default to the interface name so now they show up as igb2, etc. There is probably no reason this should not be the default.
Here is a before and after from the switch:
The second is "configure system chassisid OPNsense" which overrides the default of using a MAC address to say OPNsense which is far more useful. There is probabbly no reason this should not also be the default.
Here is a before and after from the switch: (this is with the first change above already completed)
The last one is to add descriptions to the interfaces so that it is obvious from the switch not only the physical ports on the OPNsense router they are connected to but also of which LAG they are members. Since the LAG is visible on the Interfaces -> Assignments page this speeds up troubleshooting.
configure ports igb2,igb3 lldp portdescription lagg0
configure ports igb4,igb5 lldp portdescription lagg1
These would be especially nice to have on the Services -> LLDPd page so they saved with the config and so they are visible when they need to be changed.
Here is the last switch output above after the the change:
The text was updated successfully, but these errors were encountered: