diff options
author | Juan Vidal <juan.vidal.allende@ericsson.com> | 2017-03-24 15:16:21 +0000 |
---|---|---|
committer | Juan Vidal <juan.vidal.allende@ericsson.com> | 2017-04-05 09:54:53 +0000 |
commit | 8a66eac4bbb898d943d461845e371381425a9cf7 (patch) | |
tree | 3439ef943c73f78fc5f7b137a665c922a28a2399 /sfc/tests/functest/sfc_symmetric_chain.py | |
parent | 6d6d321145befb9f823a9d8af40e5dce83f0dbc7 (diff) |
Harmonize vxlan-tool calls
Renamed vxlan_firewall and vxlan_tool_stop to start_vxlan_tool and
stop_vxlan_tool respectively. Added some comments to explain their
behavior.
Cleaned both functions to use ".format()" string formatting, which
improves readability.
Modified behavior of start_vxlan_tool: now it does NOT block
traffic by default
Change-Id: I6754b020a474be1e9adf2d83e7c9f5053930b702
Signed-off-by: Juan Vidal <juan.vidal.allende@ericsson.com>
Diffstat (limited to 'sfc/tests/functest/sfc_symmetric_chain.py')
-rw-r--r-- | sfc/tests/functest/sfc_symmetric_chain.py | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/sfc/tests/functest/sfc_symmetric_chain.py b/sfc/tests/functest/sfc_symmetric_chain.py index 7f58f770..1a699ca3 100644 --- a/sfc/tests/functest/sfc_symmetric_chain.py +++ b/sfc/tests/functest/sfc_symmetric_chain.py @@ -216,7 +216,7 @@ def main(): blocked_port = TESTCASE_CONFIG.blocked_source_port logger.info("Firewall started, blocking traffic port %d" % blocked_port) - test_utils.vxlan_firewall(sf_floating_ip, port=blocked_port) + test_utils.start_vxlan_tool(sf_floating_ip, block=blocked_port) logger.info("Wait for ODL to update the classification rules in OVS") t1.join() @@ -234,7 +234,8 @@ def main(): results.add_to_summary(2, "FAIL", "HTTP works") logger.info("Test if HTTP from port %s is blocked" % blocked_port) - if test_utils.is_http_blocked(client_floating_ip, server_ip, blocked_port): + if test_utils.is_http_blocked( + client_floating_ip, server_ip, blocked_port): results.add_to_summary(2, "PASS", "HTTP Blocked") else: error = ('\033[91mTEST 2 [FAILED] ==> HTTP WORKS\033[0m') |