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 20:08:39 GMT
Filed a JIRA issue for this:
https://issues.apache.org/jira/browse/CLOUDSTACK-1277.
Fix has been checked into 4.1 (Commit
07ce770e9711ab5ddfc9382129b35c96aadc7846) and ported to master.

Thanks
-min


On 2/14/13 10:48 AM, "Min Chen" <min.chen@citrix.com> wrote:

>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.j
>>>a
>>>va
>>> +++ 
>>>b/api/src/org/apache/cloudstack/api/command/user/vm/ResetVMPasswordCmd.j
>>>a
>>>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