incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Min Chen <min.c...@citrix.com>
Subject Re: Password has been reset to undefined
Date Thu, 14 Feb 2013 18:48:09 GMT
I found the problem for this. The reason is that in UserVmVO class,
"password" is defined as "transit", so it is not stored in Database. When
we use id to find this user vm from user_vm_view, the password set from
reset routine internally is lost. I will file a defect on this and provide
a fix today.

Thanks
-min

On 2/14/13 2:13 AM, "Rohit Yadav" <bhaisaab@apache.org> wrote:

>Hi Sinisa, thanks for reporting this. Please file this issue on JIRA.
>But AFAIK the response generator is supposed to create the response,
>the cmd api class should not add the password. I don't remember if it
>was an issue, let's ask Min as she did a lot of the response
>generating stuff.
>
>Hi Min, can you check please?
>
>Regards.
>
>On Thu, Feb 14, 2013 at 3:36 PM, Sinisa Denic
><sdenic@peacebellservers.com> wrote:
>> Sorry I forgot to set cs-dev mailing list in cc in my previous post so
>>other users can see.
>> I will repeat:
>>
>> In order to get password on UI after reset password for virtualmachine
>>in CS4.1.0-SNAPSHOT
>> I had to made this modification:
>>
>> --- 
>>a/api/src/org/apache/cloudstack/api/command/user/vm/ResetVMPasswordCmd.ja
>>va
>> +++ 
>>b/api/src/org/apache/cloudstack/api/command/user/vm/ResetVMPasswordCmd.ja
>>va
>> @@ -114,6 +114,7 @@ public class ResetVMPasswordCmd extends
>>BaseAsyncCmd {
>>          if (result != null){
>>              UserVmResponse response =
>>_responseGenerator.createUserVmResponse("virtualmachine", result).get(0);
>>              response.setResponseName(getCommandName());
>> +            response.setPassword(password);
>>              this.setResponseObject(response);
>>          } else {
>>              throw new ServerApiException(ApiErrorCode.INTERNAL_ERROR,
>>"Failed to reset vm password");
>>
>> Do you maybe know where the problem could reside,as I spent a few days
>>on it :(?


Mime
View raw message