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