Re: Review Request: CLOUDSTACK-1453: support restore for VM created from ISO

2013-05-24 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9779/ --- (Updated May 24, 2013, 10:19 a.m.) Review request for cloudstack, Nitin Mehta

Re: Review Request: CLOUDSTACK-1453: support restore for VM created from ISO

2013-05-24 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9779/#review20990 --- Commit a34e577d1be5a25d7eb884147212ab51786b55e2 in branch

Re: Review Request: CLOUDSTACK-1453: support restore for VM created from ISO

2013-05-24 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9779/#review20991 --- Ship it! Ship It! - Koushik Das On May 24, 2013, 10:19 a.m.,

Re: Review Request: CLOUDSTACK-1453: support restore for VM created from ISO

2013-05-23 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9779/ --- (Updated May 24, 2013, 5:52 a.m.) Review request for cloudstack, Nitin Mehta

Re: Review Request: CLOUDSTACK-1453: support restore for VM created from ISO

2013-05-23 Thread Harikrishna Patnala
On April 1, 2013, 7:41 a.m., Koushik Das wrote: server/src/com/cloud/vm/UserVmManagerImpl.java, line 3884 https://reviews.apache.org/r/9779/diff/1/?file=267170#file267170line3884 Handling of iso case is not clear. Are you expecting the iso id in the newTemplateId parameter? Now

Re: Review Request: CLOUDSTACK-1453: support restore for VM created from ISO

2013-04-01 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9779/#review18578 --- api/src/org/apache/cloudstack/api/command/user/vm/RestoreVMCmd.java