Shipped with commit ffe90c00599917bfbe5e22e369fd248358074600 From: Nitin Mehta <nitin.me...@citrix.com<mailto:nitin.me...@citrix.com>> Reply-To: Nitin Mehta <nitin.me...@citrix.com<mailto:nitin.me...@citrix.com>> Date: Wednesday 15 May 2013 3:49 PM To: Abhinandan Prateek <aprat...@apache.org<mailto:aprat...@apache.org>>, Nitin Mehta <nitin.me...@citrix.com<mailto:nitin.me...@citrix.com>> Cc: "cloudstack-...@incubator.apache.org<mailto:cloudstack-...@incubator.apache.org>" <cloudstack-...@incubator.apache.org<mailto:cloudstack-...@incubator.apache.org>>, Harikrishna Patnala <harikrishna.patn...@citrix.com<mailto:harikrishna.patn...@citrix.com>> Subject: Re: Review Request: Fix for 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/10903/ Ship it! Ship It! - Nitin On May 15th, 2013, 10:06 a.m., Harikrishna Patnala wrote: Review request for cloudstack, Abhinandan Prateek and Nitin Mehta. By Harikrishna Patnala. Updated May 15, 2013, 10:06 a.m. 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 Fixed by updating the patch files that has entries to copy scipts on xenserver. Here we added Add-To-VCPUs-Params-Live.sh Added a check on Host params whether host restricts Dynamic memory control(DMC) to able to allow scale up VM. If DMC is not enabled then static max and min are set to SO. Testing Tested: tried scaling up a vm and checked on xenserver for the new values. Bugs: CLOUDSTACK-2085 Diffs * plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/CitrixResourceBase.java (46ae35a) * plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer56FP1Resource.java (96a90a6) * plugins/hypervisors/xen/test/com/cloud/hypervisor/xen/resource/CitrixResourceBaseTest.java (7392cb1) * scripts/vm/hypervisor/xenserver/Add-To-VCPUs-Params-Live.sh (0fadcd8) * scripts/vm/hypervisor/xenserver/add_to_vcpus_params_live.sh (PRE-CREATION) * scripts/vm/hypervisor/xenserver/vmops (30b5300) * 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) View Diff<https://reviews.apache.org/r/10903/diff/>
