Re: [libvirt] [PATCH] storage: Allow multiple hosts for a storage pool

[Daniel Veillard <veillard redhat com>]

On Wed, Apr 25, 2012 at 11:47:17AM +0100, Daniel P. Berrange wrote:
> On Wed, Apr 25, 2012 at 12:43:09PM +0200, Wido den Hollander wrote:
> > The current storage pools for NFS and iSCSI only require one host to
> > connect to. Future storage pools like RBD and Sheepdog will require
> > multiple hosts.
> > 
> > This patch allows multiple source hosts and rewrites the current
> > storage drivers.
> > 
> > Signed-off-by: Wido den Hollander <wido widodh nl>
> > ---
> >  src/conf/storage_conf.c             |   72 +++++++++++++++++++++++-----------
> >  src/conf/storage_conf.h             |    5 +-
> >  src/esx/esx_storage_driver.c        |    6 ++-
> >  src/storage/storage_backend_fs.c    |   34 +++++++++++++---
> >  src/storage/storage_backend_iscsi.c |   36 ++++++++++++++---
> >  src/test/test_driver.c              |    4 +-
> >  6 files changed, 117 insertions(+), 40 deletions(-)
> 
> ACK, looks good to me

 pushed now !

Daniel

-- 
Daniel Veillard      | libxml Gnome XML XSLT toolkit  http://xmlsoft.org/
daniel veillard com  | Rpmfind RPM search engine http://rpmfind.net/
http://veillard.com/ | virtualization library  http://libvirt.org/