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. -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list