> On Nov. 4, 2013, 10:48 a.m., Koushik Das wrote: > > api/src/org/apache/cloudstack/api/command/admin/host/ListHostsCmd.java, > > line 92 > > <https://reviews.apache.org/r/15018/diff/2/?file=374631#file374631line92> > > > > Why is this change required for CPU sockets?
We want to show the cpu sockets per hypervisor, but currently we dont have filter on hypervisor host type. This will add that filter. > On Nov. 4, 2013, 10:48 a.m., Koushik Das wrote: > > core/src/com/cloud/agent/api/StartupRoutingCommand.java, line 45 > > <https://reviews.apache.org/r/15018/diff/2/?file=374633#file374633line45> > > > > Why Integer and not int? In some hypervisors cpusockets cannot be retrieved so it can be null in those cases. > On Nov. 4, 2013, 10:48 a.m., Koushik Das wrote: > > plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java, > > line 8076 > > <https://reviews.apache.org/r/15018/diff/2/?file=374639#file374639line8076> > > > > Why Integer? Can this be null? In some hypervisors cpusockets cannot be retrieved so it can be null in those cases. > On Nov. 4, 2013, 10:48 a.m., Koushik Das wrote: > > server/src/com/cloud/api/query/QueryManagerImpl.java, line 1549 > > <https://reviews.apache.org/r/15018/diff/2/?file=374640#file374640line1549> > > > > This change doesn't look related This is included to add filter based on hypervisor host type.(Xenserver, vmware, kvm, Hyperv, Lxc) - Harikrishna ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15018/#review28094 ----------------------------------------------------------- On Nov. 5, 2013, 9:23 a.m., Harikrishna Patnala wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/15018/ > ----------------------------------------------------------- > > (Updated Nov. 5, 2013, 9:23 a.m.) > > > Review request for cloudstack and Kishan Kavala. > > > Bugs: CLOUDSTACK-4908 > https://issues.apache.org/jira/browse/CLOUDSTACK-4908 > > > Repository: cloudstack-git > > > Description > ------- > > CLOUDSTACK-4908: CPU socket count of host > Adding cpu socket count of the host in hostresponse > > > Diffs > ----- > > api/src/com/cloud/host/Host.java 17b0ba8 > api/src/org/apache/cloudstack/api/command/admin/host/ListHostsCmd.java > 95b9ac3 > api/src/org/apache/cloudstack/api/response/HostResponse.java 687687d > core/src/com/cloud/agent/api/StartupRoutingCommand.java d52666b > core/test/org/apache/cloudstack/api/agent/test/CheckOnHostCommandTest.java > 1b68276 > > engine/orchestration/src/org/apache/cloudstack/engine/datacenter/entity/api/db/EngineHostVO.java > 249c2db > engine/schema/src/com/cloud/host/HostVO.java c814b94 > > plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtComputingResource.java > 8d3a0e9 > > plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java > 423abe6 > > plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java > 3323a15 > server/src/com/cloud/api/query/QueryManagerImpl.java 51ee249 > server/src/com/cloud/api/query/dao/HostJoinDaoImpl.java 97f8bf9 > server/src/com/cloud/api/query/vo/HostJoinVO.java fbc59cd > server/src/com/cloud/resource/ResourceManagerImpl.java e9d5193 > setup/db/db/schema-421to430.sql 803f1d9 > vmware-base/src/com/cloud/hypervisor/vmware/mo/HostMO.java 108a3ec > vmware-base/src/com/cloud/hypervisor/vmware/mo/VmwareHypervisorHost.java > f7288f1 > > vmware-base/src/com/cloud/hypervisor/vmware/mo/VmwareHypervisorHostResourceSummary.java > fa11dc3 > > Diff: https://reviews.apache.org/r/15018/diff/ > > > Testing > ------- > > > Thanks, > > Harikrishna Patnala > >
