Re: Review Request: CLOUDSTACK-822 system.vm.password is not encrypted

2013-02-05 Thread Kishan Kavala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8859/#review16108 --- Ship it! Ship It! - Kishan Kavala On Jan. 10, 2013, 12:40 p.m.,

Re: Review Request: CLOUDSTACK-822 system.vm.password is not encrypted

2013-02-01 Thread Kishan Kavala
On Jan. 31, 2013, 10:15 p.m., Rohit Yadav wrote: Two ship it, is it already merged? If not, confirm I can commit it. saksham srivastava wrote: Still not merged. Merged. commit 482ea4a71e40f35ea3a222e56fa7b9760c21f8a2 - Kishan

Re: Review Request: CLOUDSTACK-822 system.vm.password is not encrypted

2013-01-31 Thread Rohit Yadav
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8859/#review15979 --- Two ship it, is it already merged? If not, confirm I can commit it.

Re: Review Request: CLOUDSTACK-822 system.vm.password is not encrypted

2013-01-31 Thread saksham srivastava
On Jan. 31, 2013, 10:15 p.m., Rohit Yadav wrote: Two ship it, is it already merged? If not, confirm I can commit it. Still not merged. - saksham --- This is an automatically generated e-mail. To reply, visit:

RE: Review Request: CLOUDSTACK-822 system.vm.password is not encrypted

2013-01-17 Thread Saksham Srivastava
; Rajesh Battala; Chiradeep Vittal; Rohit Yadav Subject: RE: Review Request: CLOUDSTACK-822 system.vm.password is not encrypted Chiradeep, I have filed a doc bug, CLOUDSTACK-991, to track this one. -Original Message- From: Chiradeep Vittal [mailto:chiradeep.vit...@citrix.com] Sent: 17

Re: Review Request: CLOUDSTACK-822 system.vm.password is not encrypted

2013-01-16 Thread Kishan Kavala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8859/#review15387 --- Ship it! Looks good to me. - Kishan Kavala On Jan. 10, 2013,

Re: Review Request: CLOUDSTACK-822 system.vm.password is not encrypted

2013-01-16 Thread Chip Childers
Can we get an answer to Chiradeep's question below before this is committed? On Thu, Jan 10, 2013 at 1:49 PM, Chiradeep Vittal chiradeep.vit...@citrix.com wrote: The question around how the cloud admin can log in to the system vm without visibility into the actual password needs to be resolved.

RE: Review Request: CLOUDSTACK-822 system.vm.password is not encrypted

2013-01-16 Thread Saksham Srivastava
Cc: Saksham Srivastava; Kishan Kavala; Rajesh Battala; Chiradeep Vittal; Rohit Yadav Subject: Re: Review Request: CLOUDSTACK-822 system.vm.password is not encrypted Can we get an answer to Chiradeep's question below before this is committed? On Thu, Jan 10, 2013 at 1:49 PM, Chiradeep Vittal

Re: Review Request: CLOUDSTACK-822 system.vm.password is not encrypted

2013-01-16 Thread Chiradeep Vittal
Vittal; Rohit Yadav Subject: Re: Review Request: CLOUDSTACK-822 system.vm.password is not encrypted Can we get an answer to Chiradeep's question below before this is committed? On Thu, Jan 10, 2013 at 1:49 PM, Chiradeep Vittal chiradeep.vit...@citrix.com wrote: The question around how the cloud

RE: Review Request: CLOUDSTACK-822 system.vm.password is not encrypted

2013-01-16 Thread Ram Ganesh
Yadav Subject: Re: Review Request: CLOUDSTACK-822 system.vm.password is not encrypted This also needs to be documented. Can you raise a documentation issue? What about the upgrade from 4.0 case? Are we encrypting previously unencrypted passwords? On 1/16/13 10:05 AM, Saksham Srivastava

Re: Review Request: CLOUDSTACK-822 system.vm.password is not encrypted

2013-01-10 Thread Kishan Kavala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8859/#review15235 --- Parameter category can be set Secure instead of Hidden. This will

Re: Review Request: CLOUDSTACK-822 system.vm.password is not encrypted

2013-01-10 Thread saksham srivastava
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8859/ --- (Updated Jan. 10, 2013, 12:40 p.m.) Review request for cloudstack and Kishan

Re: Review Request: CLOUDSTACK-822 system.vm.password is not encrypted

2013-01-10 Thread Chiradeep Vittal
The question around how the cloud admin can log in to the system vm without visibility into the actual password needs to be resolved. Can the UI display the unencrypted password whenever the console is viewed? On 1/10/13 4:40 AM, Saksham Srivastava saksham.srivast...@citrix.com wrote:

Re: Review Request: CLOUDSTACK-822 system.vm.password is not encrypted

2013-01-09 Thread saksham srivastava
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8859/ --- (Updated Jan. 9, 2013, 8:43 a.m.) Review request for cloudstack and Kishan

Re: Review Request: CLOUDSTACK-822 system.vm.password is not encrypted

2013-01-09 Thread Rajesh Battala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8859/#review15196 --- Ship it! Ship It! - Rajesh Battala On Jan. 9, 2013, 8:43 a.m.,

Re: Review Request: CLOUDSTACK-822 system.vm.password is not encrypted

2013-01-09 Thread Chiradeep Vittal
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8859/#review15221 --- server/src/com/cloud/server/ConfigurationServerImpl.java

Re: Review Request: CLOUDSTACK-822 system.vm.password is not encrypted

2013-01-09 Thread Marcus Sorensen
The only time I log in via VNC is when link local ssh doesn't work. So if we don't know the password then doing ssh to link local isn't an option. On Jan 9, 2013 6:54 PM, Rohit Yadav bhais...@apache.org wrote: On Jan. 10, 2013, 1:04 a.m., Chiradeep Vittal wrote:

Review Request: CLOUDSTACK-822 system.vm.password is not encrypted

2013-01-07 Thread saksham srivastava
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8859/ --- Review request for cloudstack and Kishan Kavala. Description --- Parameter