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., S

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 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: https://reviews.apa

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-17 Thread Saksham Srivastava
avala; 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.

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

2013-01-16 Thread Ram Ganesh
hiradeep Vittal; Rohit 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? &

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

2013-01-16 Thread Chiradeep Vittal
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, 201

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

2013-01-16 Thread Saksham Srivastava
dstack-dev@incubator.apache.org 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, C

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 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. Can the > UI display the

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, 12:4

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" wrote: > >---

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 Kav

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-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" wrote: > > > > On Jan. 10, 2013, 1:04 a.m., Chiradeep Vittal wrote: > > > server/src/com/cloud/server/Configurat

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

2013-01-09 Thread Chiradeep Vittal
That will work, but the console access via the console proxy will be rendered useless. On 1/9/13 5:54 PM, "Rohit Yadav" wrote: > > >> On Jan. 10, 2013, 1:04 a.m., Chiradeep Vittal wrote: >> > server/src/com/cloud/server/ConfigurationServerImpl.java, line 535 >> > >>

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

2013-01-09 Thread Rohit Yadav
> On Jan. 10, 2013, 1:04 a.m., Chiradeep Vittal wrote: > > server/src/com/cloud/server/ConfigurationServerImpl.java, line 535 > > > > > > So how is the admin supposed to be able to login via the console now if > > he c

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 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., sa

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 Kaval