----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12937/ -----------------------------------------------------------
(Updated July 25, 2013, 10:44 a.m.) Review request for cloudstack and Nitin Mehta. Changes ------- Included check for system vm on xenserver and throwing an exception accordingly Bugs: 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 (updated) ----- 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
