On Fri, Nov 14, 2008 at 05:51:22PM +0000, Daniel P. Berrange wrote:
> This patch adds support for the DEFINED and UNDEFINED events in the
> QEMU drive. This was more involved than I expected, because when the
> daemon gets SIGHUP it re-loads config files and we need to broadcast
> events for each new config file it finds. So I had to add a callback
> to the internal virDomainLoadAllConfigs method.

  okay,

> The LoadConfig method also had a bogus line of code resetting the
> domain state to SHUTOFF, which made all your active VMs suddenlly
> appear inactive, after the daemon got SIGHUP !

 whoops :-)


  Looks fine, +1

Daniel

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

--
Libvir-list mailing list
Libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to