cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Niels de Vos" <nde...@redhat.com>
Subject Re: Review Request 15932: Add support for Primary Storage on Gluster using the libvirt backend
Date Tue, 14 Jan 2014 15:54:54 GMT

-----------------------------------------------------------
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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message