Re: [libvirt] [PATCH v2 3/9] admin: Introduce virAdmConnectIsAlive

2015-11-04 Thread Martin Kletzander
On Wed, Nov 04, 2015 at 01:56:40PM +0100, Erik Skultety wrote: +int +virAdmConnectIsAlive(virAdmConnectPtr conn) +{ +bool ret; +remoteAdminPrivPtr priv = conn->privateData; + +VIR_DEBUG("conn=%p", conn); + +virResetLastError(); + +virObjectLock(priv); +ret = virNetClientIs

Re: [libvirt] [PATCH v2 3/9] admin: Introduce virAdmConnectIsAlive

2015-11-04 Thread Erik Skultety
>> +int >> +virAdmConnectIsAlive(virAdmConnectPtr conn) >> +{ >> +bool ret; >> +remoteAdminPrivPtr priv = conn->privateData; >> + >> +VIR_DEBUG("conn=%p", conn); >> + >> +virResetLastError(); >> + >> +virObjectLock(priv); >> +ret = virNetClientIsOpen(priv->client); >> +v

Re: [libvirt] [PATCH v2 3/9] admin: Introduce virAdmConnectIsAlive

2015-11-03 Thread Martin Kletzander
On Fri, Oct 16, 2015 at 08:12:20PM +0200, Erik Skultety wrote: Since most of our APIs rely on an acive functional connection to a daemon and we have such a mechanism in libvirt already, there's need to have such a way in libvirt-admin as well. By introducing a new public API, this patch provides

[libvirt] [PATCH v2 3/9] admin: Introduce virAdmConnectIsAlive

2015-10-16 Thread Erik Skultety
Since most of our APIs rely on an acive functional connection to a daemon and we have such a mechanism in libvirt already, there's need to have such a way in libvirt-admin as well. By introducing a new public API, this patch provides support to check for an active connection. --- include/libvirt/l