Re: [libvirt] [PATCH 36/41] remote: open secondary drivers via remote driver if needed

2019-07-29 Thread Andrea Bolognani
On Mon, 2019-07-29 at 14:46 +0100, Daniel P. Berrangé wrote: > On Mon, Jul 29, 2019 at 10:33:08AM +0200, Andrea Bolognani wrote: > > On Tue, 2019-07-23 at 17:03 +0100, Daniel P. Berrangé wrote: > > > +VIR_DEBUG("Pointing secondary drivers to primary"); > > > +priv->interfaceConn =

Re: [libvirt] [PATCH 36/41] remote: open secondary drivers via remote driver if needed

2019-07-29 Thread Daniel P . Berrangé
On Mon, Jul 29, 2019 at 10:33:08AM +0200, Andrea Bolognani wrote: > On Tue, 2019-07-23 at 17:03 +0100, Daniel P. Berrangé wrote: > [...] > > +++ b/src/remote/remote_daemon_dispatch.c > > @@ -1954,18 +1954,35 @@ remoteGetHypervisorConn(virNetServerClientPtr > > client) > > } > > > > > >

Re: [libvirt] [PATCH 36/41] remote: open secondary drivers via remote driver if needed

2019-07-29 Thread Andrea Bolognani
On Tue, 2019-07-23 at 17:03 +0100, Daniel P. Berrangé wrote: [...] > +++ b/src/remote/remote_daemon_dispatch.c > @@ -1954,18 +1954,35 @@ remoteGetHypervisorConn(virNetServerClientPtr client) > } > > > +static virConnectPtr > +remoteGetSecondaryConn(bool readonly, virConnectPtr *conn, const

Re: [libvirt] [PATCH 36/41] remote: open secondary drivers via remote driver if needed

2019-07-25 Thread Daniel P . Berrangé
On Tue, Jul 23, 2019 at 05:03:14PM +0100, Daniel P. Berrangé wrote: > When the client has a connection to one of the hypervisor specific > daemons (eg virtqemud), the app may still expect to use the secondary > driver APIs (storage, network, etc). None of these will be registered in > the

[libvirt] [PATCH 36/41] remote: open secondary drivers via remote driver if needed

2019-07-23 Thread Daniel P . Berrangé
When the client has a connection to one of the hypervisor specific daemons (eg virtqemud), the app may still expect to use the secondary driver APIs (storage, network, etc). None of these will be registered in the hypervisor daemon, so we must explicitly open a connection to each of the daemons