Re: Review Request 12775: Migrate the dozens of ssh calls for KVM VmDataCommand to a single router command

2013-07-26 Thread Marcus Sorensen
I did, I'll close the review request On Fri, Jul 26, 2013 at 3:39 PM, Animesh Chaturvedi animesh.chaturv...@citrix.com wrote: This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12775/ On July 24th, 2013, 2:08 p.m. UTC, *Wido den Hollander* wrote:

Re: Review Request 12775: Migrate the dozens of ssh calls for KVM VmDataCommand to a single router command

2013-07-26 Thread Animesh Chaturvedi
On July 24, 2013, 2:08 p.m., Wido den Hollander wrote: So it took me some time to get around reviewing this, but it seems fine to me. I patched a running 4.1 cluster with this and works like a charm. DHCP entries were loaded at the same rate, but the user metadata went a lot

Re: Review Request 12775: Migrate the dozens of ssh calls for KVM VmDataCommand to a single router command

2013-07-24 Thread Wido den Hollander
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12775/#review23764 --- Ship it! So it took me some time to get around reviewing this, but

Re: Review Request 12775: Migrate the dozens of ssh calls for KVM VmDataCommand to a single router command

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

Review Request 12775: Migrate the dozens of ssh calls for KVM VmDataCommand to a single router command

2013-07-19 Thread Marcus Sorensen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12775/ --- Review request for cloudstack, ilya musayev and Wido den Hollander. Bugs: