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

2013-08-11 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13441/#review24979 --- Commit 726114a95df0c0525617225e07295b8dff10a8af in branch refs/heads

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/#review24978 --- Ship it! Ship It! - Koushik Das On Aug. 11, 2013, 5:28 p.m., Har

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

2013-08-11 Thread ASF Subversion and Git Services
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13441/#review24977 --- Commit a7dea8eecdee071e09cc90d6e7e47c8b93746bb1 in branch refs/heads

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

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

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 unit

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 Da

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 > > > > > > CPU weight computation is not considering the numbe

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 --- plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/Citri