Skip to content

Commit 2f1bd3f

Browse files
committed
tests: add nexthop/interface's down/up topo for kernel routes
Signed-off-by: anlan_cs <[email protected]>
1 parent 4d2ac71 commit 2f1bd3f

File tree

3 files changed

+88
-0
lines changed

3 files changed

+88
-0
lines changed
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
{
2+
"5.5.6.7/32":[
3+
{
4+
"prefix":"5.5.6.7/32",
5+
"prefixLen":32,
6+
"protocol":"kernel",
7+
"vrfName":"default",
8+
"internalFlags":0,
9+
"internalNextHopNum":1,
10+
"internalNextHopActiveNum":0,
11+
"nexthops":[
12+
{
13+
"flags":0,
14+
"interfaceName":"r1-eth2"
15+
}
16+
]
17+
18+
}
19+
]
20+
}
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
1+
{
2+
"5.5.6.7/32":[
3+
{
4+
"prefix":"5.5.6.7/32",
5+
"prefixLen":32,
6+
"protocol":"kernel",
7+
"vrfName":"default",
8+
"internalFlags":8,
9+
"internalNextHopNum":1,
10+
"internalNextHopActiveNum":1,
11+
"nexthops":[
12+
{
13+
"flags":3,
14+
"fib":true,
15+
"interfaceName":"r1-eth2",
16+
"active":true
17+
}
18+
]
19+
}
20+
]
21+
}

tests/topotests/zebra_multiple_connected/test_zebra_multiple_connected.py

+47
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,9 @@ def build_topo(tgen):
6565
switch.add_link(tgen.gears["r2"])
6666
switch.add_link(tgen.gears["r3"])
6767

68+
# Create a p2p connection between r1 and r2
69+
tgen.add_link(tgen.gears["r1"], tgen.gears["r2"])
70+
6871

6972
#####################################################
7073
##
@@ -222,6 +225,50 @@ def test_zebra_kernel_route_blackhole_add():
222225
result, _ = topotest.run_and_expect(test_func, None, count=20, wait=1)
223226
assert result, "Blackhole Route should have not been removed\n{}".format(_)
224227

228+
def test_zebra_kernel_route_interface_linkdown():
229+
"Test that a kernel routes should be affected by interface change"
230+
231+
tgen = get_topogen()
232+
if tgen.routers_have_failure():
233+
pytest.skip(tgen.errors)
234+
235+
router = tgen.gears["r1"]
236+
router.run("ip route add 5.5.6.7/32 via 10.0.1.66 dev r1-eth2")
237+
238+
kernel = "{}/{}/ip_route_kernel_interface_up.json".format(CWD, router.name)
239+
expected = json.loads(open(kernel).read())
240+
241+
test_func = partial(
242+
topotest.router_json_cmp, router, "show ip route 5.5.6.7/32 json", expected
243+
)
244+
result, _ = topotest.run_and_expect(test_func, None, count=20, wait=1)
245+
assert result, "Kernel Route should be selected:\n{}".format(_)
246+
247+
# link down
248+
router2 = tgen.gears["r2"]
249+
router2.run("ip link set dev r2-eth2 down")
250+
251+
kernel = "{}/{}/ip_route_kernel_interface_down.json".format(CWD, router.name)
252+
expected = json.loads(open(kernel).read())
253+
254+
test_func = partial(
255+
topotest.router_json_cmp, router, "show ip route 5.5.6.7/32 json", expected
256+
)
257+
result, _ = topotest.run_and_expect(test_func, None, count=20, wait=1)
258+
assert result, "Kernel Route should not be selected:\n{}".format(_)
259+
260+
# link up
261+
router2 = tgen.gears["r2"]
262+
router2.run("ip link set dev r2-eth2 up")
263+
264+
kernel = "{}/{}/ip_route_kernel_interface_up.json".format(CWD, router.name)
265+
expected = json.loads(open(kernel).read())
266+
267+
test_func = partial(
268+
topotest.router_json_cmp, router, "show ip route 5.5.6.7/32 json", expected
269+
)
270+
result, _ = topotest.run_and_expect(test_func, None, count=20, wait=1)
271+
assert result, "Kernel Route should be selected:\n{}".format(_)
225272

226273
if __name__ == "__main__":
227274
args = ["-s"] + sys.argv[1:]

0 commit comments

Comments
 (0)