Re: Review Request 16086: Added a temp sleep fix for vm start command

2013-12-10 Thread Girish Shilamkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16086/#review30181 --- Ship it! Committed to 4.2, 4.3 and master. - Girish Shilamkar On

Re: Review Request 16086: Added a temp sleep fix for vm start command

2013-12-09 Thread Santhosh Edukulla
> On Dec. 9, 2013, 8:24 a.m., Sebastien Goasguen wrote: > > what happens if the machine starts only after 31 s ? > > Can we have a polling mechanism instead. 1. I mentioned it as part of "Note" under review description. Here, i mentioned to do more cleaner way once time permits. 2. Actually,

Re: Review Request 16086: Added a temp sleep fix for vm start command

2013-12-09 Thread Sebastien Goasguen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16086/#review29993 --- what happens if the machine starts only after 31 s ? Can we have a p

Re: Review Request 16086: Added a temp sleep fix for vm start command

2013-12-09 Thread SrikanteswaraRao Talluri
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16086/#review29992 --- Can you add more description about why this sleep is required? or ad

Re: Review Request 16086: Added a temp sleep fix for vm start command

2013-12-06 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16086/ --- (Updated Dec. 6, 2013, 6:28 p.m.) Review request for cloudstack and Girish Shil

Review Request 16086: Added a temp sleep fix for vm start command

2013-12-06 Thread Santhosh Edukulla
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16086/ --- Review request for cloudstack. Repository: cloudstack-git Description ---