]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Update SolidFire to use target driver model
authorJohn Griffith <john.griffith8@gmail.com>
Thu, 11 Jun 2015 14:52:38 +0000 (08:52 -0600)
committerJohn Griffith <john.griffith8@gmail.com>
Mon, 15 Jun 2015 02:33:16 +0000 (20:33 -0600)
commit857f47592033d75e7e433bdcf2a07f88f5af0411
tree23a3db71cfce4769797eab8bdaeb69d2a37947e3
parent27b6169869088608839556425de75e8281cb8eff
Update SolidFire to use target driver model

A while back we broke the inheritance chain of transport and backend
by moving the target drivers outside of the drivers.

This means you can instantiate and use any target (data transport)
method that your device supports without requiring a separate
driver (again, if your device can support it).  This was
particularly important for the LVM ref driver and the 5 iSCSI
options that go with it.

This patch converts the SolidFire driver to use the same model.  For
the most part it's pretty simple:
* Create an internal class for the target driver
* Instantiate the target driver that you want to use
* wrap the old transport calls that come to the driver to call
  the new target driver objects

This exposed some issues with the abc work that was being done.
There are a number of items that are marked as required but are
really only necessary for LVM based drivers.  So this also
creates a san/3'rd party iSCSI subclass as well that has the
more appropriate abc settings for such devices.

Change-Id: I02ceaac58d0e4176ccad423763ae0101cfb8446b
cinder/volume/drivers/solidfire.py
cinder/volume/targets/iscsi.py