----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17747/#review64041 -----------------------------------------------------------
Hari, your fix is already on 4.3; but not on 4.4, 4.5 or master. Can you please see that? I see a patch from Daan on 4.4, 4.5 and master: 0839fbce67963578a18478f90e5fbcfd7be11912 Please advise changes if any on all these branches. Thanks. - Rohit Yadav On Oct. 9, 2014, 9:27 a.m., Harikrishna Patnala wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/17747/ > ----------------------------------------------------------- > > (Updated Oct. 9, 2014, 9:27 a.m.) > > > Review request for cloudstack, Abhinandan Prateek, Kishan Kavala, and Nitin > Mehta. > > > Bugs: CLOUDSTACK-6023 > https://issues.apache.org/jira/browse/CLOUDSTACK-6023 > > > Repository: cloudstack-git > > > Description > ------- > > CLOUDSTACK-6023: Non windows instances are created on XenServer with a > vcpu-max above supported xenserver limits > > Changed the VCPU max limit to 16 and provided a cluster level configuration > parameter for this max limit named xen.vm.vcpu.max > > > Diffs > ----- > > api/src/com/cloud/agent/api/to/VirtualMachineTO.java 4b36df5 > plugins/hypervisors/xenserver/src/com/cloud/hypervisor/XenServerGuru.java > c1de8bb > > plugins/hypervisors/xenserver/src/com/cloud/hypervisor/xenserver/resource/CitrixResourceBase.java > e3c3943 > > Diff: https://reviews.apache.org/r/17747/diff/ > > > Testing > ------- > > > Thanks, > > Harikrishna Patnala > >
