]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Correct a few changes in the VMAX driver
authorXing Yang <xing.yang@emc.com>
Fri, 27 Feb 2015 05:45:09 +0000 (00:45 -0500)
committerXing Yang <xing.yang@emc.com>
Fri, 27 Feb 2015 20:25:16 +0000 (15:25 -0500)
The following patch made a few wrong changes in the VMAX driver.
https://review.openstack.org/#/c/145780/

The following two files contain the wrong changes:
https://review.openstack.org/#/c/145780/15/cinder/tests/
test_emc_vmax.py line 604
https://review.openstack.org/#/c/145780/15/cinder/volume/drivers/
emc/emc_vmax_https.py

This patch fixed them.

Change-Id: I3f078ec7915c904ab910eb9ef9f07ac351282489

cinder/tests/test_emc_vmax.py
cinder/volume/drivers/emc/emc_vmax_https.py

index 0f233799c89751fac0c9212075bcf1293f655134..1a9506aed4bb42d64bc17543137a1b557027f55e 100644 (file)
@@ -601,7 +601,7 @@ class FakeEcomConnection():
         result = None
         if ResultClass == 'EMC_StorageHardwareID':
             result = self._assoc_hdwid()
-        elif ResultClass == 'EMC_iSHEADCSIProtocolEndpoint':
+        elif ResultClass == 'EMC_iSCSIProtocolEndpoint':
             result = self._assoc_endpoint()
         elif ResultClass == 'EMC_StorageVolume':
             result = self._assoc_storagevolume(objectpath)
index c44f5912b254599c13a56917cbea47e9ae13df49..7098c3d1abcd2fd59e880d787a7d86b129de27ee 100644 (file)
@@ -80,7 +80,7 @@ class OpenSSLConnectionDelegator(object):
     a delegator must be used.
     """
     def __init__(self, *args, **kwargs):
-        self.connection = SSL.Connection(*args, **kwargs)
+        self.connection = SSL.GreenConnection(*args, **kwargs)
 
     def __getattr__(self, name):
         return getattr(self.connection, name)