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