[libvirt PATCH v2 0/6] APIs for reporting tainting and deprecation messages

Daniel P. Berrangé posted 6 patches 3 years, 2 months ago
Test syntax-check failed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20210209160109.1229387-1-berrange@redhat.com
include/libvirt/libvirt-domain.h    |  9 +++++
src/conf/domain_conf.c              | 45 ++++++++++++++++++++--
src/conf/domain_conf.h              |  5 +++
src/driver-hypervisor.h             |  6 +++
src/libvirt-domain.c                | 54 +++++++++++++++++++++++++++
src/libvirt_private.syms            |  3 ++
src/libvirt_public.syms             |  5 +++
src/qemu/qemu_domain.c              |  5 +++
src/qemu/qemu_domain.h              |  3 ++
src/qemu/qemu_driver.c              | 58 +++++++++++++++++++++++++++++
src/qemu/qemu_process.c             |  5 +++
src/remote/remote_daemon_dispatch.c | 45 ++++++++++++++++++++++
src/remote/remote_driver.c          | 44 ++++++++++++++++++++++
src/remote/remote_protocol.x        | 21 ++++++++++-
src/remote_protocol-structs         | 11 ++++++
tools/virsh-domain-monitor.c        | 10 +++++
16 files changed, 325 insertions(+), 4 deletions(-)
[libvirt PATCH v2 0/6] APIs for reporting tainting and deprecation messages
Posted by Daniel P. Berrangé 3 years, 2 months ago
This is a follow up to

  https://listman.redhat.com/archives/libvir-list/2021-January/msg00988.html

I pushed the first non-API parts of that series already.

This posting takes a different approach to the APIs. Instead of separte
APIs for tainting and deprecations, there is now one API for reporting
general informational messages. This is explicitly only targetted at
humans.

v2:

 - Change formatting for better translation
 - Fix leak of strings

Daniel P. Berrangé (6):
  conf: record deprecation messages against the domain
  qemu: record deprecation messages against the domain
  src: define virDomainGetMessages API
  remote: add RPC support for the virDomainGetMessages API
  qemu: implement virDomainGetMessages API
  tools: report messages for 'dominfo' command

 include/libvirt/libvirt-domain.h    |  9 +++++
 src/conf/domain_conf.c              | 45 ++++++++++++++++++++--
 src/conf/domain_conf.h              |  5 +++
 src/driver-hypervisor.h             |  6 +++
 src/libvirt-domain.c                | 54 +++++++++++++++++++++++++++
 src/libvirt_private.syms            |  3 ++
 src/libvirt_public.syms             |  5 +++
 src/qemu/qemu_domain.c              |  5 +++
 src/qemu/qemu_domain.h              |  3 ++
 src/qemu/qemu_driver.c              | 58 +++++++++++++++++++++++++++++
 src/qemu/qemu_process.c             |  5 +++
 src/remote/remote_daemon_dispatch.c | 45 ++++++++++++++++++++++
 src/remote/remote_driver.c          | 44 ++++++++++++++++++++++
 src/remote/remote_protocol.x        | 21 ++++++++++-
 src/remote_protocol-structs         | 11 ++++++
 tools/virsh-domain-monitor.c        | 10 +++++
 16 files changed, 325 insertions(+), 4 deletions(-)

-- 
2.29.2


Re: [libvirt PATCH v2 0/6] APIs for reporting tainting and deprecation messages
Posted by Michal Privoznik 3 years, 2 months ago
On 2/9/21 5:01 PM, Daniel P. Berrangé wrote:
> This is a follow up to
> 
>    https://listman.redhat.com/archives/libvir-list/2021-January/msg00988.html
> 
> I pushed the first non-API parts of that series already.
> 
> This posting takes a different approach to the APIs. Instead of separte
> APIs for tainting and deprecations, there is now one API for reporting
> general informational messages. This is explicitly only targetted at
> humans.
> 
> v2:
> 
>   - Change formatting for better translation
>   - Fix leak of strings
> 
> Daniel P. Berrangé (6):
>    conf: record deprecation messages against the domain
>    qemu: record deprecation messages against the domain
>    src: define virDomainGetMessages API
>    remote: add RPC support for the virDomainGetMessages API
>    qemu: implement virDomainGetMessages API
>    tools: report messages for 'dominfo' command
> 
>   include/libvirt/libvirt-domain.h    |  9 +++++
>   src/conf/domain_conf.c              | 45 ++++++++++++++++++++--
>   src/conf/domain_conf.h              |  5 +++
>   src/driver-hypervisor.h             |  6 +++
>   src/libvirt-domain.c                | 54 +++++++++++++++++++++++++++
>   src/libvirt_private.syms            |  3 ++
>   src/libvirt_public.syms             |  5 +++
>   src/qemu/qemu_domain.c              |  5 +++
>   src/qemu/qemu_domain.h              |  3 ++
>   src/qemu/qemu_driver.c              | 58 +++++++++++++++++++++++++++++
>   src/qemu/qemu_process.c             |  5 +++
>   src/remote/remote_daemon_dispatch.c | 45 ++++++++++++++++++++++
>   src/remote/remote_driver.c          | 44 ++++++++++++++++++++++
>   src/remote/remote_protocol.x        | 21 ++++++++++-
>   src/remote_protocol-structs         | 11 ++++++
>   tools/virsh-domain-monitor.c        | 10 +++++
>   16 files changed, 325 insertions(+), 4 deletions(-)
> 

Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

Michal