Isis srv6 topo1 ping #17848
Open
Isis srv6 topo1 ping #17848
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The ping test does not detect when the command fails. > 2025-01-13 15:38:27,494 INFO: topo: [+] check rt1 fc00:0:9::1 0% packet loss > 2025-01-13 15:38:27,494 WARNING: topo: Waiting time is too small (count=10, wait=1), using default values (count=20, wait=3) > 2025-01-13 15:38:28,501 WARNING: rt1: Router(rt1): proc failed: rc 1 pid 2028454 > args: /usr/bin/nsenter --mount=/proc/2026950/ns/mnt --net=/proc/2026950/ns/net --uts=/proc/2026950/ns/uts -F --wd=/tmp/topotests/isis_srv6_topo1.test_isis_srv6_topo1/rt1 /bin/bash -c ping6 fc00:0:9::1 -c 1 -w 1 > stdout: PING fc00:0:9::1(fc00:0:9::1) 56 data bytes > > --- fc00:0:9::1 ping statistics --- > 1 packets transmitted, 0 received, 100% packet loss, time 0ms > stderr: *empty* > 2025-01-13 15:38:28,501 INFO: topo: PING fc00:0:9::1(fc00:0:9::1) 56 data bytes > > --- fc00:0:9::1 ping statistics --- > 1 packets transmitted, 0 received, 100% packet loss, time 0ms > > > PASSED > The match string is not precise enough. Complete it. Signed-off-by: Philippe Guibert <[email protected]>
When creating a sharp recursive route to the srv6 route, the srv6 segment is missing the MAC address. This results in not being able to successfully send the ping traffic. The following fix proposes to make the MAC resolution at the first attempt. This is a non optimal fix, since setting a route is not sufficient to establish a ping; in real use case scenario, this kind of scenario is not applicable then. Signed-off-by: Philippe Guibert <[email protected]>
|
@cscarpitta , are there any suggestions on the srv6 setup on how to avoid doing MAC set on the setup ? |
pguibert6WIND
requested review from
ton31337 and
cscarpitta
and removed request for
ton31337
|
It looks like this still has problems?
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix a ping test that should never pass.