incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "bharat kumar" <bharat.ku...@citrix.com>
Subject Re: Review Request: Cloudstack-711 Cpu and ram overcommit at cluster level.
Date Thu, 21 Feb 2013 16:56:45 GMT

-----------------------------------------------------------
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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message