----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12679/ -----------------------------------------------------------
(Updated July 18, 2013, 4:06 p.m.) Review request for cloudstack and Wido den Hollander. Bugs: CLOUDSTACK-904 Repository: cloudstack-git Description ------- Cloudstack defines a vm with multiple CPUs instead of one CPU with multiple cores. For Windows as well as RHEL guests, the license is based on the amount of CPU sockets. The definition of VCPUs should therefore translate to the amount of cores, not CPUs. Diffs ----- plugins/hypervisors/kvm/src/com/cloud/hypervisor/kvm/resource/LibvirtVMDef.java 5120870 Diff: https://reviews.apache.org/r/12679/diff/ Testing ------- Only tested the output of the append rule. Thanks, Rene Diepstraten