cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "SrikanteswaraRao Talluri" <srikanteswararao.tall...@citrix.com>
Subject Re: Review Request 25187: test_delete_account and test_releaseIP failing in advanced zone
Date Tue, 16 Sep 2014 11:47:03 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/25187/#review53510
-----------------------------------------------------------


Commit 06b96d1ccf85455587d5098402de14c2930328f0 in cloudstack's branch refs/heads/master

Please mark this as submitted.

- SrikanteswaraRao Talluri


On Sept. 3, 2014, 3:17 p.m., Alex Brett wrote:
> 
> -----------------------------------------------------------
> 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
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message