virsh: Fix cleanup in 'event' command and introduce 'await' command

[PATCH 0/7] virsh: Fix cleanup in 'event' command and introduce 'await' command

Peter Krempa via Devel posted 7 patches 3 months ago

Download series mbox

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

docs/manpages/virsh.rst    |  30 ++++
src/util/virthread.h       |   7 +
tools/virsh-domain-event.c | 321 ++++++++++++++++++++++++++++++++++++-
tools/virsh.c              |   4 +-
tools/virt-admin.c         |   4 +-
tools/vsh.c                |  46 ++++--
tools/vsh.h                |   3 +-
7 files changed, 398 insertions(+), 17 deletions(-)

Expand all Fold all

[PATCH 0/7] virsh: Fix cleanup in 'event' command and introduce 'await' command

Posted by Peter Krempa via Devel 3 months ago

'virsh await' is a helper/syntax-sugar command which allows you to wait
until a domain reaches a certain state. Currently this series implements
conditions named 'domain-inactive' and 'guest-agent-available'.

Peter Krempa (7):
  virthread: Register auto cleanup function for virMutex
  virthread: Add infrastructure for static virCond definitions
  virsh: cmdEvent: Ensure that event callbacks are unregistered before
    returning
  vshCommandRun: Convert to directly return the exit code
  vsh: Add support for commands with more return values
  virsh: Introduce 'await' command for waiting until target domain state
    is reached
  virsh: await: Introduce 'guest-agent-available' condition

 docs/manpages/virsh.rst    |  30 ++++
 src/util/virthread.h       |   7 +
 tools/virsh-domain-event.c | 321 ++++++++++++++++++++++++++++++++++++-
 tools/virsh.c              |   4 +-
 tools/virt-admin.c         |   4 +-
 tools/vsh.c                |  46 ++++--
 tools/vsh.h                |   3 +-
 7 files changed, 398 insertions(+), 17 deletions(-)

-- 
2.49.0

Re: [PATCH 0/7] virsh: Fix cleanup in 'event' command and introduce 'await' command

Posted by Ján Tomko via Devel 3 months ago

On a Thursday in 2025, Peter Krempa via Devel wrote:
>'virsh await' is a helper/syntax-sugar command which allows you to wait
>until a domain reaches a certain state. Currently this series implements
>conditions named 'domain-inactive' and 'guest-agent-available'.
>
>Peter Krempa (7):
>  virthread: Register auto cleanup function for virMutex
>  virthread: Add infrastructure for static virCond definitions
>  virsh: cmdEvent: Ensure that event callbacks are unregistered before
>    returning
>  vshCommandRun: Convert to directly return the exit code
>  vsh: Add support for commands with more return values
>  virsh: Introduce 'await' command for waiting until target domain state
>    is reached
>  virsh: await: Introduce 'guest-agent-available' condition
>
> docs/manpages/virsh.rst    |  30 ++++
> src/util/virthread.h       |   7 +
> tools/virsh-domain-event.c | 321 ++++++++++++++++++++++++++++++++++++-
> tools/virsh.c              |   4 +-
> tools/virt-admin.c         |   4 +-
> tools/vsh.c                |  46 ++++--
> tools/vsh.h                |   3 +-
> 7 files changed, 398 insertions(+), 17 deletions(-)
>

Reviewed-by: Ján Tomko <jtomko@redhat.com>

Jano