zebra: fix static srv6 segment-list sid order #15616
Merged
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
pguibert6WIND
force-pushed
the
srv6_seg_list_sid_order
branch
from
aae7b9f to
a3536fa
Compare
|
CI:rerun libyang version may have changed during execution which caused the failure |
|
ci:rerun |
pguibert6WIND
force-pushed
the
srv6_seg_list_sid_order
branch
3 times, most recently
from
2cad071 to
c4f20bb
Compare
|
ci:rerun |
pguibert6WIND
force-pushed
the
srv6_seg_list_sid_order
branch
from
c4f20bb to
718c465
Compare
When configuring a SID list by vtysh, the segment list obtained in iproute2 is the exact opposite: > >vtysh: >ipv6 route 2005::/64 eth0 segments 2001:db8:aaaa::7/2002::2/2003::3/2004::4 > >root@r1:/# ip -6 route >2005::/64 nhid 6 encap seg6 mode encap segs 4 [ 2004::4 2003::3 2002::2 2001:db8:aaaa::7 ] dev dummy0 proto 196 metric 20 pref medium > Fix this by keeping the same vtysh config and swap the segment's order of the list in the rt_netlink.c > >root@r1:/# ip -6 route >2005::/64 nhid 6 encap seg6 mode encap segs 4 [ 2001:db8:aaaa::7 2002::2 2003::3 2004::4 ] dev dummy0 proto 196 metric 20 pref medium > Fixes: f20cf14 ("bgpd,lib,sharpd,zebra: srv6 introduce multiple segs/SIDs in nexthop") Signed-off-by: Dmytro Shytyi <[email protected]>
pguibert6WIND
force-pushed
the
srv6_seg_list_sid_order
branch
from
718c465 to
8e82e2d
Compare
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.
When configuring a SID list by vtysh, the segment list obtained in iproute2 is the exact opposite:
Fix this by keeping the same vtysh config and swap the segment's order of the list in the rt_netlink.c
Fixes: f20cf14 ("bgpd,lib,sharpd,zebra: srv6 introduce multiple segs/SIDs in nexthop")