----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14648/#review27279 -----------------------------------------------------------
This looks fine except pep8 is broken in a few places. Can you please run pep8 *.py in tools/marvin/marvin to ensure the formatting is maintained? Please upload the patch that corrects these issues. - Prasanna Santhanam On Oct. 15, 2013, 12:10 p.m., Santhosh Edukulla wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/14648/ > ----------------------------------------------------------- > > (Updated Oct. 15, 2013, 12:10 p.m.) > > > Review request for cloudstack and Prasanna Santhanam. > > > Repository: cloudstack-git > > > Description > ------- > > Added few naming convention changes--minor. > > > Diffs > ----- > > tools/marvin/marvin/deployDataCenter.py beed8c8 > > Diff: https://reviews.apache.org/r/14648/diff/ > > > Testing > ------- > > > Thanks, > > Santhosh Edukulla > >
