[libvirt] [PATCH 0/2] Slightly rework domain ID handling

Michal Privoznik mprivozn at redhat.com
Tue Mar 28 15:15:06 UTC 2017


The first patch fixes an actual bug: our virConnectGetAllDomainStats API does
not fill out the domain ID, therefore users just see domain name and UUID.

And while fixing it I realized, that our virGetDomain function can do better.

However, given that we are in the freeze, I'd like to push just the first patch
and save the second for after the release as it touches a lot of places and
thus has a great potential o breaking things. Although, in my testing it
actually fixed some other places that we forgot to set domain ID.

Michal Privoznik (2):
  qemuDomainGetStats: Copy domain ID too
  virGetDomain: Set domain ID too

 daemon/remote.c             |  5 +----
 src/bhyve/bhyve_driver.c    | 20 +++++-------------
 src/conf/domain_event.c     | 11 +++++-----
 src/conf/virdomainobjlist.c | 10 +++------
 src/datatypes.c             |  8 ++++++--
 src/datatypes.h             |  3 ++-
 src/esx/esx_driver.c        | 49 ++++++++++++++-------------------------------
 src/hyperv/hyperv_wmi.c     | 15 +++++---------
 src/libxl/libxl_driver.c    | 20 +++++-------------
 src/libxl/libxl_migration.c |  2 +-
 src/lxc/lxc_driver.c        | 20 +++++-------------
 src/openvz/openvz_driver.c  | 24 ++++++----------------
 src/phyp/phyp_driver.c      | 12 +++--------
 src/qemu/qemu_driver.c      | 23 ++++++++-------------
 src/qemu/qemu_migration.c   |  4 ++--
 src/remote/remote_driver.c  |  5 +----
 src/test/test_driver.c      | 20 +++++-------------
 src/uml/uml_driver.c        | 15 +++++---------
 src/vbox/vbox_common.c      | 44 +++++++++++++++-------------------------
 src/vmware/vmware_driver.c  | 20 +++++-------------
 src/vz/vz_driver.c          | 22 ++++++--------------
 src/xen/xen_driver.c        | 28 ++++++--------------------
 src/xenapi/xenapi_driver.c  | 30 +++++++++++----------------
 23 files changed, 129 insertions(+), 281 deletions(-)

-- 
2.10.2




More information about the libvir-list mailing list