Skip to content

Conversation

@raja-rajasekar
Copy link
Contributor

Static daemons in the current code does not remove/add SRv6 sids configured with behavior uA on an interface when it receives the interface events.

Fixing this.

Initial state:
ip -6 -d route  | grep End.X
unicast fcbb:bbbb:fe5::/48 nhid 16  encap seg6local action End.X nh6 fe80::c0f9:81ff:fe68:6455 dev r1-eth1 proto 196 scope global metric 20 pref medium vtysh -c "sh ipv6 route" | grep seg6
S>* fcbb:bbbb:fe5::/48 [1/0] is directly connected, r1-eth1, seg6local uA nh6 fe80::c0f9:81ff:fe68:6455, weight 1, 00:00:20

ip link set down r1-eth1
vtysh -c "sh ipv6 route" | grep seg6
S>  fcbb:bbbb:fe5::/48 [1/0] is directly connected, r1-eth1 inactive, seg6local uA nh6 fe80::c0f9:81ff:fe68:6455, weight 1, 00:00:28 ip -6 -d route  | grep End.X

ip link set up r1-eth1
vtysh -c "sh ipv6 route" | grep seg6
S>* fcbb:bbbb:fe5::/48 [1/0] is directly connected, r1-eth1, seg6local uA nh6 fe80::c0f9:81ff:fe68:6455, weight 1, 00:00:41 ip -6 -d route  | grep End.X

Static daemons in the current code does not remove/add SRv6 sids configured
with behavior uA on an interface when it receives the interface events.

Fixing this.

Initial state:
ip -6 -d route  | grep End.X
unicast fcbb:bbbb:fe5::/48 nhid 16  encap seg6local action End.X nh6 fe80::c0f9:81ff:fe68:6455 dev r1-eth1 proto 196 scope global metric 20 pref medium
vtysh -c "sh ipv6 route" | grep seg6
S>* fcbb:bbbb:fe5::/48 [1/0] is directly connected, r1-eth1, seg6local uA nh6 fe80::c0f9:81ff:fe68:6455, weight 1, 00:00:20

ip link set down r1-eth1
vtysh -c "sh ipv6 route" | grep seg6
S>  fcbb:bbbb:fe5::/48 [1/0] is directly connected, r1-eth1 inactive, seg6local uA nh6 fe80::c0f9:81ff:fe68:6455, weight 1, 00:00:28
ip -6 -d route  | grep End.X

ip link set up r1-eth1
vtysh -c "sh ipv6 route" | grep seg6
S>* fcbb:bbbb:fe5::/48 [1/0] is directly connected, r1-eth1, seg6local uA nh6 fe80::c0f9:81ff:fe68:6455, weight 1, 00:00:41
ip -6 -d route  | grep End.X

Signed-off-by: Rajasekar Raja <[email protected]>
@raja-rajasekar
Copy link
Contributor Author

ci:rerun

1 similar comment
@raja-rajasekar
Copy link
Contributor Author

ci:rerun

Copy link
Contributor

@cscarpitta cscarpitta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@cscarpitta cscarpitta merged commit 47644ad into FRRouting:master Jun 16, 2025
14 checks passed
@cscarpitta
Copy link
Contributor

@Mergifyio backport dev/10.4

@mergify
Copy link

mergify bot commented Jun 16, 2025

backport dev/10.4

✅ Backports have been created

Details

@raja-rajasekar raja-rajasekar deleted the rajasekarr/fix_intf_flap_srv6_endx branch June 16, 2025 03:34
ton31337 added a commit that referenced this pull request Jun 16, 2025
static: [SRv6] Fixing uninstall and reinstall uA Sids upon Intf flaps (backport #19027)
cscarpitta added a commit to cscarpitta/sonic-buildimage that referenced this pull request Jul 17, 2025
- staticd: Do not log uninitialized nexthop variable (FRRouting/frr#18271)
- staticd: fix deref of NULL pointer in srv6 code (FRRouting/frr#18890)
- static: [SRv6] Fixing uninstall and reinstall uA Sids upon Intf flaps (FRRouting/frr#19027)

Signed-off-by: Carmine Scarpitta <[email protected]>
cscarpitta added a commit to cscarpitta/sonic-buildimage that referenced this pull request Jul 17, 2025
- staticd: Do not log uninitialized nexthop variable (FRRouting/frr#18271)
- staticd: fix deref of NULL pointer in srv6 code (FRRouting/frr#18890)
- static: [SRv6] Fixing uninstall and reinstall uA Sids upon Intf flaps (FRRouting/frr#19027)

Signed-off-by: Carmine Scarpitta <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants