]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commitdiff
Set 'driver_volume_type' to 'gpfs'
authorNilesh Bhosale <nilesh.bhosale@in.ibm.com>
Fri, 16 Jan 2015 10:10:18 +0000 (15:40 +0530)
committerNilesh Bhosale <nilesh.bhosale@in.ibm.com>
Fri, 16 Jan 2015 10:10:18 +0000 (15:40 +0530)
nova LibvirtGPFSVolumeDriver class implemented recently for GPFS volume
driver expects the 'driver_volume_type' as 'gpfs' from the Cinder GPFS
driver. Making the required change in the GPFS cinder driver to
complete the fix.

Change-Id: I880b93e5e3608b63872e6937d02482a3fe368efd
Related-Bug: #1405044

cinder/tests/test_gpfs.py
cinder/volume/drivers/ibm/gpfs.py

index a994fff8ec962cec9d0c0b10f482ebad96631fbb..8f44d99b124baeac5c57b499b5df6015138b167b 100644 (file)
@@ -1028,6 +1028,7 @@ class GPFSDriverTestCase(test.TestCase):
         self.assertEqual(data['data']['name'], 'test')
         self.assertEqual(data['data']['device_path'], os.path.join(
             self.driver.configuration.gpfs_mount_point_base, 'test'))
+        self.assertEqual(data['driver_volume_type'], 'gpfs')
 
     def test_terminate_connection(self):
         self.assertEqual(None, self.driver.terminate_connection('', ''))
index c5732eba4834afce27e17f3396ec7151c389cf3a..2316feaf0a70b8fbb8d369bd8aca913391c580b0 100644 (file)
@@ -657,7 +657,7 @@ class GPFSDriver(driver.VolumeDriver):
 
     def initialize_connection(self, volume, connector):
         return {
-            'driver_volume_type': 'local',
+            'driver_volume_type': 'gpfs',
             'data': {
                 'name': volume['name'],
                 'device_path': self.local_path(volume),