Re: Review Request 13441: CLOUDSTACK-3850: CPU cap should be per VM not per VCPU

2013-08-11 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13441/#review24974 --- 'mvn clean install' on 4.2 branch resulted in build failure. XS

Re: Review Request 13441: CLOUDSTACK-3850: CPU cap should be per VM not per VCPU

2013-08-10 Thread Koushik Das
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13441/#review24968 ---

Re: Review Request 13441: CLOUDSTACK-3850: CPU cap should be per VM not per VCPU

2013-08-10 Thread Harikrishna Patnala
On Aug. 10, 2013, 1:11 p.m., Koushik Das wrote: plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java, line 663 https://reviews.apache.org/r/13441/diff/1/?file=339183#file339183line663 CPU weight computation is not considering the number of CPUs in

Re: Review Request 13441: CLOUDSTACK-3850: CPU cap should be per VM not per VCPU

2013-08-10 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13441/ --- (Updated Aug. 10, 2013, 7:08 p.m.) Review request for cloudstack and Koushik

Review Request 13441: CLOUDSTACK-3850: CPU cap should be per VM not per VCPU

2013-08-09 Thread Harikrishna Patnala
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13441/ --- Review request for cloudstack and Koushik Das. Bugs: CLOUDSTACK-3850