[RFE] Associate ports and portgroups with volume connectors

Bug #1715529 reported by Hironori Shiina
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ironic
In Progress
Wishlist
Hironori Shiina

Bug Description

This RFE adds new volume connector types, 'port' and 'portgroup'. A volume connector with one of these types has a UUID of a port or portgroup. The types make it easier to get an IP address provided for a volume connector since we can get a VIF attached to the connector from port or portgroup.

For attaching an iSCSI volume to a node, a few cinder backend driver may require IP address of the node. If the storage network is managed by neutron, we can get an IP address from a VIF attached to a port or a porgroup. When a volume connector was designed initially, nova associates a MAC address with a VIF. Then, there is a type 'mac' for volume connector to retrieve VIFs with a MAC address. Now that API for attaching and detaching VIF was introduced, we can get a VIF for a volume connector by associating the volume connector with the port or the portgorup.

Tags: rfe-approved
Revision history for this message
Julia Kreger (juliaashleykreger) wrote :

I don't think this requires a spec, and it seems strait forward enough. Partially in nova, which as I understand it has an open blueprint for, but supporting this internally would be cleaner. I'm for approving this RFE, although Hironori indicated that he would post proof of concept code soon.

Changed in ironic:
importance: Undecided → Wishlist
status: New → Confirmed
status: Confirmed → Triaged
Revision history for this message
Ruby Loo (rloo) wrote :

I agree, I don't think it requires a spec. Am approving it. Thanks Hironori!

tags: added: rfe-approved
removed: rfe
Revision history for this message
Julia Kreger (juliaashleykreger) wrote :

I think we kind of set ourselves up for this with the original bfv effort, so I'm +2 approval without spec, given we made the data storage such to support new things.

Revision history for this message
Ruby Loo (rloo) wrote :

Hironori submitted this patch: https://review.openstack.org/#/c/532442/

Changed in ironic:
assignee: nobody → Hironori Shiina (shiina-hironori)
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/python-ironicclient 2.2.0

This issue was fixed in the openstack/python-ironicclient 2.2.0 release.

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.