ch: A couple of cleanups

[PATCH 0/4] ch: A couple of cleanups

Michal Privoznik posted 4 patches 1 month, 2 weeks ago

Download series mbox

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1709910136.git.mprivozn@redhat.com

src/bhyve/bhyve_domain.c     |  3 ++-
src/ch/ch_domain.c           |  5 ++++-
src/ch/ch_driver.c           |  9 ++++-----
src/ch/ch_process.c          |  4 ++--
src/conf/capabilities.c      | 23 +++++++++++++++++------
src/conf/capabilities.h      |  6 ++++--
src/conf/domain_conf.c       |  5 ++++-
src/libxl/libxl_domain.c     |  3 ++-
src/libxl/xen_common.c       |  5 ++++-
src/lxc/lxc_domain.c         |  3 ++-
src/openvz/openvz_conf.c     |  3 ++-
src/qemu/qemu_capabilities.c | 10 +++++-----
src/vmware/vmware_driver.c   |  3 ++-
src/vmx/vmx.c                |  3 ++-
src/vz/vz_driver.c           |  3 ++-
tests/chxml2xmlmock.c        | 33 +++++++++++++++++++++++++++++++++
tests/chxml2xmltest.c        |  2 +-
tests/meson.build            |  3 +++
18 files changed, 95 insertions(+), 31 deletions(-)
create mode 100644 tests/chxml2xmlmock.c

Expand all Fold all

[PATCH 0/4] ch: A couple of cleanups

Posted by Michal Privoznik 1 month, 2 weeks ago

As I rewrote and merged [1] I've broken the build because not
chxml2xmltest relies on host having /dev/kvm accessible. Here are some
fixes and cleanups I've come up with. And here [2] is the green
pipeline (not sure about the x86_64-fedora-rawhide-clang failure,
investigating, but seems unrelated).

1: https://lists.libvirt.org/archives/list/devel@lists.libvirt.org/thread/OEP6HH4DACM47GL7BHS2APPYKH7PZESK/
2: https://gitlab.com/MichalPrivoznik/libvirt/-/pipelines/1206477954

Michal Prívozník (4):
  tests: Introduce chxml2xmlmock
  ch: Demote error when CH driver fails to init
  capabilities: Allow suppressing error message from
    virCapabilitiesDomainDataLookup()
  capabilities: Allow suppressing error message from
    virCapabilitiesDomainSupported()

 src/bhyve/bhyve_domain.c     |  3 ++-
 src/ch/ch_domain.c           |  5 ++++-
 src/ch/ch_driver.c           |  9 ++++-----
 src/ch/ch_process.c          |  4 ++--
 src/conf/capabilities.c      | 23 +++++++++++++++++------
 src/conf/capabilities.h      |  6 ++++--
 src/conf/domain_conf.c       |  5 ++++-
 src/libxl/libxl_domain.c     |  3 ++-
 src/libxl/xen_common.c       |  5 ++++-
 src/lxc/lxc_domain.c         |  3 ++-
 src/openvz/openvz_conf.c     |  3 ++-
 src/qemu/qemu_capabilities.c | 10 +++++-----
 src/vmware/vmware_driver.c   |  3 ++-
 src/vmx/vmx.c                |  3 ++-
 src/vz/vz_driver.c           |  3 ++-
 tests/chxml2xmlmock.c        | 33 +++++++++++++++++++++++++++++++++
 tests/chxml2xmltest.c        |  2 +-
 tests/meson.build            |  3 +++
 18 files changed, 95 insertions(+), 31 deletions(-)
 create mode 100644 tests/chxml2xmlmock.c

-- 
2.43.0
_______________________________________________
Devel mailing list -- devel@lists.libvirt.org
To unsubscribe send an email to devel-leave@lists.libvirt.org

Re: [PATCH 0/4] ch: A couple of cleanups

Posted by Peter Krempa 1 month, 2 weeks ago

On Fri, Mar 08, 2024 at 16:15:22 +0100, Michal Privoznik wrote:
> As I rewrote and merged [1] I've broken the build because not
> chxml2xmltest relies on host having /dev/kvm accessible. Here are some
> fixes and cleanups I've come up with. And here [2] is the green
> pipeline (not sure about the x86_64-fedora-rawhide-clang failure,
> investigating, but seems unrelated).
> 
> 1: https://lists.libvirt.org/archives/list/devel@lists.libvirt.org/thread/OEP6HH4DACM47GL7BHS2APPYKH7PZESK/
> 2: https://gitlab.com/MichalPrivoznik/libvirt/-/pipelines/1206477954

The fedora rawhide test started failing with more errors (those visible
in the pipeline above) after update to the container image, thus it's
separate.
_______________________________________________
Devel mailing list -- devel@lists.libvirt.org
To unsubscribe send an email to devel-leave@lists.libvirt.org