Re: Review Request 15933: Add Gluster to the list of protocols in the Management Server

2014-12-04 Thread Niels de Vos
4.2). - Niels --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15933/#review63663 --- On Feb. 19, 2014, 9:27 a.m., Nie

Re: Review Request 18412: Gluster should store disk images in qcow2 format

2014-02-23 Thread Niels de Vos
tVMDef.java 290c5a9 plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/storage/KVMStorageProcessor.java 1c37607 Diff: https://reviews.apache.org/r/18412/diff/ Testing (updated) --- Test results described with this setup: http://blog.nixpanic.net/2014_02_23_archive.html Thanks, Niels de Vos

Re: Review Request 15932: Add support for Primary Storage on Gluster using the libvirt backend

2014-02-23 Thread Niels de Vos
st merged it into master and pushed. > > So gluster is in master right now! Niels, can I ask you to test it all > again? Just to make sure the code all works like you intended. > > Niels de Vos wrote: > Thanks Wido! > This seems to be working OK for me. Note that th

Review Request 18412: Gluster should store disk images in qcow2 format

2014-02-23 Thread Niels de Vos
rs/kvm/src/com/cloud/hypervisor/kvm/storage/KVMStorageProcessor.java 1c37607 Diff: https://reviews.apache.org/r/18412/diff/ Testing --- Thanks, Niels de Vos

Re: Review Request 15932: Add support for Primary Storage on Gluster using the libvirt backend

2014-02-22 Thread Niels de Vos
utions contain backports that add sufficient functionality too. - Niels --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15932/#review34859 ----------- On Feb. 19, 2014, 9:24 a.m., Niels de Vos wrot

Re: Review Request 15933: Add Gluster to the list of protocols in the Management Server

2014-02-19 Thread Niels de Vos
fd5705b Diff: https://reviews.apache.org/r/15933/diff/ Testing --- Some screenshots and verification: - http://blog.nixpanic.net/2013/12/using-gluster-as-primary-storage-in.html Thanks, Niels de Vos

Re: Review Request 15933: Add Gluster to the list of protocols in the Management Server

2014-02-19 Thread Niels de Vos
fd5705b Diff: https://reviews.apache.org/r/15933/diff/ Testing --- Some screenshots and verification: - http://blog.nixpanic.net/2013/12/using-gluster-as-primary-storage-in.html Thanks, Niels de Vos

Re: Review Request 15933: Add Gluster to the list of protocols in the Management Server

2014-02-19 Thread Niels de Vos
2a15967 ui/scripts/system.js 8159124 ui/scripts/zoneWizard.js fd5705b Diff: https://reviews.apache.org/r/15933/diff/ Testing --- Some screenshots and verification: - http://blog.nixpanic.net/2013/12/using-gluster-as-primary-storage-in.html Thanks, Niels de Vos

Re: Review Request 15932: Add support for Primary Storage on Gluster using the libvirt backend

2014-02-19 Thread Niels de Vos
- See http://blog.nixpanic.net/2013/12/using-gluster-as-primary-storage-in.html Thanks, Niels de Vos

Re: Review Request 15932: Add support for Primary Storage on Gluster using the libvirt backend

2014-02-07 Thread Niels de Vos
ly generated e-mail. To reply, visit: https://reviews.apache.org/r/15932/#review33922 ----------- On Jan. 14, 2014, 4:54 p.m., Niels de Vos wrote: > > --- > Th

Re: Review Request 15932: Add support for Primary Storage on Gluster using the libvirt backend

2014-01-14 Thread Niels de Vos
3/12/using-gluster-as-primary-storage-in.html Thanks, Niels de Vos

Re: Review Request 15932: Add support for Primary Storage on Gluster using the libvirt backend

2014-01-14 Thread Niels de Vos
> On Dec. 4, 2013, 5:17 p.m., John Burwell wrote: > > > > Niels de Vos wrote: > Thanks John! I'll have a go at impoving the patch and will update it > (hopefully) soon. > > Amogh Vasekar wrote: > Reminder - > Hi, > This review has be

Re: Review Request 15932: Add support for Primary Storage on Gluster using the libvirt backend

2013-12-05 Thread Niels de Vos
, visit: https://reviews.apache.org/r/15932/#review29591 --- On Dec. 1, 2013, 3:07 p.m., Niels de Vos wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://

Review Request 15932: Add support for Primary Storage on Gluster using the libvirt backend

2013-12-01 Thread Niels de Vos
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

Review Request 15933: Add Gluster to the list of protocols in the Management Server

2013-12-01 Thread Niels de Vos
pache.org/r/15933/diff/ Testing --- Some screenshots and verification: - http://blog.nixpanic.net/2013/12/using-gluster-as-primary-storage-in.html Thanks, Niels de Vos