Nitin thanks for the review comments. will create a branch and fix the issues.
Bharat. On Jan 30, 2013, at 8:35 PM, bharat kumar <bharat.ku...@citrix.com<mailto:bharat.ku...@citrix.com>> wrote: This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9156/ Review request for cloudstack, Abhinandan Prateek, Prachi Damle, and Nitin Mehta. By bharat kumar. Summary (updated) Cloudstack-711 Cpu and ram overcommit at cluster level. Description (updated) 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. Testing (updated) Did manual testing. Bugs: 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) View Diff<https://reviews.apache.org/r/9156/diff/>
