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

Marcos Paulo de Souza posted 5 patches 1 week ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20180710233102.28186-1-marcos.souza.org@gmail.com
Test syntax-check passed
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(-)

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

Posted by Marcos Paulo de Souza 1 week ago
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

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

Posted by Michal Privoznik 1 week ago
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