Re: Review Request: CLOUDSTACK-2085: VM weight on xen remain same as before vmscaleup
--- 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
--- 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
--- 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