-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15932/
-----------------------------------------------------------

(Updated Jan. 14, 2014, 4:54 p.m.)


Review request for cloudstack.


Changes
-------

Addressed all/most of the previous comments.

This change also extend the previous version of the patch, and adds support for 
QEMU+libgfapi native communication. Mounting the the Gluster volume is only 
used to perform management tasks on the storage pool (create, delete disks, 
etc.).

Comments much appreciated, thanks,
Niels


Repository: cloudstack-git


Description
-------

The support for Gluster as Primary Storage is mostly based on the
implementation for NFS. Like NFS, libvirt can address a Gluster environment
through the 'netfs' pool-type.


Diffs (updated)
-----

  api/src/com/cloud/storage/Storage.java 07b6667 
  
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java
 182cb22 
  
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtStoragePoolDef.java
 e181cea 
  
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtStoragePoolXMLParser.java
 a707a0b 
  
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java 
6aaabc5 
  
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/KVMStorageProcessor.java
 aaefc16 
  
plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/LibvirtStorageAdaptor.java
 0760e51 
  
plugins/storage/volume/default/src/org/apache/cloudstack/storage/datastore/lifecycle/CloudStackPrimaryDataStoreLifeCycleImpl.java
 7555c1e 

Diff: https://reviews.apache.org/r/15932/diff/


Testing
-------

See http://blog.nixpanic.net/2013/12/using-gluster-as-primary-storage-in.html


Thanks,

Niels de Vos

Reply via email to