[openstack-dev] [nova][cinder] Refactor ISCSIDriver to support other iSCSI transports besides TCP

Shlomi Sasson shlomis at mellanox.com
Thu Mar 27 16:45:23 UTC 2014

Of course I'm aware of that.. I'm the one who pushed it there in the first place :)
But it was not the best way to handle this.. I think that the right/better approach is as suggested.

I'm planning to remove the existing ISERDriver code, this will eliminate significant code and class duplication, and will work with all the iSCSI vendors who supports both tcp and rdma without the need to modify their plug-in drivers.

From: John Griffith [mailto:john.griffith at solidfire.com]
Sent: Wednesday, March 26, 2014 22:47
To: OpenStack Development Mailing List (not for usage questions)
Subject: Re: [openstack-dev] [nova][cinder] Refactor ISCSIDriver to support other iSCSI transports besides TCP

On Wed, Mar 26, 2014 at 12:18 PM, Eric Harney <eharney at redhat.com<mailto:eharney at redhat.com>> wrote:
On 03/25/2014 11:07 AM, Shlomi Sasson wrote:

> I am not sure what will be the right approach to handle this, I already have the code, should I open a bug or blueprint to track this issue?
> Best Regards,
> Shlomi
A blueprint around this would be appreciated.  I have had similar
thoughts around this myself, that these should be options for the LVM
iSCSI driver rather than different drivers.

These options also mirror how we can choose between tgt/iet/lio in the
LVM driver today.  I've been assuming that RDMA support will be added to
the LIO driver there at some point, and this seems like a nice way to
enable that.

OpenStack-dev mailing list
OpenStack-dev at lists.openstack.org<mailto:OpenStack-dev at lists.openstack.org>

I'm open to improving this, but I am curious you know there's an ISER subclass in iscsi for Cinder currently right?
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20140327/b6fac88e/attachment.html>

More information about the OpenStack-dev mailing list