cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "David Nalley" <da...@gnsa.us>
Subject Re: Review Request 12679: CLOUDSTACK-904: Changed multiple vcpus to one vcpu with multiple sockets
Date Thu, 18 Jul 2013 16:06:39 GMT

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


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