----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9156/ -----------------------------------------------------------
(Updated Feb. 1, 2013, 11:11 a.m.) Review request for cloudstack, Abhinandan Prateek, Prachi Damle, and Nitin Mehta. Changes ------- updated the diff with a test to check the CheckIfHostHasCapacity function in CapacityManager. 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/vm/VirtualMachineProfile.java 0fab443 api/src/org/apache/cloudstack/api/ApiConstants.java 58a7831 api/src/org/apache/cloudstack/api/command/admin/cluster/AddClusterCmd.java c64883c api/src/org/apache/cloudstack/api/response/ClusterResponse.java e436c99 plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java 065d3be 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 2a5258c server/src/com/cloud/api/ApiDBUtils.java 0b08b26 server/src/com/cloud/api/ApiResponseHelper.java 1c8849a server/src/com/cloud/capacity/CapacityManager.java fffb41f server/src/com/cloud/capacity/CapacityManagerImpl.java c410109 server/src/com/cloud/capacity/dao/CapacityDao.java 0c0723b server/src/com/cloud/capacity/dao/CapacityDaoImpl.java a2df475 server/src/com/cloud/deploy/BareMetalPlanner.java 7616a38 server/src/com/cloud/deploy/FirstFitPlanner.java 06e46fb server/src/com/cloud/hypervisor/HypervisorGuruBase.java 2efe6d3 server/src/com/cloud/resource/ResourceManagerImpl.java f82424a server/src/com/cloud/vm/VirtualMachineManagerImpl.java e0647bb server/src/com/cloud/vm/VirtualMachineProfileImpl.java e83d6a0 server/test/com/cloud/capacity/CapacityManagerTest.java PRE-CREATION Diff: https://reviews.apache.org/r/9156/diff/ Testing ------- Did manual testing. Thanks, bharat kumar
