incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Rohit Yadav" <rohit.ya...@citrix.com>
Subject Re: Review Request: Fix for CLOUDSTACK-648
Date Mon, 17 Dec 2012 18:40:14 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8637/#review14595
-----------------------------------------------------------


Pl. fix those evil trailing whitespaces and tabs (no tabs, only whitespaces please).
Not sure if we should it this way. Probably the whole user and project management should be
moved out of cloudstack as an adapter or business layer. (not sure about the best way, no
flames please :)

- Rohit Yadav


On Dec. 17, 2012, 3:39 p.m., Isaac Chiang wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/8637/
> -----------------------------------------------------------
> 
> (Updated Dec. 17, 2012, 3:39 p.m.)
> 
> 
> Review request for cloudstack and Gavin Lee.
> 
> 
> Description
> -------
> 
> This patch is to fix the CLOUDSTACK-436 issue described in https://issues.apache.org/jira/browse/CLOUDSTACK-648.
> 
> 
> Diffs
> -----
> 
>   api/src/com/cloud/api/response/UserResponse.java 920eb59 
>   client/tomcatconf/commands.properties.in 149547e 
>   server/src/com/cloud/api/ApiDBUtils.java 3b5f634 
>   server/src/com/cloud/api/ApiResponseHelper.java ebe8415 
>   ui/scripts/accounts.js 324c5f5 
>   ui/scripts/sharedFunctions.js f1b7efa 
> 
> Diff: https://reviews.apache.org/r/8637/diff/
> 
> 
> Testing
> -------
> 
> 1. A CloudStack normal user can modify its own password.
> 2. Domain admin can change it's domain user's password, including subdomain admins and
subdomain users.
> 3. In the same domain, a domain admin cannot update cannot modify another domain admin's
password.
> 
> 
> Thanks,
> 
> Isaac Chiang
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message