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

Michal Privoznik posted 2 patches 7 years ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1490713844.git.mprivozn@redhat.com
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(-)
[libvirt] [PATCH 0/2] Slightly rework domain ID handling
Posted by Michal Privoznik 7 years ago
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

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