Re: Review Request: (CLOUDSTACK-1325) add password in response of RestoreVM

2013-05-27 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9539/#review21071 --- Commit a5723992a0eeea9fc383c656b0d61dfc9e541ff9 in branch refs/heads/

RE: Review Request: (CLOUDSTACK-1325) add password in response of RestoreVM

2013-05-13 Thread Harikrishna Patnala
; Abhinandan Prateek Cc: Sateesh Chodapuneedi; cloudstack; Wei Zhou Subject: RE: Review Request: (CLOUDSTACK-1325) add password in response of RestoreVM Abhi following up on this can you review and apply > -Original Message- > From: Prasanna Santhanam [mailto:nore...@reviews.apache.o

RE: Review Request: (CLOUDSTACK-1325) add password in response of RestoreVM

2013-05-09 Thread Animesh Chaturvedi
apuneedi; Prasanna Santhanam; cloudstack; Wei Zhou > Subject: Re: Review Request: (CLOUDSTACK-1325) add password in response > of RestoreVM > > > --- > This is an automatically generated e-mail. To reply, visit:

Re: Review Request: (CLOUDSTACK-1325) add password in response of RestoreVM

2013-05-05 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9539/ --- (Updated May 6, 2013, 5:12 a.m.) Review request for cloudstack, Abhinandan Prate

Re: Review Request: (CLOUDSTACK-1325) add password in response of RestoreVM

2013-04-25 Thread Wei Zhou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9539/ --- (Updated April 25, 2013, 10:18 a.m.) Review request for cloudstack, Sateesh Chod

Re: Review Request: (CLOUDSTACK-1325) add password in response of RestoreVM

2013-04-25 Thread Harikrishna Patnala
> On April 25, 2013, 8:55 a.m., Harikrishna Patnala wrote: > > Ship It! Not a right time to give comments but came across a difference when comparing with my patch on master. we need to load vm details "_vmDao.loadDetails(vm)" before getting the details "vm.getDetail("SSH.PublicKey")". since d

Re: Review Request: (CLOUDSTACK-1325) add password in response of RestoreVM

2013-04-25 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9539/#review19694 --- Ship it! Ship It! - Harikrishna Patnala On April 2, 2013, 1:24 p.

Re: Review Request: (CLOUDSTACK-1325) add password in response of RestoreVM

2013-04-02 Thread Wei Zhou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9539/ --- (Updated April 2, 2013, 1:24 p.m.) Review request for cloudstack, Sateesh Chodap

Re: Review Request: (CLOUDSTACK-1325) add password in response of RestoreVM

2013-03-26 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9539/#review18386 --- server/src/com/cloud/vm/UserVmManagerImpl.java

Re: Review Request: (CLOUDSTACK-1325) add password in response of RestoreVM

2013-03-21 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9539/ --- (Updated March 22, 2013, 4:42 a.m.) Review request for cloudstack, Sateesh Choda