You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Alena Prokharchyk <al...@citrix.com> on 2014/04/01 18:39:51 UTC

Re: Review Request 19751: Fixed [CLOUDSTACK-5962]: Value of Global parameter "custom.diskoffering.size.min" is not reflected in UI during new instance creation

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


Java part of the fix looks fine to me.

- Alena Prokharchyk


On March 28, 2014, 2:45 p.m., Gabor Apati-Nagy wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19751/
> -----------------------------------------------------------
> 
> (Updated March 28, 2014, 2:45 p.m.)
> 
> 
> Review request for cloudstack, Brian Federle and Jessica Wang.
> 
> 
> Bugs: CLOUDSTACK-5962
>     https://issues.apache.org/jira/browse/CLOUDSTACK-5962
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Fix for [CLOUDSTACK-5962]: Value of Global parameter "custom.diskoffering.size.min" is not reflected in UI during new instance creation.
> 
> Added fields to /api and /server classes for CustomDiskOfferingMinSize to be available in CapabilitiesResponse. Fixed UI code in Instance Wizard to have this config value as the minimum selectable option when we are in custom disk size mode.
> 
> 
> Diffs
> -----
> 
>   api/src/org/apache/cloudstack/api/ApiConstants.java 87fa8ee 
>   api/src/org/apache/cloudstack/api/command/user/config/ListCapabilitiesCmd.java d47645b 
>   api/src/org/apache/cloudstack/api/response/CapabilitiesResponse.java c8de587 
>   server/src/com/cloud/server/ManagementServerImpl.java 7431891 
>   ui/index.jsp dce2aa8 
>   ui/scripts/instanceWizard.js 03ee344 
>   ui/scripts/ui-custom/instanceWizard.js c5e0ddb 
> 
> Diff: https://reviews.apache.org/r/19751/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Gabor Apati-Nagy
> 
>