----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21380/ -----------------------------------------------------------
Review request for cloudstack and Girish Shilamkar. Bugs: CLOUDSTACK-6618 https://issues.apache.org/jira/browse/CLOUDSTACK-6618 Repository: cloudstack-git Description ------- Fixing issues in SSHClient and config. 1) Public port for natrule was 2222, it should be 22. 2) RetriesCount and delay in sshClient was defaulted to 0, it should be read from incoming parameters. 3) Indetation of "return" statement was wrong, due to which it was returning in first attempt itself without going for maximum no. of retry counts. Diffs ----- test/integration/smoke/test_network.py 6beb071 tools/marvin/marvin/config/test_data.py 7001c42 tools/marvin/marvin/sshClient.py 76f3b6d Diff: https://reviews.apache.org/r/21380/diff/ Testing ------- Yes. Test for port forwarding on source NAT ... === TestName: test_01_port_fwd_on_src_nat | Status : SUCCESS === ok Test for port forwarding on non source NAT ... SKIP: skip ---------------------------------------------------------------------- Ran 2 tests in 1922.008s OK (SKIP=1) Thanks, Gaurav Aradhye