You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cloudstack.apache.org by Pranav Saxena <pr...@citrix.com> on 2013/02/22 13:52:28 UTC
Cpu and ram overcommit at cluster level - UI integration done
Hi Bharat ,
I have merged the UI changes for your feature as well with asf/master . You could probably test them out .
Regards,
Pranav
-----Original Message-----
From: Abhinandan Prateek [mailto:noreply@reviews.apache.org] On Behalf Of Abhinandan Prateek
Sent: Friday, February 22, 2013 5:17 PM
To: Prachi Damle; Abhinandan Prateek; Nitin Mehta
Cc: cloudstack; Bharat Kumar
Subject: Re: Review Request: Cloudstack-711 Cpu and ram overcommit at cluster level.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/9156/#review16934
-----------------------------------------------------------
Ship it!
Ship It!
- Abhinandan Prateek
On Feb. 22, 2013, 11:34 a.m., bharat kumar wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/9156/
> -----------------------------------------------------------
>
> (Updated Feb. 22, 2013, 11:34 a.m.)
>
>
> Review request for cloudstack, Abhinandan Prateek, Prachi Damle, and Nitin Mehta.
>
>
> Description
> -------
>
> This patch enables cpu and ram overcommit on a per cluster basis(only for xenserver.).
> The dashboard will show actual values instead of the overcommit values.
>
>
> This addresses bug CLOUDSTACK-711.
>
>
> Diffs
> -----
>
> api/src/com/cloud/agent/api/to/VirtualMachineTO.java 8f3f0eb
> api/src/com/cloud/resource/ResourceService.java 1e77cc8
> api/src/com/cloud/vm/VirtualMachineProfile.java 0fab443
> api/src/org/apache/cloudstack/api/ApiConstants.java b12e4cd
> api/src/org/apache/cloudstack/api/command/admin/cluster/AddClusterCmd.java 912c396
> api/src/org/apache/cloudstack/api/command/admin/cluster/UpdateClusterCmd.java 058c7eb
> api/src/org/apache/cloudstack/api/response/ClusterResponse.java 551e530
> plugins/hypervisors/baremetal/src/com/cloud/baremetal/manager/BareMetalPlanner.java fe51eb0
> plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java 9472ea2
> plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java acfd9cf
> plugins/hypervisors/simulator/src/com/cloud/agent/manager/MockVmManagerImpl.java 40a5b98
> plugins/hypervisors/simulator/src/com/cloud/resource/AgentRoutingResource.java 721e5f7
> plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareManagerImpl.java 70f98cc
> plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java 5cac253
> plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java 33ad18d
> plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XcpServerResource.java 0ce91bc
> plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer56FP1Resource.java 58b8a03
> server/src/com/cloud/agent/manager/allocator/impl/FirstFitAllocator.java a25e401
> server/src/com/cloud/api/ApiDBUtils.java c28daef
> server/src/com/cloud/api/ApiResponseHelper.java 41cf96e
> server/src/com/cloud/capacity/CapacityManager.java 656e744
> server/src/com/cloud/capacity/CapacityManagerImpl.java 74152ff
> server/src/com/cloud/capacity/ComputeCapacityListener.java 8ea695a
> server/src/com/cloud/capacity/dao/CapacityDao.java 0c0723b
> server/src/com/cloud/capacity/dao/CapacityDaoImpl.java 3582614
> server/src/com/cloud/deploy/FirstFitPlanner.java b452da0
> server/src/com/cloud/hypervisor/HypervisorGuruBase.java e158962
> server/src/com/cloud/resource/ResourceManagerImpl.java 98044fb
> server/src/com/cloud/vm/VirtualMachineManagerImpl.java 2b70ff6
> server/src/com/cloud/vm/VirtualMachineProfileImpl.java eb9e5ad
> server/test/com/cloud/capacity/CapacityManagerTest.java PRE-CREATION
> server/test/com/cloud/resource/MockResourceManagerImpl.java 889318b
> vmware-base/src/com/cloud/hypervisor/vmware/util/VmwareHelper.java 47ff8e2
>
> Diff: https://reviews.apache.org/r/9156/diff/
>
>
> Testing
> -------
>
> Did manual testing.
>
>
> Thanks,
>
> bharat kumar
>
>