> On Oct. 22, 2013, 6:37 a.m., Prasanna Santhanam wrote: > > 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.
The changes for this went in with patch# d4072c6 - master. - Santhosh ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14648/#review27279 ----------------------------------------------------------- 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 > >
