[libvirt] [PATCH 2/4] Convert libvirtd over to the new RPC handling APIs
Daniel P. Berrange
berrange at redhat.com
Tue Jun 28 16:24:11 UTC 2011
On Mon, Jun 27, 2011 at 04:01:53PM -0600, Eric Blake wrote:
> On 06/27/2011 08:24 AM, Daniel P. Berrange wrote:
> > This guts the libvirtd daemon, removing all its networking and
> > RPC handling code. Instead it calls out to the new virServerPtr
> > APIs for all its RPC & networking work
> >
> > As a fallout all libvirtd daemon error reporting now takes place
> > via the normal internal error reporting APIs. There is no need
> > to call separate error reporting APIs in RPC code, nor should
> > code use VIR_WARN/VIR_ERROR for reporting fatal problems anymore.
> >
> > * daemon/qemu_dispatch_*.h, daemon/remote_dispatch_*.h: Remove
> > old generated dispatcher code
> > * daemon/qemu_dispatch.h, daemon/remote_dispatch.h: New dispatch
> > code
> > * daemon/dispatch.c, daemon/dispatch.h: Remove obsoleted code
> > * daemon/remote.c, daemon/remote.h: Rewrite for new dispatch
> > APIs
> > * daemon/libvirtd.c, daemon/libvirtd.h: Remove all networking
> > code
> > * daemon/stream.c, daemon/stream.h: Update for new APIs
> > * daemon/Makefile.am: Link to libvirt-net-rpc-server.la
> > +++ b/daemon/Makefile.am
> > @@ -3,33 +3,21 @@
> > CLEANFILES =
> > +++ b/daemon/libvirtd.c
> > @@ -23,31 +23,13 @@
> >
> > #include <grp.h>
> > -#include <signal.h>
> > -#include <netdb.h>
> > -#include <locale.h>
>
> A bit too prune-happy; compilation failed. And even then, I'm getting a
> test failure:
>
> 31) corrupted config audit_logging ... OK
> ./daemon-conf: line 98: kill: (22788) - No such process
> 32) valid config file (sleeping 2 seconds) ... FAILED
> FAIL: daemon-conf
This turned out to be a bogus check I had added to the daemon
making it refuse to startup as non-root if you specified a
unix socket group.
Daniel
--
|: http://berrange.com -o- http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org -o- http://virt-manager.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org -o- http://live.gnome.org/gtk-vnc :|
More information about the libvir-list
mailing list