On Fri, Jul 14, 2017 at 07:12:52PM +0200, Markus Armbruster wrote: > Here's what I propose to do: > > 1. Stop abusing values the schema accepts, but are invalid to mean "do > something else entirely". > > 2. Add a first class null type to QAPI. > > 3. Turn MigrationParameters members tls-creds and tls-hostname into > alternate of str and null. Deprecate "". > > 4. Add a null member to alternate BlockdefRef. Deprecate "". > > I got patches for 2., and I intend to work on 3. and 4. > > Since this is "only" about "less than general and ugly", we may decide > to leave things as they are if my patches turn out even uglier. > > Meanwhile, opinions?
That sounds ok in principle to me, so worth at least proposing the patches so we can see how it works out in practice. Regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|