summaryrefslogtreecommitdiffstats
path: root/test/test_cnat.py
diff options
context:
space:
mode:
authorhedi bouattour <hedibouattour2010@gmail.com>2023-09-11 14:48:12 +0000
committerDave Wallace <dwallacelf@gmail.com>2023-10-06 18:18:00 +0000
commit589fe7ca61111d742ed2e6249728eb54423bab16 (patch)
tree609d6d731b766eee6801b0e85ebf47d00a4bdd05 /test/test_cnat.py
parentb953532ed498dbd5c297383afda211104c4596e0 (diff)
cnat: add flow hash config to cnat translation
Type: feature this patch adds a hash config field to cnat translation to use it in load balancing instead of always using default one Change-Id: I5b79642ca8b365b5dcc06664f6c100a9d3830a29 Signed-off-by: hedi bouattour <hedibouattour2010@gmail.com>
Diffstat (limited to 'test/test_cnat.py')
-rw-r--r--test/test_cnat.py92
1 files changed, 90 insertions, 2 deletions
diff --git a/test/test_cnat.py b/test/test_cnat.py
index 096e9f443a9..a7f949db799 100644
--- a/test/test_cnat.py
+++ b/test/test_cnat.py
@@ -110,11 +110,12 @@ class Endpoint(object):
class Translation(VppObject):
- def __init__(self, test, iproto, vip, paths):
+ def __init__(self, test, iproto, vip, paths, fhc):
self._test = test
self.vip = vip
self.iproto = iproto
self.paths = paths
+ self.fhc = fhc
self.id = None
def __str__(self):
@@ -140,6 +141,7 @@ class Translation(VppObject):
"ip_proto": self._vl4_proto(),
"n_paths": len(self.paths),
"paths": self._encoded_paths(),
+ "flow_hash_config": self.fhc,
}
)
self._test.registry.register(self, self._test.logger)
@@ -381,6 +383,41 @@ class TestCNatTranslation(CnatCommonTestCase):
i.admin_down()
super(TestCNatTranslation, self).tearDown()
+ def cnat_fhc_translation(self):
+ """CNat Translation"""
+ self.logger.info(self.vapi.cli("sh cnat client"))
+ self.logger.info(self.vapi.cli("sh cnat translation"))
+
+ for nbr, translation in enumerate(self.mbtranslations):
+ vip = translation.vip
+
+ #
+ # Flows to the VIP with same ips and different source ports are loadbalanced identically
+ # in both cases of flow hash 0x03 (src ip and dst ip) and 0x08 (dst port)
+ #
+ ctx = CnatTestContext(self, translation.iproto, vip.is_v6)
+ for src_pgi, sport in product(range(N_REMOTE_HOSTS), [1234, 1233]):
+ # from client to vip
+ ctx.cnat_send(self.pg0, src_pgi, sport, self.pg1, vip.ip, vip.port)
+ dport1 = ctx.rxs[0][ctx.L4PROTO].dport
+ ctx._test.assertIn(
+ dport1,
+ [translation.paths[0][DST].port, translation.paths[1][DST].port],
+ )
+ ctx.cnat_expect(self.pg0, src_pgi, sport, self.pg1, nbr, dport1)
+
+ ctx.cnat_send(
+ self.pg0, src_pgi, sport + 122, self.pg1, vip.ip, vip.port
+ )
+ dport2 = ctx.rxs[0][ctx.L4PROTO].dport
+ ctx._test.assertIn(
+ dport2,
+ [translation.paths[0][DST].port, translation.paths[1][DST].port],
+ )
+ ctx.cnat_expect(self.pg0, src_pgi, sport + 122, self.pg1, nbr, dport2)
+
+ ctx._test.assertEqual(dport1, dport2)
+
def cnat_translation(self):
"""CNat Translation"""
self.logger.info(self.vapi.cli("sh cnat client"))
@@ -494,6 +531,46 @@ class TestCNatTranslation(CnatCommonTestCase):
ctx.cnat_expect(self.pg0, 0, 1234, self.pg2, 0, vip.port)
ctx.cnat_send_icmp_return_error().cnat_expect_icmp_error_return()
+ def _make_multi_backend_translations(self):
+ self.translations = []
+ self.mbtranslations = []
+ self.mbtranslations.append(
+ Translation(
+ self,
+ TCP,
+ Endpoint(ip="30.0.0.5", port=5555, is_v6=False),
+ [
+ (
+ Endpoint(is_v6=False),
+ Endpoint(pg=self.pg1, pgi=0, port=4001, is_v6=False),
+ ),
+ (
+ Endpoint(is_v6=False),
+ Endpoint(pg=self.pg1, pgi=0, port=4005, is_v6=False),
+ ),
+ ],
+ 0x03, # hash only on dst ip and src ip
+ ).add_vpp_config()
+ )
+ self.mbtranslations.append(
+ Translation(
+ self,
+ TCP,
+ Endpoint(ip="30.0.0.6", port=5555, is_v6=False),
+ [
+ (
+ Endpoint(is_v6=False),
+ Endpoint(pg=self.pg1, pgi=1, port=4006, is_v6=False),
+ ),
+ (
+ Endpoint(is_v6=False),
+ Endpoint(pg=self.pg1, pgi=1, port=4007, is_v6=False),
+ ),
+ ],
+ 0x08, # hash only on dst port
+ ).add_vpp_config()
+ )
+
def _make_translations_v4(self):
self.translations = []
self.translations.append(
@@ -507,6 +584,7 @@ class TestCNatTranslation(CnatCommonTestCase):
Endpoint(pg=self.pg1, pgi=0, port=4001, is_v6=False),
)
],
+ 0x9F,
).add_vpp_config()
)
self.translations.append(
@@ -520,6 +598,7 @@ class TestCNatTranslation(CnatCommonTestCase):
Endpoint(pg=self.pg1, pgi=1, port=4002, is_v6=False),
)
],
+ 0x9F,
).add_vpp_config()
)
self.translations.append(
@@ -533,6 +612,7 @@ class TestCNatTranslation(CnatCommonTestCase):
Endpoint(pg=self.pg1, pgi=2, port=4003, is_v6=False),
)
],
+ 0x9F,
).add_vpp_config()
)
@@ -549,6 +629,7 @@ class TestCNatTranslation(CnatCommonTestCase):
Endpoint(pg=self.pg1, pgi=0, port=4001, is_v6=True),
)
],
+ 0x9F,
).add_vpp_config()
)
self.translations.append(
@@ -562,6 +643,7 @@ class TestCNatTranslation(CnatCommonTestCase):
Endpoint(pg=self.pg1, pgi=1, port=4002, is_v6=True),
)
],
+ 0x9F,
).add_vpp_config()
)
self.translations.append(
@@ -575,6 +657,7 @@ class TestCNatTranslation(CnatCommonTestCase):
Endpoint(pg=self.pg1, pgi=2, port=4003, is_v6=True),
)
],
+ 0x9F,
).add_vpp_config()
)
@@ -598,6 +681,11 @@ class TestCNatTranslation(CnatCommonTestCase):
self._make_translations_v4()
self.cnat_translation()
+ def test_cnat_fhc(self):
+ # """ CNat Translation flow hash config """
+ self._make_multi_backend_translations()
+ self.cnat_fhc_translation()
+
class TestCNatSourceNAT(CnatCommonTestCase):
"""CNat Source NAT"""
@@ -797,7 +885,7 @@ class TestCNatDHCP(CnatCommonTestCase):
(Endpoint(pg=self.pg1, is_v6=is_v6), Endpoint(pg=self.pg3, is_v6=is_v6)),
]
ep = Endpoint(pg=self.pg0, is_v6=is_v6)
- t = Translation(self, TCP, ep, paths).add_vpp_config()
+ t = Translation(self, TCP, ep, paths, 0x9F).add_vpp_config()
# Add an address on every interface
# and check it is reflected in the cnat config
for pg in self.pg_interfaces: