-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25187/
-----------------------------------------------------------
(Updated Sept. 3, 2014, 3:17 p.m.)
Review request for cloudstack, Ashutosh Kelkar, Santhosh Edukulla, and
SrikanteswaraRao Talluri.
Bugs: CLOUDSTACK-7448
https://issues.apache.org/jira/browse/CLOUDSTACK-7448
Repository: cloudstack-git
Description
-------
CLOUDSTACK-4840 changed test_data.py to make the lbrule publicport be 22,
instead of 2222. In doing so, this caused the following tests to fail, as they
hit a problem where they tried to use port 22 for both the lbrule and for other
purposes:
integration.smoke.test_network.TestDeleteAccount.test_delete_account
integration.smoke.test_network.TestReleaseIP.test_releaseIP
The reason the change appears to have been made was that in
test_lb_secondary_ip.py, despite setting up the load balancer using lbrule, the
tests then used the SSH port from natrule to try and access the VM. By changing
lbrule to use port 22 (the same as natrule) this avoided the problem.
This patch updates test_lb_secondary_ip.py to use the SSH port in lbrule where
necessary to access the VMs, and reverts the change to test_data.py
Diffs
-----
test/integration/component/test_lb_secondary_ip.py f54caa6
tools/marvin/marvin/config/test_data.py fca2442
Diff: https://reviews.apache.org/r/25187/diff/
Testing
-------
Ran tests in test_lb_secondary_ip.py with changes in place, verified results
equivalent to those seen on regular regression runs.
Ran test_delete_account and test_releaseIP to verify these now pass as expected.
Thanks,
Alex Brett
