Re: [libvirt] missing libvirt events related to snapshots and media-change for inactive VMs?

2018-09-24 Thread Peter Krempa
On Fri, Sep 21, 2018 at 08:39:20 +0200, Philipp Hahn wrote:
> Hello,
> 
> Am 20.09.18 um 08:10 schrieb Philipp Hahn:
> > event-test.py is bad at handling newer livecycle events.
> > Attached are two patches to improve that.
> > 
> > Philipp Hahn (2):
> >   event-test.py: Sync list of domain lifecycle events
> >   event-test.py: Future proof lifecycle event handling
> > 
> >  examples/event-test.py | 18 --
> >  1 file changed, 12 insertions(+), 6 deletions(-)
> 
> While looking at the events reported by said script I noticed that there
> are no events for creating / deleting snapshot, at leat when the VM is
> inactive. For an active VM Qemu reports several events:
>   VIR_DOMAIN_EVENT_DEFINED_FROM_SNAPSHOT
>   VIR_DOMAIN_EVENT_STOPPED_FROM_SNAPSHOT
>   VIR_DOMAIN_EVENT_STARTED_FROM_SNAPSHOT
>   VIR_DOMAIN_EVENT_SUSPENDED_FROM_SNAPSHOT
>   VIR_DOMAIN_EVENT_RESUMED_FROM_SNAPSHOT
> 
> Also there is no event generated when doing "virsh change-media" for an
> inactive domain; I would have expected an
> VIR_DOMAIN_EVENT_DEFINED_UPDATED event.
> For an active domain you get the
> libvirt.VIR_DOMAIN_EVENT_TRAY_CHANGE_OPEN / CLOSE events.
> 
> My current work-around is to do polling, but I want to get rid of that.
> Did I miss something or is it okay to add events for tha?

The UPDATED event would make sense for both online/offline VMs, but it
wasn't implemented there yet.

Tray change events for offline VMs obviously don't make sense. Also for
offline VMs the media change always succeeded if the API was successful.

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


Re: [libvirt] missing libvirt events related to snapshots and media-change for inactive VMs?

2018-09-21 Thread Philipp Hahn
Hello,

Am 20.09.18 um 08:10 schrieb Philipp Hahn:
> event-test.py is bad at handling newer livecycle events.
> Attached are two patches to improve that.
> 
> Philipp Hahn (2):
>   event-test.py: Sync list of domain lifecycle events
>   event-test.py: Future proof lifecycle event handling
> 
>  examples/event-test.py | 18 --
>  1 file changed, 12 insertions(+), 6 deletions(-)

While looking at the events reported by said script I noticed that there
are no events for creating / deleting snapshot, at leat when the VM is
inactive. For an active VM Qemu reports several events:
  VIR_DOMAIN_EVENT_DEFINED_FROM_SNAPSHOT
  VIR_DOMAIN_EVENT_STOPPED_FROM_SNAPSHOT
  VIR_DOMAIN_EVENT_STARTED_FROM_SNAPSHOT
  VIR_DOMAIN_EVENT_SUSPENDED_FROM_SNAPSHOT
  VIR_DOMAIN_EVENT_RESUMED_FROM_SNAPSHOT

Also there is no event generated when doing "virsh change-media" for an
inactive domain; I would have expected an
VIR_DOMAIN_EVENT_DEFINED_UPDATED event.
For an active domain you get the
libvirt.VIR_DOMAIN_EVENT_TRAY_CHANGE_OPEN / CLOSE events.

My current work-around is to do polling, but I want to get rid of that.
Did I miss something or is it okay to add events for tha?

Thanks in advance.
Philipp

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