----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12358/ -----------------------------------------------------------
(Updated July 9, 2013, 3:02 p.m.) Review request for cloudstack, Abhinandan Prateek and Nitin Mehta. Changes ------- Changed the way of getting right template. Instead of shuffling the hypervisor list, we make a try on each hypervisor until it gets a right template. Bugs: CLOUDSTACK-3228 Repository: cloudstack-git Description ------- 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 Shuffling the hypervisors list before getting the first hypervisor in list so that on every retry of system vm deployment it chooses different hypervisor and checking whether template is downloaded or not. Diffs (updated) ----- engine/schema/src/com/cloud/storage/dao/VMTemplateDaoImpl.java 9e75990 plugins/hypervisors/xen/src/com/cloud/hypervisor/xen/resource/XenServer56FP1Resource.java 407f069 server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java 5983aa7 server/src/com/cloud/resource/ResourceManager.java e35e89a server/src/com/cloud/resource/ResourceManagerImpl.java 41c6ad7 server/src/com/cloud/storage/secondary/SecondaryStorageManagerImpl.java 05256a8 server/test/com/cloud/resource/MockResourceManagerImpl.java 18cff80 Diff: https://reviews.apache.org/r/12358/diff/ Testing ------- tested locally Thanks, Harikrishna Patnala