Re: Review Request: CLOUDSTACK-2085: VM weight on xen remain same as before vmscaleup

2013-05-20 Thread ASF Subversion and Git Services

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10881/#review20758
---


Commit 8da7ec05f1c7dae0aab827ae0568e120111c6579 in branch refs/heads/master 
from Prasanna Santhanam 
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=8da7ec0 ]

CLOUDSTACK-2085: DMC enable guest scaling

When dmc is enabled allow cloudstack to scale the VM between dynamic
ranges. This requires a further commit where there is enough memory
available between dynamic-max and static-max for the VM to scale under
memory pressure. See the TODO in Xenserver56FP1Resource.java

Signed-off-by: Prasanna Santhanam 


- ASF Subversion and Git Services


On May 1, 2013, 10:18 a.m., Harikrishna Patnala wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10881/
> ---
> 
> (Updated May 1, 2013, 10:18 a.m.)
> 
> 
> Review request for cloudstack, Abhinandan Prateek and Nitin Mehta.
> 
> 
> Description
> ---
> 
> CLOUDSTACK-2085: VM weight on xen remain same as before vmscaleup ;because 
> "Add-To-VCPUs-Params-Live.sh" is not getting copied on xs host 
> Fix: created entry for script "Add-To-VCPUs-Params-Live.sh" in the patch 
> files that has entries to copy scipts on xenserver.
> 
> 
> This addresses bug CLOUDSTACK-2085.
> 
> 
> Diffs
> -
> 
>   
> plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java
>  46ae35a 
>   scripts/vm/hypervisor/xenserver/xcpserver/patch b7961bb 
>   scripts/vm/hypervisor/xenserver/xenserver56/patch 36dba3d 
>   scripts/vm/hypervisor/xenserver/xenserver56fp1/patch d20e60f 
>   scripts/vm/hypervisor/xenserver/xenserver60/patch c9125f4 
> 
> Diff: https://reviews.apache.org/r/10881/diff/
> 
> 
> Testing
> ---
> 
> Tested locally by scaling up the vm and checked that VCPU param got updated 
> with new weight. 
> 
> 
> Thanks,
> 
> Harikrishna Patnala
> 
>



Re: Review Request: CLOUDSTACK-2085: VM weight on xen remain same as before vmscaleup

2013-05-01 Thread Abhinandan Prateek

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10881/#review20002
---


Double check on the Xen versions/license that could support this feature for 
rest you need to flag an error.

- Abhinandan Prateek


On May 1, 2013, 10:18 a.m., Harikrishna Patnala wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/10881/
> ---
> 
> (Updated May 1, 2013, 10:18 a.m.)
> 
> 
> Review request for cloudstack, Abhinandan Prateek and Nitin Mehta.
> 
> 
> Description
> ---
> 
> CLOUDSTACK-2085: VM weight on xen remain same as before vmscaleup ;because 
> "Add-To-VCPUs-Params-Live.sh" is not getting copied on xs host 
> Fix: created entry for script "Add-To-VCPUs-Params-Live.sh" in the patch 
> files that has entries to copy scipts on xenserver.
> 
> 
> This addresses bug CLOUDSTACK-2085.
> 
> 
> Diffs
> -
> 
>   
> plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java
>  46ae35a 
>   scripts/vm/hypervisor/xenserver/xcpserver/patch b7961bb 
>   scripts/vm/hypervisor/xenserver/xenserver56/patch 36dba3d 
>   scripts/vm/hypervisor/xenserver/xenserver56fp1/patch d20e60f 
>   scripts/vm/hypervisor/xenserver/xenserver60/patch c9125f4 
> 
> Diff: https://reviews.apache.org/r/10881/diff/
> 
> 
> Testing
> ---
> 
> Tested locally by scaling up the vm and checked that VCPU param got updated 
> with new weight. 
> 
> 
> Thanks,
> 
> Harikrishna Patnala
> 
>



Review Request: CLOUDSTACK-2085: VM weight on xen remain same as before vmscaleup

2013-05-01 Thread Harikrishna Patnala

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/10881/
---

Review request for cloudstack, Abhinandan Prateek and Nitin Mehta.


Description
---

CLOUDSTACK-2085: VM weight on xen remain same as before vmscaleup ;because 
"Add-To-VCPUs-Params-Live.sh" is not getting copied on xs host 
Fix: created entry for script "Add-To-VCPUs-Params-Live.sh" in the patch files 
that has entries to copy scipts on xenserver.


This addresses bug CLOUDSTACK-2085.


Diffs
-

  
plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java
 46ae35a 
  scripts/vm/hypervisor/xenserver/xcpserver/patch b7961bb 
  scripts/vm/hypervisor/xenserver/xenserver56/patch 36dba3d 
  scripts/vm/hypervisor/xenserver/xenserver56fp1/patch d20e60f 
  scripts/vm/hypervisor/xenserver/xenserver60/patch c9125f4 

Diff: https://reviews.apache.org/r/10881/diff/


Testing
---

Tested locally by scaling up the vm and checked that VCPU param got updated 
with new weight. 


Thanks,

Harikrishna Patnala