diff options
| author |   Neale Ranns <neale@graphiant.com> | 2022-02-17 09:08:47 +0000 |
|---|---|---|
| committer |   Ole Tr�an <otroan@employees.org> | 2022-03-04 16:14:11 +0000 |
| commit | 5c6dd17a373a2c56e57f02426d66a79af7faa19c (patch) | |
| tree | 154930db914285fda1636eb47faef2b1275e0190 /test/test_mpls.py | |
| parent | bc91e86674d446e024a957318d42a3bbd3280bf1 (diff) | |
ip: rate-limit the sending of ICMP error messages
Type: improvement
For error conditions, such as TTL expired, dest unreach, etc, Rate limit the sending of ICMP error messages.
The rate limiting is done based on src,dst IP address of the received packet.
the rate limit has been chosen, somewhat arbitrarily, to be 1e-3. This is the same limit as the ARP throttling.
Signed-off-by: Neale Ranns <neale@graphiant.com>
Change-Id: I4a0b791cde8c941a9bf37de6aa5da56779d3cef4
Diffstat (limited to 'test/test_mpls.py')
| -rw-r--r-- | test/test_mpls.py | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/test/test_mpls.py b/test/test_mpls.py index 34645a9fce1..f7709d10cc1 100644 --- a/test/test_mpls.py +++ b/test/test_mpls.py @@ -366,7 +366,8 @@ class TestMPLS(VppTestCase): def verify_capture_ip6_icmp(self, src_if, capture, sent): try: - self.assertEqual(len(capture), len(sent)) + # rate limited ICMP + self.assertTrue(len(capture) <= len(sent)) for i in range(len(capture)): tx = sent[i] @@ -549,7 +550,7 @@ class TestMPLS(VppTestCase): [VppMplsLabel(333, ttl=64)], dst_ip=self.pg1.remote_ip6, hlim=1) - rx = self.send_and_expect(self.pg0, tx, self.pg0) + rx = self.send_and_expect_some(self.pg0, tx, self.pg0) self.verify_capture_ip6_icmp(self.pg0, rx, tx) # @@ -591,7 +592,7 @@ class TestMPLS(VppTestCase): tx = self.create_stream_labelled_ip6(self.pg0, [VppMplsLabel(334)], dst_ip=self.pg1.remote_ip6, hlim=0) - rx = self.send_and_expect(self.pg0, tx, self.pg0) + rx = self.send_and_expect_some(self.pg0, tx, self.pg0) self.verify_capture_ip6_icmp(self.pg0, rx, tx) # @@ -1500,7 +1501,7 @@ class TestMPLS(VppTestCase): [VppMplsLabel(34)], dst_ip="ff01::1", hlim=1) - rx = self.send_and_expect(self.pg0, tx, self.pg0) + rx = self.send_and_expect_some(self.pg0, tx, self.pg0) self.verify_capture_ip6_icmp(self.pg0, rx, tx) # |