----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9156/ -----------------------------------------------------------
(Updated Feb. 21, 2013, 4:56 p.m.) Review request for cloudstack, Abhinandan Prateek, Prachi Damle, and Nitin Mehta. Changes ------- Added a Test and made some changes to update cluster command to enable overcommit update. Removed tabs. 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 (updated) ----- 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 cd7d700 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/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 e6d5742 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 e6b1bf1 server/src/com/cloud/api/ApiResponseHelper.java a94e935 server/src/com/cloud/capacity/CapacityManager.java fffb41f server/src/com/cloud/capacity/CapacityManagerImpl.java 4787c7b 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 c33bfaf server/src/com/cloud/deploy/BareMetalPlanner.java 829a466 server/src/com/cloud/deploy/FirstFitPlanner.java 66a24ac server/src/com/cloud/hypervisor/HypervisorGuruBase.java e158962 server/src/com/cloud/resource/ResourceManagerImpl.java 79ccdb3 server/src/com/cloud/vm/VirtualMachineManagerImpl.java d0a2305 server/src/com/cloud/vm/VirtualMachineProfileImpl.java e83d6a0 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