Re: [PATCHv4 0/8] glusterfs storage pool

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 23/11/13 11:20, Eric Blake wrote:
v3: https://www.redhat.com/archives/libvir-list/2013-November/msg00348.html

Depends on:
https://www.redhat.com/archives/libvir-list/2013-November/msg00955.html

Changes since then, addressing review feedback:
- rebase to other improvements in the meantime
- New patches 4-7
- pool changed to require <name>volume</name> to have no slash,
with subdirectory within a volume selected by <dir path=.../>
which must begin with slash
- documentation improved to match actual testing
- directories, symlinks are handled
- volume owner and timestamps are handled
- volume xml tests added, with several bugs in earlier version
fixed along the way
- compared gluster pool with a netfs pool to ensure both can
see the same level of detail from the same gluster storage

If you think it will help review, ask me to provide an interdiff
from v3 (although I have not done it yet).

Eric Blake (8):
   storage: initial support for linking with libgfapi
   storage: document gluster pool
   storage: implement rudimentary glusterfs pool refresh
   storage: add network-dir as new storage volume type
   storage: improve directory support in gluster pool
   storage: improve allocation stats reported on gluster files
   storage: improve handling of symlinks in gluster
   storage: probe qcow2 volumes in gluster pool


Looked through the whole set, except the version nit pointed out by
Daniel, I didn't see any other problem. So ACK.

/btw, I need to support the gluster pool for domain config after these
patches are pushed.

Regards,
Osier

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]