[RFE] plugin for setting local link connection switch info from LLDP system name

[Mark Goddard]

The existing local_link_connection inspector plugin parses the mandatory LLDP switch chassis ID and port ID TLVs and uses these to populate the switch_id and port_id fields in the ironic port's local_link_connection attribute.

Some Neutron drivers (including ML2 genericswitch mechanism driver) expect the switch_info field in an ironic port's local_link_connection attribute to contain the system name of a switch. Some Neutron mechanism drivers expect switch_info to contain something other than the system name, so adding this support to the existing local_link_connection plugin would not work in every case.

We propose adding a plugin that will store the system name (TLV type 5) received via LLDP (if present) in the switch_info field of the Ironic port's local_link_connection attribute.

As a secondary proposal, we could refactor the existing GenericLocalLinkConnectionHook plugin into a base class and subclass to allow for reuse of the inventory data processing and ironic port patching logic.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/452228

[OpenStack Infra (hudson-openstack)]

Change abandoned by Mark Goddard (<email address hidden>) on branch: master
Review: https://review.openstack.org/452228
Reason: Abandoning as it was decided to write a spec around this during an ironic meeting: http://eavesdrop.openstack.org/meetings/ironic/2017/ironic.2017-04-03-17.00.log.html#l-250.