cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Gaurav Aradhye" <gaurav.arad...@clogeny.com>
Subject Review Request 28635: CLOUDSTACK-8003: test_egress_fw_rules.py - Remove test case dependency on each other and improve cleanup code
Date Wed, 03 Dec 2014 06:01:38 GMT

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

Review request for cloudstack and SrikanteswaraRao Talluri.


Bugs: CLOUDSTACK-8003
    https://issues.apache.org/jira/browse/CLOUDSTACK-8003


Repository: cloudstack-git


Description
-------

Test cases in this test suite use a common account and hence in cleanup operations, we explicitly
delete all the resources created in a test case.

This adds to the running time of the test cases.
Also in case, there is any issue in cleanup operation, all other test cases will get affected.

Hence, it is wise to create a new account for each test cases and delete the account in cleanup.


Diffs
-----

  test/integration/component/test_egress_fw_rules.py 4f95925 

Diff: https://reviews.apache.org/r/28635/diff/


Testing
-------

Yes.

2 test cases failed because db info in my config was wrong.

Test By-default the communication from guest n/w to public n/w is NOT allowed. ... === TestName:
test_01_1_egress_fr1 | Status : SUCCESS ===
ok
Test By-default the communication from guest n/w to public n/w is allowed. ... === TestName:
test_01_egress_fr1 | Status : SUCCESS ===
ok
Test Allow Communication using Egress rule with CIDR + Port Range + Protocol. ... === TestName:
test_02_1_egress_fr2 | Status : SUCCESS ===
ok
Test Allow Communication using Egress rule with CIDR + Port Range + Protocol. ... === TestName:
test_02_egress_fr2 | Status : SUCCESS ===
ok
Test Communication blocked with network that is other than specified ... === TestName: test_03_1_egress_fr3
| Status : EXCEPTION ===
ERROR
Test Communication blocked with network that is other than specified ... === TestName: test_03_egress_fr3
| Status : SUCCESS ===
ok
Test Create Egress rule and check the Firewall_Rules DB table ... === TestName: test_04_1_egress_fr4
| Status : FAILED ===
FAIL
Test Create Egress rule and check the Firewall_Rules DB table ... === TestName: test_04_egress_fr4
| Status : FAILED ===
FAIL
Test Create Egress rule and check the IP tables ... SKIP: Skip
Test Create Egress rule and check the IP tables ... SKIP: Skip
Test Create Egress rule without CIDR ... === TestName: test_06_1_egress_fr6 | Status : SUCCESS
===
ok
Test Create Egress rule without CIDR ... === TestName: test_06_egress_fr6 | Status : SUCCESS
===
ok
Test Create Egress rule without End Port ... === TestName: test_07_1_egress_fr7 | Status :
SUCCESS ===
ok
Test Create Egress rule without End Port ... === TestName: test_07_egress_fr7 | Status : SUCCESS
===
ok
Test Port Forwarding and Egress Conflict ... SKIP: Skip
Test Port Forwarding and Egress Conflict ... SKIP: Skip
Test Delete Egress rule ... === TestName: test_09_1_egress_fr9 | Status : SUCCESS ===
ok
Test Delete Egress rule ... === TestName: test_09_egress_fr9 | Status : SUCCESS ===
ok
Test Invalid CIDR and Invalid Port ranges ... === TestName: test_10_1_egress_fr10 | Status
: SUCCESS ===
ok
Test Invalid CIDR and Invalid Port ranges ... === TestName: test_10_egress_fr10 | Status :
SUCCESS ===
ok
Test Regression on Firewall + PF + LB + SNAT ... === TestName: test_11_1_egress_fr11 | Status
: EXCEPTION ===
ERROR
Test Regression on Firewall + PF + LB + SNAT ... === TestName: test_11_egress_fr11 | Status
: SUCCESS ===
ok
Test Reboot Router ... === TestName: test_12_1_egress_fr12 | Status : SUCCESS ===
ok
Test Reboot Router ... === TestName: test_12_egress_fr12 | Status : SUCCESS ===
ok
Test Redundant Router : Master failover ... === TestName: test_13_1_egress_fr13 | Status :
SUCCESS ===
ok
Test Redundant Router : Master failover ... === TestName: test_13_egress_fr13 | Status : SUCCESS
===
ok


Thanks,

Gaurav Aradhye


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