> On July 31, 2012, 9:27 a.m., Prasanna Santhanam wrote: > > Bug Report: http://bugs.cloudstack.org/browse/CS-15776 > > > > Saw quite a few whitespace changes as well. Ran a pychecker on the patch > > and it looks fine. > > > >
Actually the bug report was similar but for a different review. I didn't create one for this review. As for the trailing spaces, I wasn't sure if I should clean that up, but I thought it's a good practice in general to get rid of them. Let me know if I should leave them. - Tomoe ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6227/#review9632 ----------------------------------------------------------- On July 31, 2012, 8:29 a.m., Tomoe Sugihara wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/6227/ > ----------------------------------------------------------- > > (Updated July 31, 2012, 8:29 a.m.) > > > Review request for cloudstack and edison su. > > > Description > ------- > > Fix the agent path according to 7a0a9231c355fee42c67799abe111edcd79998bb > > Signed-off-by: Tomoe Sugihara <to...@midokura.com> > > > Diffs > ----- > > python/lib/cloudutils/serviceConfig.py > 539e26ae7322934de08576918a376c0487157d97 > > Diff: https://reviews.apache.org/r/6227/diff/ > > > Testing > ------- > > > Thanks, > > Tomoe Sugihara > >
