On 12/19/24 18:07, Praveen K Paladugu wrote:
> LogContext management is now moved from Qemu driver to hypervisor. After
> migrating Qemu to use domain_logcontext, I extended ch driver to use also use
> domain_logcontext to capture early boot failures within domain specific log
> files.
>
> Changes in V2:
> * refactored the patches to ensure all of them build.
> * ch driver will use virtlogd to consolidate logs from hypervisor and its
> domains.
>
> Praveen K Paladugu (11):
> hypervisor: copy qemu log context mgmt to hypervisor
> hypervisor: rename reference to qemu in domain_logcontext
> hypervisor: drop qemu specific args in domainLogContextNew
> hypervisor: Build domain_logcontext
> libvirt_private: export symbols from domain_logcontext
> qemu: Modify qemu driver to use domainLogContext
> qemu: delete qemu_logcontext files
> ch: Enable logging for ch domains
> ch: move curl_data and curl_callback definitions
> ch: Enable logging curl responses from ch
> ch: configure ch driver to use virlogd
>
> po/POTFILES | 2 +-
> src/ch/ch_conf.c | 1 +
> src/ch/ch_conf.h | 2 +
> src/ch/ch_monitor.c | 84 ++++++++++++-------
> src/ch/ch_monitor.h | 6 +-
> src/ch/ch_process.c | 36 ++++++--
> .../domain_logcontext.c} | 78 +++++++++--------
> src/hypervisor/domain_logcontext.h | 45 ++++++++++
> src/hypervisor/meson.build | 1 +
> src/libvirt_private.syms | 6 ++
> src/qemu/meson.build | 1 -
> src/qemu/qemu_domain.c | 28 +++----
> src/qemu/qemu_domain.h | 12 +--
> src/qemu/qemu_logcontext.h | 41 ---------
> src/qemu/qemu_nbdkit.c | 12 ++-
> src/qemu/qemu_process.c | 45 +++++-----
> 16 files changed, 236 insertions(+), 164 deletions(-)
> rename src/{qemu/qemu_logcontext.c => hypervisor/domain_logcontext.c} (79%)
> create mode 100644 src/hypervisor/domain_logcontext.h
> delete mode 100644 src/qemu/qemu_logcontext.h
>
Is this still something you're interested in? I mean, I've just merged a
patch that allows CH driver to parse a config file. So that might be
used to configure logging for CH driver. If so, can you please rebase
and resend?
Michal
