You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@cloudstack.apache.org by Kishan Kavala <Ki...@citrix.com> on 2014/05/05 08:16:04 UTC

Re: Review Request 15349: Overprivisoning changes needed

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

Ship it!


commit 99b4cf788eded0f6f155ff8cda51aeb2e505f764

- Kishan Kavala


On April 17, 2014, 10:01 a.m., bharat kumar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/15349/
> -----------------------------------------------------------
> 
> (Updated April 17, 2014, 10:01 a.m.)
> 
> 
> Review request for cloudstack, Kishan Kavala and Nitin Mehta.
> 
> 
> Bugs: Cloudstack-5077.
>     https://issues.apache.org/jira/browse/Cloudstack-5077.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> 
> 1.) This includes the change to reserve cpu and memory only when vmware.reserve.cpu/mem is true, regardless of overcommit. 
> 2.) populate the default value of the cpu overcommit at cluster level form the global setting when up grading. 
> 
> 
> Diffs
> -----
> 
>   engine/schema/src/com/cloud/upgrade/dao/Upgrade410to420.java 82b6d84 
>   engine/schema/src/com/cloud/upgrade/dao/Upgrade420to421.java c3a6b06 
>   plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java e89777d 
>   setup/db/db/schema-420to421.sql 42e9c1e 
> 
> Diff: https://reviews.apache.org/r/15349/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> bharat kumar
> 
>