Review Request 12943: Replaced virsh() by python-libvirt functions

2013-07-25 Thread Rene Diepstraten
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12943/ --- Review request for cloudstack and Wido den Hollander. Repository: cloudstack-gi

Re: Review Request 12943: Replaced virsh() by python-libvirt functions

2013-07-25 Thread Jenkins Cloudstack.org
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12943/#review23844 --- Review 12943 PASSED the build test The url of build cloudstack-maste

Re: Review Request 12943: Replaced virsh() by python-libvirt functions

2013-07-25 Thread Rajesh Battala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12943/#review23848 --- Ship it! Ship It! - Rajesh Battala On July 25, 2013, 2:01 p.m.,

Re: Review Request 12943: Replaced virsh() by python-libvirt functions

2013-07-25 Thread Wido den Hollander
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12943/#review23852 --- Ship it! Seems good to me! - Wido den Hollander On July 25, 2013

Re: Review Request 12943: Replaced virsh() by python-libvirt functions

2013-07-26 Thread Animesh Chaturvedi
> On July 25, 2013, 4:09 p.m., Wido den Hollander wrote: > > Seems good to me! Wido, Rajesh if the patch is good please commit and help close the issue - Animesh --- This is an automatically generated e-mail. To reply, visit: https://re