Re: [libvirt] [PATCHv5 19/19] qemu: Add support for networked disks for block pull/block rebase

On 06/19/2014 07:59 AM, Peter Krempa wrote: Same caveats as in 18/19 about not necessarily working in mixed-source chains (for that, we'd need to use node-names); but as it is definitely more powerful than what libvirt previously supported, it's still worth including under the incremental improvement umbrella. Trailing space in the error message. This treats relative name with no base as a hard error, which is okay but should be documented. No trailing '.'. Once again, back to the question of whether it is nicer for the flag to be advisory (best effort to use relative, but absolute fallback is okay) or mandatory (fail if the request cannot be honored). At this point, I'm leaning towards mandatory (it's easier to relax mandatory to advisory later than it is to give advisory now and tighten it up later; and I like to know if my explicit request cannot be honored). But the documentation needs to match what we choose, and it would help to have Adam's insight as a client of this flag. -- Eric Blake eblake redhat com +1-919-301-3266 Libvirt virtualization library http://libvirt.org