----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15341/#review28506 -----------------------------------------------------------
i can't see the diff - SrikanteswaraRao Talluri On Nov. 8, 2013, 7:01 a.m., Santhosh Edukulla wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/15341/ > ----------------------------------------------------------- > > (Updated Nov. 8, 2013, 7:01 a.m.) > > > Review request for cloudstack and SrikanteswaraRao Talluri. > > > Bugs: CLOUDSTACK-5099 > https://issues.apache.org/jira/browse/CLOUDSTACK-5099 > > > Repository: cloudstack-git > > > Description > ------- > > 1. Utils.py has wrong reference for keyword argument, keyPairFilesLocation > 2. We followed uniform naming convention of Camel casing for classes but here > its not. Changed it. > 3. Renaming remotesshClient.py to sshClient.py, it is more apt name ssh can > be to local as well remote > > > Diffs > ----- > > > Diff: https://reviews.apache.org/r/15341/diff/ > > > Testing > ------- > > > Thanks, > > Santhosh Edukulla > >