diff options
author | tomsou <soth@intracom-telecom.com> | 2017-03-28 12:20:37 +0000 |
---|---|---|
committer | tomsou <soth@intracom-telecom.com> | 2017-03-28 12:39:15 +0000 |
commit | 0e26e7dfd4ff41ae7c8389218bca138346864922 (patch) | |
tree | 95a0ee4145749f322e95591eb64f250b10e7a93c /sdnvpn/test/functest/testcase_7.py | |
parent | 5960fab7230bebbffc439457d5542d086cb3f8af (diff) |
Fix security group rule conflict
-open_icmp_ssh function, that creates security
group rules for icmp and http (here the name of
the function is misleading), is splitted into 2
new functions one for each rule (open_icmp() and
open_http_port()).
Test cases that used the old function have been
updated
-In the original implementation of open_icmp_ssh()
and open_bgp_port(), the direction parameter was
missing. This was leading to an error message
-A check for the existence of security group rule
has been added and used before the creation of each
security group rule. If a rule already exists,
an info message is printed and the testcase
continues
This change is strongly dependent on https://gerrit.opnfv.org/gerrit/#/c/31925
of Functest repo. NOT TO BE MERGED before
JIRA: SDNVPN-103
Change-Id: Icb96954556f6d7294cf3454f045dbca4b9be672d
Signed-off-by: tomsou <soth@intracom-telecom.com>
Diffstat (limited to 'sdnvpn/test/functest/testcase_7.py')
-rw-r--r-- | sdnvpn/test/functest/testcase_7.py | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/sdnvpn/test/functest/testcase_7.py b/sdnvpn/test/functest/testcase_7.py index 3bc9afb..00e9eef 100644 --- a/sdnvpn/test/functest/testcase_7.py +++ b/sdnvpn/test/functest/testcase_7.py @@ -74,7 +74,9 @@ def main(): sg_id = os_utils.create_security_group_full(neutron_client, TESTCASE_CONFIG.secgroup_name, TESTCASE_CONFIG.secgroup_descr) - test_utils.open_icmp_ssh(neutron_client, sg_id) + test_utils.open_icmp(neutron_client, sg_id) + test_utils.open_http_port(neutron_client, sg_id) + vm_2 = test_utils.create_instance( nova_client, TESTCASE_CONFIG.instance_2_name, |