Re: [PATCH] storage: Add new events for *PoolBuild() and *PoolDelete().

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 09/09/2017 05:09 PM, Julio Faracco wrote:
> This commit adds new events for two methods and operations: *PoolBuild() and
> *PoolDelete(). Using the event-test and the commands set below we have the
> following outputs:
> 
> $ sudo ./event-test
> Registering event callbacks
> myStoragePoolEventCallback EVENT: Storage pool test Defined 0
> myStoragePoolEventCallback EVENT: Storage pool test Created 0
> myStoragePoolEventCallback EVENT: Storage pool test Started 0
> myStoragePoolEventCallback EVENT: Storage pool test Stopped 0
> myStoragePoolEventCallback EVENT: Storage pool test Deleted 0
> myStoragePoolEventCallback EVENT: Storage pool test Undefined 0
> 
> Another terminal:
> $ sudo virsh pool-define test.xml
> Pool test defined from test.xml
> 
> $ sudo virsh pool-build test
> Pool test built
> 
> $ sudo virsh pool-start test
> Pool test started
> 
> $ sudo virsh pool-destroy test
> Pool test destroyed
> 
> $ sudo virsh pool-delete test
> Pool test deleted
> 
> $ sudo virsh pool-undefine test
> Pool test has been undefined
> 
> This commits can be a solution for RHBZ #1475227.
> 
> Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1475227
> 
> Signed-off-by: Julio Faracco <jcfaracco@xxxxxxxxx>
> ---
>  examples/object-events/event-test.c |  4 ++++
>  include/libvirt/libvirt-storage.h   |  2 ++
>  src/storage/storage_driver.c        | 17 +++++++++++++++++
>  src/test/test_driver.c              | 14 ++++++++++++++
>  tools/virsh-pool.c                  |  4 +++-
>  5 files changed, 40 insertions(+), 1 deletion(-)

Looking good. Before pushing I'm adding two test cases to objecteventtest.

ACKed and pushed.

Michal

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



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]
  Powered by Linux