]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Port NetApp E-Series iSCSI driver to ABC model
authorTom Barron <tpb@dyncloud.net>
Sat, 20 Jun 2015 08:57:54 +0000 (04:57 -0400)
committerTom Barron <tpb@dyncloud.net>
Mon, 6 Jul 2015 15:21:57 +0000 (15:21 +0000)
This commit modifies the NetApp E-Series volume driver to the
new ABC driver model.  The corresponding FibreChannel driver
already uses that model since it was recently implemented and
was written with an appropriate ABC class signature in the first
place.  A corresponding class signature is now used for iSCSI.

This commit also modifies a couple doc strings to match the
corresponding FibreChannel driver.

Change-Id: I624efd6e511f15d34fff1d97a0848c2610185e17
Partially-implements: blueprint abc-driver-update

cinder/volume/drivers/netapp/eseries/iscsi_driver.py

index 3de100b718498a0af2d2d85532fd3ab929ea0910..bd1af9f77fe73e777b9d956d75228f81062d3191 100644 (file)
@@ -15,7 +15,7 @@
 #    License for the specific language governing permissions and limitations
 #    under the License.
 """
-iSCSI driver for NetApp E-series storage systems.
+Volume driver for NetApp E-Series iSCSI storage systems.
 """
 
 from oslo_log import log as logging
@@ -28,7 +28,13 @@ from cinder.volume.drivers.netapp import utils as na_utils
 LOG = logging.getLogger(__name__)
 
 
-class NetAppEseriesISCSIDriver(driver.ISCSIDriver):
+class NetAppEseriesISCSIDriver(driver.BaseVD,
+                               driver.ManageableVD,
+                               driver.ExtendVD,
+                               driver.CloneableVD,
+                               driver.TransferVD,
+                               driver.SnapshotVD):
+    """NetApp E-Series iSCSI volume driver."""
 
     DRIVER_NAME = 'NetApp_iSCSI_ESeries'