-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20621/
-----------------------------------------------------------

(Updated May 23, 2014, 12:39 p.m.)


Review request for cloudstack and Abhinandan Prateek.


Changes
-------

Added the correct patch.


Bugs: CLOUDSTACK-6529
    https://issues.apache.org/jira/browse/CLOUDSTACK-6529


Repository: cloudstack-git


Description (updated)
-------

1. In deployDataCenter when a host addition fails, it is not continuing 
further. Added a change, to continue adding all hosts, even if any addition 
fails. Do a cleanup only when all host addition fails. Tested the changes on 
local setup 

2. Removed another unwanted arguments from class host which are not part of 
addHost call.
3. Added an exit check based upon correct input file path type.


Diffs (updated)
-----

  tools/marvin/marvin/configGenerator.py 66609da 
  tools/marvin/marvin/deployDataCenter.py 4d8be58 

Diff: https://reviews.apache.org/r/20621/diff/


Testing
-------

Tested the changes that it is not breaking existing deployDataCenter.


Thanks,

Santhosh Edukulla

Reply via email to