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 27985: CLOUDSTACK-7860: test_eip_elb.py - Move Netscler info out of the test case. Read it from config. Fix attribute error. Fix pep8 issues. Fix imports.
Date Thu, 13 Nov 2014 12:49:26 GMT

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



test/integration/component/test_eip_elb.py
<https://reviews.apache.org/r/27985/#comment102773>

    As we are not using this flag,  can we remove this?


- SrikanteswaraRao Talluri


On Nov. 13, 2014, 12:35 p.m., Gaurav Aradhye wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/27985/
> -----------------------------------------------------------
> 
> (Updated Nov. 13, 2014, 12:35 p.m.)
> 
> 
> Review request for cloudstack and SrikanteswaraRao Talluri.
> 
> 
> Bugs: CLOUDSTACK-7860
>     https://issues.apache.org/jira/browse/CLOUDSTACK-7860
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> Move netscaler info out of the test case. Read it from config file.
> 
> Other changes:
> 1. Fix attribute errors
> 2. Fix pep8 issues
> 3. Fix imports
> 
> 
> Diffs
> -----
> 
>   test/integration/component/test_eip_elb.py c35ea9b 
> 
> Diff: https://reviews.apache.org/r/27985/diff/
> 
> 
> Testing
> -------
> 
> Yes. 4 test cases failed out of 9. Separate analysis should be done for these becase
these are assertion errors.
> 
> Log:
> Test EIP by deploying an instance ... === TestName: test_01_eip_by_deploying_instance
| Status : FAILED ===
> FAIL
> Test associate new IP and enable static NAT for new IP and the VM ... === TestName: test_02_acquire_ip_enable_static_nat
| Status : SUCCESS ===
> ok
> Test disable static NAT and release EIP acquired ... === TestName: test_03_disable_static_nat
| Status : FAILED ===
> FAIL
> Test disable static NAT with system = True ... === TestName: test_04_disable_static_nat_system
| Status : SUCCESS ===
> ok
> Test EIO after destroying instance ... === TestName: test_05_destroy_instance | Status
: FAILED ===
> FAIL
> Test ELB by creating a LB rule ... === TestName: test_01_elb_create | Status : SUCCESS
===
> ok
> Test ELB by acquiring IP and then creating a LB rule ... === TestName: test_02_elb_acquire_and_create
| Status : SUCCESS ===
> ok
> Test delete LB rule generated with public IP with is_system = 1 ... === TestName: test_03_elb_delete_lb_system
| Status : SUCCESS ===
> ok
> Test delete LB rule generated on EIP ... === TestName: test_04_delete_lb_on_eip | Status
: FAILED ===
> FAIL
> 
> 
> Failure details:
> 2014-11-12 23:51:58,884 - CRITICAL - FAILED: test_01_eip_by_deploying_instance: ['Traceback
(most recent call last):\n', '  File "/usr/local/lib/python2.7/unittest/
> case.py", line 318, in run\n    testMethod()\n', '  File "/root/test/test_eip_elb.py",
line 361, in test_01_eip_by_deploying_instance\n    "User source IP should be
> enabled for INAT service")\n', '  File "/usr/local/lib/python2.7/unittest/case.py", line
494, in assertEqual\n    assertion_func(first, second, msg=msg)\n', '  File "
> usr/local/lib/python2.7/unittest/case.py", line 487, in _baseAssertEqual\n    raise self.failureException(msg)\n',
'AssertionError: User source IP should be enabled
> for INAT service\n']
> 
> 
> 2014-11-12 23:53:20,415 - CRITICAL - FAILED: test_03_disable_static_nat: ['Traceback
(most recent call last):\n', '  File "/usr/local/lib/python2.7/unittest/case.py",
> line 318, in run\n    testMethod()\n', '  File "/root/test/test_eip_elb.py", line 704,
in test_03_disable_static_nat\n    "User source IP should be enabled for INAT
> service"\n', '  File "/usr/local/lib/python2.7/unittest/case.py", line 494, in assertEqual\n
   assertion_func(first, second, msg=msg)\n', '  File "/usr/local/lib/
> python2.7/unittest/case.py", line 487, in _baseAssertEqual\n    raise self.failureException(msg)\n',
'AssertionError: User source IP should be enabled for INAT
> service\n']
> 
> 
> 2014-11-12 23:54:06,560 - CRITICAL - FAILED: test_05_destroy_instance: ['Traceback (most
recent call last):\n', '  File "/usr/local/lib/python2.7/unittest/case.py",
> line 318, in run\n    testMethod()\n', '  File "/root/test/test_eip_elb.py", line 876,
in test_05_destroy_instance\n    "show ip should return nothing after VM
> destroy"\n', '  File "/usr/local/lib/python2.7/unittest/case.py", line 494, in assertEqual\n
   assertion_func(first, second, msg=msg)\n', '  File "/usr/local/lib/
> python2.7/unittest/case.py", line 487, in _baseAssertEqual\n    raise self.failureException(msg)\n',
'AssertionError: show ip should return nothing after VM
> destroy\n']
> 
> 
> 2014-11-12 23:56:24,592 - CRITICAL - FAILED: test_04_delete_lb_on_eip: ['Traceback (most
recent call last):\n', '  File "/usr/local/lib/python2.7/unittest/case.py",
> line 318, in run\n    testMethod()\n', '  File "/root/test/test_eip_elb.py", line 1469,
in test_04_delete_lb_on_eip\n    "List Public IP address should return valid
> IP address for network"\n', '  File "/usr/local/lib/python2.7/unittest/case.py", line
494, in assertEqual\n    assertion_func(first, second, msg=msg)\n', '  File "/
> usr/local/lib/python2.7/unittest/case.py", line 487, in _baseAssertEqual\n    raise self.failureException(msg)\n',
'AssertionError: List Public IP address should
> return valid IP address for network\n']
> 
> 
> Thanks,
> 
> Gaurav Aradhye
> 
>


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