Re: [PATCH] systemd: add debug message when virDBusCallMethod failed

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Sep 05, 2013 at 02:21:53PM +0200, Peter Krempa wrote:
> On 09/05/13 13:03, Daniel P. Berrange wrote:
> > On Wed, Sep 04, 2013 at 10:58:23AM +0800, Gao feng wrote:
> >> Generate debug message when systemd doesn't support
> >> interface org.freedesktop.machine1.Manager.
> >>
> >> Signed-off-by: Gao feng <gaofeng@xxxxxxxxxxxxxx>
> >> ---
> >>  src/util/virsystemd.c | 1 +
> >>  1 file changed, 1 insertion(+)
> >>
> >> diff --git a/src/util/virsystemd.c b/src/util/virsystemd.c
> >> index 3e69ef6..c631557 100644
> >> --- a/src/util/virsystemd.c
> >> +++ b/src/util/virsystemd.c
> >> @@ -233,6 +233,7 @@ int virSystemdCreateMachine(const char *name,
> >>          if (err->code == VIR_ERR_DBUS_SERVICE &&
> >>              STREQ(err->str2, "org.freedesktop.DBus.Error.ServiceUnknown")) {
> >>              virResetLastError();
> >> +            VIR_DEBUG("systemd doesn't support interface org.freedesktop.machine1.Manager");
> > 
> > Is this really needed ?  The virRaiseError call will have already reported
> > this scenario surely ?
> 
> Yes, it will. It is actually spamming the log with "error" messages
> every time you start a VM on a non-systemd host even when it is later
> successful using the old approach.

We likely need to invoke  org.freedesktop.DBus.ListActivatableNames
and/or org.freedesktop.DBus.NameHasOwner to find out if systemd is
available without triggering an error.

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 :|

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




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]