search

sonic-net / sonic-host-services

4 stars 80 forks source link

[BFD]Fix BFD blackout issue #150

Closed dgsudharsan closed 2 months ago

dgsudharsan commented 2 months ago

Fixes https://github.com/sonic-net/sonic-buildimage/issues/19762

When caclmgrd processes any ACL table change, this results in rules flushed and readded which was introduced in https://github.com/sonic-net/sonic-host-services/pull/114 . However in this flow the BFD and vxlan rules were added at the end which may result in traffic loss because of the DROP rule for ip2me installed. To overcome this added BFD and vxlan rules at the very start so that there is no drop seen for BFD.

Existing UT should verify the flows.

Additionally manually verified

prsunny commented 2 months ago

@ganglyu , can you please merge this PR?

mssonicbld commented 2 months ago

Cherry-pick PR to 202405: https://github.com/sonic-net/sonic-host-services/pull/162

mssonicbld commented 1 month ago

Cherry-pick PR to 202311: https://github.com/sonic-net/sonic-host-services/pull/165