On 07/04/2012 03:47 AM, Daniel P. Berrange wrote: > From: "Daniel P. Berrange" <berrange@xxxxxxxxxx> > > The previous check for YAJL would have many undesirable > consequences, the most important being that it caused the > capabilities XML to loose all <guest> elements. There is s/loose/lose/ > no user visible feedback as to what is wrong in this respect, > merely a syslog message. The empty capabilities causes > libvirtd to then throw away all guest XML configs that are > stored. > > This changes the code so that the check for YAJL is only > performed at the time we attempt to spawn a QEMU process > > error: Failed to start domain vm-vnc > error: unsupported configuration: this qemu binary requires libvirt to be compiled with yajl > > Signed-off-by: Daniel P. Berrange <berrange@xxxxxxxxxx> > --- ACK. This code will all be touched again the day we learn how to cache capabilities rather than reprobing them all the time, but as that patch has not been written and this is an improvement in behavior, it is worth pushing this now. -- Eric Blake eblake@xxxxxxxxxx +1-919-301-3266 Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list