Hi Nitin, This patch is adds the missing changes in 4.3 and master which were there earlier in 4.2. please refer to the commit fa6da31e157c96e88c470209f7ab0e8659b4c4d5 in 4.2.
Thanks, Bharat. On 12-Dec-2013, at 11:18 am, Nitin Mehta <[email protected]<mailto:[email protected]>> wrote: Hi Bharat - Can you please still answer the question here ? From: bharat kumar <[email protected]<mailto:[email protected]>> Reply-To: Bharat Kumar <[email protected]<mailto:[email protected]>> Date: Monday 9 December 2013 9:39 PM To: Jayapal Reddy Uradi <[email protected]<mailto:[email protected]>> Cc: Bharat Kumar <[email protected]<mailto:[email protected]>>, Nitin Mehta <[email protected]<mailto:[email protected]>>, cloudstack <[email protected]<mailto:[email protected]>> Subject: Re: Review Request 16122: CLOUDSTACK-4498 cherry picked from 4.2 This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16122/ On December 9th, 2013, 6:03 p.m. UTC, Nitin Mehta wrote: plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java<https://reviews.apache.org/r/16122/diff/2/?file=395664#file395664line3007>(Diff revision 2) protected StartAnswer execute(StartCommand cmd) { 3003 return (int) (vmSpec.getMinRam() / (1024 * 1024)); Why are you reserving if the flag is false ? Hi Nitin, this patch is not related to the https://reviews.apache.org/r/15349/. will upload it once this gets shipped. - bharat On December 9th, 2013, 10:39 a.m. UTC, bharat kumar wrote: Review request for cloudstack and Jayapal Reddy. By bharat kumar. Updated Dec. 9, 2013, 10:39 a.m. Bugs: CLOUDSTACK-4498<https://issues.apache.org/jira/browse/CLOUDSTACK-4498> Repository: cloudstack-git Description CLOUDSTACK-4498 we should not reserve memory and cpu for vmware VMs if the vmware.reserve.cpu and vmware.reserve.mem are set to false. Testing tested on master. Diffs * plugins/hypervisors/vmware/src/com/cloud/hypervisor/guru/VMwareGuru.java (eb3cb6b) * plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/manager/VmwareManagerImpl.java (d306b74) * plugins/hypervisors/vmware/src/com/cloud/hypervisor/vmware/resource/VmwareResource.java (69f1b74) View Diff<https://reviews.apache.org/r/16122/diff/>
