Re: [libvirt] [PATCH v3 0/5] Making libvirt aware of a server need

2018-07-11 Thread Michal Privoznik
On 07/11/2018 01:30 AM, Marcos Paulo de Souza wrote:
> Hi guys,
> 
> this is the third version of the patchset to add a new member to
> virConnectDriver in order to simplfy the check for a server. Now, libvirt does
> the work inside virConnectOpenInternal after getting the correct driver from 
> the
> URI.
> 
> v2: https://www.redhat.com/archives/libvir-list/2018-July/msg00501.html
> v1: https://www.redhat.com/archives/libvir-list/2018-July/msg00393.html
> 
> Changes from v2:
> * patch 2: Move check for remoteOnly member just before calling connectOpen
> * (only patch 2 changed in v2)
> 
> Changes from v1:
> * Adapted the code following ideas from Matthias Bolte
> 
> Marcos Paulo de Souza (5):
>   driver.h: Add remoteOnly member to virConnectDriver struct
>   libvirt.c: Return error when remoteOnly is set but server is empty
>   esx_driver: Set remoteOnly member of virConnectDriver
>   hyperv_driver: Set remoteOnly member of virConnectDriver
>   phyp_driver: Set remoteOnly member of virConnectDriver
> 
>  src/driver.h   | 2 ++
>  src/esx/esx_driver.c   | 8 +---
>  src/hyperv/hyperv_driver.c | 8 +---
>  src/libvirt.c  | 7 +++
>  src/phyp/phyp_driver.c | 7 +--
>  5 files changed, 12 insertions(+), 20 deletions(-)
> 

ACKed and pushed.

Michal

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


[libvirt] [PATCH v3 0/5] Making libvirt aware of a server need

2018-07-10 Thread Marcos Paulo de Souza
Hi guys,

this is the third version of the patchset to add a new member to
virConnectDriver in order to simplfy the check for a server. Now, libvirt does
the work inside virConnectOpenInternal after getting the correct driver from the
URI.

v2: https://www.redhat.com/archives/libvir-list/2018-July/msg00501.html
v1: https://www.redhat.com/archives/libvir-list/2018-July/msg00393.html

Changes from v2:
* patch 2: Move check for remoteOnly member just before calling connectOpen
* (only patch 2 changed in v2)

Changes from v1:
* Adapted the code following ideas from Matthias Bolte

Marcos Paulo de Souza (5):
  driver.h: Add remoteOnly member to virConnectDriver struct
  libvirt.c: Return error when remoteOnly is set but server is empty
  esx_driver: Set remoteOnly member of virConnectDriver
  hyperv_driver: Set remoteOnly member of virConnectDriver
  phyp_driver: Set remoteOnly member of virConnectDriver

 src/driver.h   | 2 ++
 src/esx/esx_driver.c   | 8 +---
 src/hyperv/hyperv_driver.c | 8 +---
 src/libvirt.c  | 7 +++
 src/phyp/phyp_driver.c | 7 +--
 5 files changed, 12 insertions(+), 20 deletions(-)

-- 
2.17.1

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