----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12358/ -----------------------------------------------------------
(Updated July 18, 2013, 9:23 a.m.) Review request for cloudstack and Nitin Mehta. Changes ------- Updated patch based on review comments and included an enhancement on retry mechanism of system vm deployment Bugs: CLOUDSTACK-3228 and CLOUDSTACK-3631 Repository: cloudstack-git Description (updated) ------- CLOUDSTACK-3228: system vms are not comming up in zone with two cluster xen and kvm;Zone host is ready, but secondary storage vm template: 3 is not ready on secondary storage: 2 CLOUDSTACK-3631: Enhance System vm deployment retry mechanism Diffs (updated) ----- engine/schema/src/com/cloud/storage/dao/VMTemplateDao.java c3d44bd engine/schema/src/com/cloud/storage/dao/VMTemplateDaoImpl.java 9e75990 plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer56FP1Resource.java 2cc592d server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java 5983aa7 server/src/com/cloud/storage/secondary/SecondaryStorageManagerImpl.java 6859b0b Diff: https://reviews.apache.org/r/12358/diff/ Testing ------- tested locally Thanks, Harikrishna Patnala