'NfsDriver' object has no attribute 'manage_existing_get_size'

Bug #1949189 reported by Ilya Popov
22
This bug affects 5 people
Affects Status Importance Assigned to Milestone
Cinder
In Progress
Medium
Andrei Yachmenev

Bug Description

It isn't possible to get volume on external NFS share under cinder management.

When one issue cinder manage command = something like:

cinder manage --id-type source-name --volume-type __DEFAULT__ --name nfs-botvol volumes:volumes@nfs-1#nfs-1 volume-5bae4ec0-ead2-4ca6-b931-1c04f949f554

Error message appears in cinder-volume log:

...
ERROR oslo_messaging.rpc.server File "/usr/lib/python3.6/site-packages/cinder/volume/flows/manager/manage_existing.py", line 55, in execute
ERROR oslo_messaging.rpc.server size = self.driver.manage_existing_get_size(volume,
ERROR oslo_messaging.rpc.server AttributeError: 'NfsDriver' object has no attribute 'manage_existing_get_size'
...

Revision history for this message
Sofia Enriquez (lsofia-enriquez) wrote :

Greetings Ilya Popov,
I did a quick check and it looks like the method is missing in the generic nfs driver.
Are you using the generic nfs driver? if not, do you mind sharing which nfs related backend are you using?
Regards,
Sofia

tags: added: generic-nfs manage nfs
Changed in cinder:
importance: Undecided → Medium
Revision history for this message
Ilya Popov (ilya-p) wrote :

Hello, Sofia !

Thank you for your attention.

I suppose I'm trying to use general NFS driver over openstack Ussuri, basic configuration info available here:

https://docs.openstack.org/cinder/ussuri/admin/blockstorage-nfs-backend.html
https://docs.openstack.org/cinder/ussuri/configuration/block-storage/drivers/nfs-volume-driver.html

In cinder.conf there is smth like:

volume_driver = cinder.volume.drivers.nfs.NfsDriver

in nfs.py NfsDriver are based on remotefs.RemoteFSSnapDriverDistributed, and class remotefs.RemoteFSSnapDriverDistributed doesn't have manage_existing() and manage_existing_get_size().

By the way class RemoteFSManageableVolumesMixin, located at the same remotefs has the neded methods manage_existing() and manage_existing_get_size()

Changed in cinder:
assignee: nobody → Andrei Yachmenev (weanew)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to cinder (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/cinder/+/819089

Changed in cinder:
status: New → In Progress
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.