cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Harikrishna Patnala" <harikrishna.patn...@citrix.com>
Subject Re: Review Request 12358: CLOUDSTACK-3228: system vms are not comming up in zone with two cluster xen and kvm
Date Tue, 09 Jul 2013 15:02:38 GMT

-----------------------------------------------------------
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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message