diff options
| author |   Neale Ranns <neale@graphiant.com> | 2021-08-13 08:10:59 +0000 |
|---|---|---|
| committer | Neale Ranns <neale@graphiant.com> | 2021-09-13 08:36:08 +0000 |
| commit | ea8adf76375389e4c46c1a2eb903743142e44a8d (patch) | |
| tree | f11a5664938575cf934e4369ae14984966ab8c7e /test/test_neighbor.py | |
| parent | cba87100f1dd6d8b07eab7d9914bb80d469b982f (diff) | |
ip-neighbor: Handle local MAC address change for incomplete adjacencies
Type: fix
When the local MAC address of an interface changes the rewrite strings
of all adjacency types need to be updated - this patch fixes the missing
case of incomplete adjacencies.
I moved the update of all adj types into the adj module, since if the
complete adjs were done by the ip-neighbour module and incomplete ones
by adj module, that would mean two walks of the adj DB, as it is not
possible to walk only a specific type.
UT is updated to include the missing case.
Signed-off-by: Neale Ranns <neale@graphiant.com>
Signed-off-by: Ivan Shvedunov <ivan4th@gmail.com>
Change-Id: I36af94976c645bdd0d4d3bc0093b24d7d077e9d7
Diffstat (limited to 'test/test_neighbor.py')
| -rw-r--r-- | test/test_neighbor.py | 24 |
1 files changed, 23 insertions, 1 deletions
diff --git a/test/test_neighbor.py b/test/test_neighbor.py index b33a70b2a3c..e893085cfdc 100644 --- a/test/test_neighbor.py +++ b/test/test_neighbor.py @@ -1372,7 +1372,7 @@ class ARPTestCase(VppTestCase): def test_arp_incomplete(self): """ ARP Incomplete""" - self.pg1.generate_remote_hosts(3) + self.pg1.generate_remote_hosts(4) p0 = (Ether(dst=self.pg0.local_mac, src=self.pg0.remote_mac) / IP(src=self.pg0.remote_ip4, @@ -1384,6 +1384,11 @@ class ARPTestCase(VppTestCase): dst=self.pg1.remote_hosts[2].ip4) / UDP(sport=1234, dport=1234) / Raw()) + p2 = (Ether(dst=self.pg0.local_mac, src=self.pg0.remote_mac) / + IP(src=self.pg0.remote_ip4, + dst="1.1.1.1") / + UDP(sport=1234, dport=1234) / + Raw()) # # a packet to an unresolved destination generates an ARP request @@ -1405,6 +1410,18 @@ class ARPTestCase(VppTestCase): static_arp.add_vpp_config() # + # add a route through remote host 3 hence we get an incomplete + # + VppIpRoute(self, "1.1.1.1", 32, + [VppRoutePath(self.pg1.remote_hosts[3].ip4, + self.pg1.sw_if_index)]).add_vpp_config() + rx = self.send_and_expect(self.pg0, [p2], self.pg1) + self.verify_arp_req(rx[0], + self.pg1.local_mac, + self.pg1.local_ip4, + self.pg1._remote_hosts[3].ip4) + + # # change the interface's MAC # self.vapi.sw_interface_set_mac_address(self.pg1.sw_if_index, @@ -1418,6 +1435,11 @@ class ARPTestCase(VppTestCase): "00:00:00:33:33:33", self.pg1.local_ip4, self.pg1._remote_hosts[2].ip4) + rx = self.send_and_expect(self.pg0, [p2], self.pg1) + self.verify_arp_req(rx[0], + "00:00:00:33:33:33", + self.pg1.local_ip4, + self.pg1._remote_hosts[3].ip4) # # packets to the resolved host also have the new source mac |