----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20621/#review43912 -----------------------------------------------------------
Commit 6f3ff068270f48e00531c462de4f7d76395658ff in cloudstack's branch refs/heads/4.4-forward from santhosh [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=6f3ff06 ] Added fix for CLOUDSTACK-6529 Fixed to continue adding all hosts, even if any addition fails and remove unused args from host class. Signed-off-by: santhosh <santhosh.eduku...@gmail.com> Signed-off-by: Abhinandan Prateek <aprat...@apache.org> - ASF Subversion and Git Services On May 23, 2014, 12:39 p.m., Santhosh Edukulla wrote: > > ----------------------------------------------------------- > 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. > > > Bugs: CLOUDSTACK-6529 > https://issues.apache.org/jira/browse/CLOUDSTACK-6529 > > > Repository: cloudstack-git > > > Description > ------- > > 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 > ----- > > 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 > >
