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

Bug #1678134 reported by Mark Goddard
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic Inspector
Mark Goddard

Bug Description

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.

Tags: rfe
Mark Goddard (mgoddard)
Changed in ironic-inspector:
assignee: nobody → Mark Goddard (mgoddard)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ironic-inspector (master)

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

Changed in ironic-inspector:
status: New → In Progress
Vladyslav Drok (vdrok)
tags: added: rfe
Changed in ironic-inspector:
importance: Undecided → Wishlist
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on ironic-inspector (master)

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.

Mark Goddard (mgoddard)
Changed in ironic-inspector:
status: In Progress → Confirmed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.