LVMVolumeDriver is missing *_export functions
Bug #1293978 reported by
glance
This bug affects 3 people
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Cinder |
Fix Released
|
Medium
|
Unassigned |
Bug Description
I run a testenviorment of openstack on just one machine, and thus like to use the LVMVolumeDriver to minimize overhead and still use cinder.
The LVMVolumeDriver is missing *_export functions and thus fails on NotImplementedError for some operations.
I currently just hacked lvm.py and added:
"""
def ensure_export(self, context, volume):
pass
def create_export(self, context, volume):
pass
def remove_export(self, context, volume):
pass
"""
And now things work.
Changed in cinder: | |
status: | New → Confirmed |
importance: | Undecided → Medium |
tags: | added: drivers lvm |
Changed in cinder: | |
milestone: | none → liberty-1 |
status: | Fix Committed → Fix Released |
Changed in cinder: | |
milestone: | liberty-1 → 7.0.0 |
To post a comment you must log in.
As noted by pylint:
************* Module cinder. volume. drivers. lvm connection' is abstract in class 'VolumeDriver' but is not overridden (abstract-method) connection' is abstract in class 'VolumeDriver' but is not overridden (abstract-method)
W: 58, 0: Method 'initialize_
W: 58, 0: Method 'ensure_export' is abstract in class 'VolumeDriver' but is not overridden (abstract-method)
W: 58, 0: Method 'remove_export' is abstract in class 'VolumeDriver' but is not overridden (abstract-method)
W: 58, 0: Method 'create_export' is abstract in class 'VolumeDriver' but is not overridden (abstract-method)
W: 58, 0: Method 'terminate_
Interesting use case as I haven't heard of anyone using LVMVolumeDriver like this directly, but it seems like a simple enough thing to fix up and may be handy for testing.