[libvirt] [PATCHv4 0/8] glusterfs storage pool

Osier Yang jyang at redhat.com
Mon Nov 25 07:43:47 UTC 2013


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




More information about the libvir-list mailing list