----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12937/#review24081 -----------------------------------------------------------
engine/schema/src/com/cloud/upgrade/dao/Upgrade410to420.java <https://reviews.apache.org/r/12937/#comment47925> Why do u need the java code for this ? This could be done by the sql query right server/src/com/cloud/server/ManagementServerImpl.java <https://reviews.apache.org/r/12937/#comment47927> Why is this required ? - Nitin Mehta On July 26, 2013, 8:41 a.m., Harikrishna Patnala wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12937/ > ----------------------------------------------------------- > > (Updated July 26, 2013, 8:41 a.m.) > > > Review request for cloudstack and Nitin Mehta. > > > Bugs: CLOUDSTACK-3637 and CLOUDSTACK-3779 > > > Repository: cloudstack-git > > > Description > ------- > > CLOUDSTACK-3779: Support System vm scaling on VMWare > > Marked the system template new system template as dynamicallyScalable > - handled upgrade case > - moved "dynamicallyScalable" flag to vm_instance table from > user_vm_details to support dynamic scaling of system vm > > This patch depends on previous review request > https://reviews.apache.org/r/12759/ . please review and commit accordingly > > > Diffs > ----- > > engine/schema/src/com/cloud/upgrade/dao/Upgrade410to420.java d4b7b6d > engine/schema/src/com/cloud/vm/VMInstanceVO.java e1d2892 > engine/schema/src/com/cloud/vm/dao/UserVmDao.java b4f9991 > engine/schema/src/com/cloud/vm/dao/UserVmDaoImpl.java 1c11563 > > plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer56FP1Resource.java > 2cc592d > server/src/com/cloud/hypervisor/HypervisorGuruBase.java 6d368bd > server/src/com/cloud/server/ManagementServerImpl.java 7ac1254 > server/src/com/cloud/vm/UserVmManagerImpl.java ae12523 > server/test/com/cloud/vm/VirtualMachineManagerImplTest.java 44a703d > setup/db/db/schema-410to420.sql 82ca403 > > Diff: https://reviews.apache.org/r/12937/diff/ > > > Testing > ------- > > > Thanks, > > Harikrishna Patnala > >
