ligang/vpp
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fib: fix sa selection for fib routed destinations

Browse Source 
The move from ip4(6)_src_address_for_packet to fib_sas4(6)_get changed
the behavior, so that the new looked only to adjacent gleans. This
caused a problem for destinations routed according to FIB table.

To reproduce:
vpp# create tap
vpp# set interface state tap0 up
vpp# set interface ip address tap0 192.168.11.1/24
vpp# ip route add 192.168.20.0/24 via 192.168.11.2

linux$ sudo ip addr add 192.168.20.1/24 dev lo
linux$ sudo ip link set tap0 up
linux$ sudo ip addr add 192.168.11.2/24 dev tap0

vpp# ping 192.168.20.1
Failed: no source address for egress interface

Type: fix
Signed-off-by: Július Milan <julius.milan@pantheon.tech>
Signed-off-by: Neale Ranns <neale@graphiant.com>
Change-Id: I22899f4dbbf8c1c85ccce72f801b92c183195b5d
This commit is contained in:
Július Milan
2021-02-16 19:20:47 +01:00
committed by Beno�t Ganne
parent cb1c4902bb
commit 98874cda58
2 changed files with 39 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

26
test/test_ping.py
View File

@ -7,7 +7,7 @@ from scapy.packet import Raw
from framework import VppTestCase
from util import ppp
from vpp_ip_route import VppIpInterfaceAddress
from vpp_ip_route import VppIpInterfaceAddress, VppIpRoute, VppRoutePath
from vpp_neighbor import VppNeighbor
""" TestPing is a subclass of VPPTestCase classes.
@ -150,3 +150,27 @@ class TestPing(VppTestCase):
for p in out:
icmp = self.verify_ping_request(p, "10.0.0.1", nbr_addr, icmp_seq)
icmp_seq = icmp_seq + 1
def test_ping_fib_routed_dst(self):
""" ping destination routed according to FIB table """
try:
self.pg1.generate_remote_hosts(1)
self.pg_enable_capture(self.pg_interfaces)
self.pg_start()
routed_dst = "10.0.2.0"
self.logger.info(self.vapi.cli("show ip4 neighbors"))
VppIpRoute(self, routed_dst, 24,
[VppRoutePath(self.pg1.remote_hosts[0].ip4,
self.pg1.sw_if_index)]).add_vpp_config()
ping_cmd = "ping %s interval 0.01 repeat 3" % routed_dst
ret = self.vapi.cli(ping_cmd)
self.logger.info(ret)
out = self.pg1.get_capture(3)
icmp_seq = 1
for p in out:
self.verify_ping_request(p, self.pg1.local_ip4, routed_dst,
icmp_seq)
icmp_seq = icmp_seq + 1
finally:
self.vapi.cli("show error")