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

Reply via email to