On 06/14/2016 02:09 AM, Peter Krempa wrote: > On Mon, Jun 13, 2016 at 13:35:04 -0400, Cole Robinson wrote: >> On 06/13/2016 12:38 PM, Jovanka Gulicoska wrote: > > [...] > >> >> I just reviewed the diff between v1 and v2 and made a couple small changes, >> diff attached. I copied the docstring from the network events since your >> version still had several typos, and fixed one spacing issue in comments in >> the test driver. Otherwise it looks like all my review bits were addressed, so >> this looks good to me. >> >> Let's give it some time to see if anyone else has comments, otherwise I'll >> push it at the end of the week (with those minor changes) > > Apart from needing to make sure that this series passes 'make check' and > the few nits pointed out through the code there's one thing that should > be added. Every event implementation has a virsh command allowing to > watch them and the virsh change is still missing. It's okay to add it > later but it should be in this release. > Ah I noticed that there was 'net-event' but I didn't realize we had 'event' for domains as well. Thanks for pointing that out, I agree a 'pool-event' command should be part of this series - Cole -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list