-----------------------------------------------------------
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