GlusterFS drivers code cleanup

Registered by Csaba Henk

The following items needs to be cleaned up (without impacting behavior):

- common: gluster option query should be done directly with 'vol get' instead of parsing 'vol info' output
- common: gluster option query should have a boolean valued variant (coercing "on", "yes", etc. to True, "off", "no", etc. to False)
- common: gluster_call: high level error handling should be included in method
- common: vol set wrapper needed with optional fault tolerance
- common: GlusterManager instantiation needs proper data validation (cf. https://bugs.launchpad.net/manila/+bug/1496733)
- volume layout: fetch the list of volumes and their option with 'vol info' instead of separate 'vol list' + getting options
- glusterfs: _setup_via_manager's conditional actions branching on helper type should be transformed to helper methods
- glusterfs-native: move module under glusterfs

Blueprint information

Status:
Complete
Approver:
Ben Swartzlander
Priority:
Low
Drafter:
Csaba Henk
Direction:
Approved
Assignee:
Csaba Henk
Definition:
Approved
Series goal:
Accepted for mitaka
Implementation:
Implemented
Milestone target:
milestone icon mitaka-3
Started by
Ben Swartzlander
Completed by
Clinton Knight

Related branches

Sprints

Whiteboard

Gerrit topic: https://review.openstack.org/#q,topic:bp/gluster-code-cleanup,n,z

Addressed by: https://review.openstack.org/261783
    gluster*: refactor gluster_call

Addressed by: https://review.openstack.org/271303
    gluster*: add proper getter/setters for volume options

Gerrit topic: https://review.openstack.org/#q,topic:gluster-code-cleanup,n,z

Addressed by: https://review.openstack.org/277141
    gluster*: clean up volume option querying

Addressed by: https://review.openstack.org/280935
    glusterfs_native: relocate module under glusterfs

Addressed by: https://review.openstack.org/282069
    glusterfs.common: move the numreduct function to toplevel

Gerrit topic: https://review.openstack.org/#q,topic:bp/smart-glusterfs-management-with-heketi,n,z

(?)

Work Items

This blueprint contains Public information 
Everyone can see this information.

Subscribers

No subscribers.