]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Brick connector revised fix for NFS drivers
authorNavneet Singh <singn@netapp.com>
Sun, 20 Oct 2013 13:00:36 +0000 (18:30 +0530)
committerNavneet Singh <singn@netapp.com>
Wed, 23 Oct 2013 00:55:13 +0000 (06:25 +0530)
commit1cd362630655798e5ff4c058b3f44862ae61c801
treec6f44df16df03201877668b7addcac34d44f5639
parent300ce61120ce179d1d0e4ffe8aa0bd4ceaddba6b
Brick connector revised fix for NFS drivers

This change fixes the error that comes
while getting brick connector and attaching volumes
in case of NFS drivers in cinder. The attribute for
mount point base was not passed to attach_volume method
as the connector initialization logic is common for all
types of protocols. It is fixed by populating
the required parameter in the RemoteFsConnector
by extracting it from connection properties. The infrastructure
for fixing this situation for remote fs drivers is implemented
in this fix. The drivers need to override the method
_get_mount_point_base in their implementation. NFS driver
implements it. GlusterFs implementation of the method to follow
in a separate patch. Hence marking it partial fix.

Change-Id: Id61a437fca1870529fa8b85f7fc2f73eae665294
Partial-Bug: #1238085
cinder/brick/initiator/connector.py
cinder/tests/test_coraid.py
cinder/utils.py
cinder/volume/driver.py
cinder/volume/drivers/nfs.py