----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12358/#review22902 -----------------------------------------------------------
server/src/com/cloud/consoleproxy/ConsoleProxyManagerImpl.java <https://reviews.apache.org/r/12358/#comment46650> Though this is a old comment, it is misleading, please provide a better error message. - Abhinandan Prateek On July 9, 2013, 10:34 a.m., Harikrishna Patnala wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12358/ > ----------------------------------------------------------- > > (Updated July 9, 2013, 10:34 a.m.) > > > Review request for cloudstack, Abhinandan Prateek and Nitin Mehta. > > > 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 > ----- > > > 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/ResourceManagerImpl.java 41c6ad7 > server/src/com/cloud/storage/secondary/SecondaryStorageManagerImpl.java > 05256a8 > > Diff: https://reviews.apache.org/r/12358/diff/ > > > Testing > ------- > > tested locally > > > Thanks, > > Harikrishna Patnala > >
