Re: [PATCH 00/25] Add support for multi-host gluster drives

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

 



On Tue, Jul 26, 2016 at 17:57:32 +0200, Ján Tomko wrote:
> On Mon, Jul 25, 2016 at 08:11:45PM +0200, Peter Krempa wrote:
> >This is a updated take based on stuff I had laying around and parts from
> >https://www.redhat.com/archives/libvir-list/2016-July/msg00872.html
> >
> >This addresses the backing store parser, adds and improves bits to the
> >JSON->commandline generator prior to plugging in the gluster support.
> >
> >This series does not yet address block jobs (snapshot/block copy) using
> >multi-host gluster volumes.
> >
> >Peter Krempa (24):
> >  tests: qemuxml2xml: Avoid crash when processing an XML that fails to
> >    parse
> >  tests: Add testing of backing store string parser
> >  util: storage: Add parser for qemu's "json" backing pseudo-protocol
> >  util: storage: Add support for host device backing specified via JSON
> >  util: storage: Add support for URI based backing volumes in qemu's
> >    JSON pseudo-protocol
> >  util: storage: Add json pseudo protocol support for gluster volumes
> >  util: storage: Add json pseudo protocol support for iSCSI volumes
> >  util: storage: Add JSON backing volume parser for 'nbd' protocol
> >  util: storage: Add JSON backing store parser for 'sheepdog' protocol
> >  util: storage: Add 'ssh' network storage protocol
> >  util: storage: Add JSON backing volume parser for 'ssh' protocol
> >  util: json: Make first argument of virJSONValueObjectForeachKeyValue
> >    const
> >  util: qemu: Add wrapper for JSON -> commandline conversion
> >  util: qemu: Add support for user-passed strings in JSON->commandline
> >  util: qemu: Allow nested objects in JSON -> commandline generator
> >  util: qemu: Allow for different approaches to format JSON arrays
> >  util: qemu: Don't generate any extra commas in
> >    virQEMUBuildCommandLineJSON
> >  qemu: command: Rename qemuBuildNetworkDriveURI to
> >    qemuBuildNetworkDriveStr
> >  qemu: command: Split out network disk URI building
> >  qemu: command: Extract drive source command line formatter
> >  qemu: command: Refactor code extracted to qemuBuildDriveSourceStr
> >  storage: gluster: Support multiple hosts in backend functions
> >  util: qemu: Add support for numbered array members
> >  qemu: command: Add infrastructure for object specified disk sources
> >
> >Prasanna Kumar Kalever (1):
> >  qemu: command: Add support for multi-host gluster disksa

[...]

> ACK series

Thanks for the review. I've pushed this now since the freeze didn't
happen yet along with the updated JSON part as per review.

Peter

--
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]