aboutsummaryrefslogtreecommitdiffstats
path: root/sfc/tests/functest/sfc_symmetric_chain.py
diff options
context:
space:
mode:
authorJuan Vidal <juan.vidal.allende@ericsson.com>2017-03-24 15:16:21 +0000
committerJuan Vidal <juan.vidal.allende@ericsson.com>2017-04-05 09:54:53 +0000
commit8a66eac4bbb898d943d461845e371381425a9cf7 (patch)
tree3439ef943c73f78fc5f7b137a665c922a28a2399 /sfc/tests/functest/sfc_symmetric_chain.py
parent6d6d321145befb9f823a9d8af40e5dce83f0dbc7 (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.py5
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')