Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bgpd: Do not start BGP session if BFD profile is in shutdown state #16194

Merged
merged 3 commits into from
Jun 18, 2024

Conversation

ton31337
Copy link
Member

No description provided.

@ton31337 ton31337 added this to the 10.1 milestone Jun 11, 2024
lib/bfd.c Outdated Show resolved Hide resolved
ton31337 added 2 commits June 11, 2024 15:51
When we receive a hard-reset notification, we always show it if it was a hard,
or not.

For sending side, we missed that. Let's display it too.

Signed-off-by: Donatas Abraitis <[email protected]>
@ton31337 ton31337 force-pushed the fix/bfd_profile_shutdown branch from 0739e59 to eaaadf5 Compare June 11, 2024 12:56
@ton31337 ton31337 requested a review from mjstapp June 11, 2024 12:57
Copy link
Contributor

@mjstapp mjstapp left a comment

Choose a reason for hiding this comment

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

Thanks - that feels better! Had just one nit...

lib/bfd.c Outdated Show resolved Hide resolved
@ton31337 ton31337 force-pushed the fix/bfd_profile_shutdown branch from eaaadf5 to 79368f8 Compare June 11, 2024 13:03
Copy link
Contributor

@mjstapp mjstapp left a comment

Choose a reason for hiding this comment

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

Thanks for the speedy fix

Copy link
Member

@riw777 riw777 left a comment

Choose a reason for hiding this comment

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

looks good

@pbrisset pbrisset self-requested a review June 11, 2024 15:05
lib/bfd.c Outdated Show resolved Hide resolved
bgpd/bgp_packet.c Show resolved Hide resolved
@ton31337 ton31337 force-pushed the fix/bfd_profile_shutdown branch 2 times, most recently from 11a488d to 83c22b6 Compare June 11, 2024 19:31
@pbrisset
Copy link

Hi,

I just try the latest fix on my system and I believe this is working as expected.
As per previous comments, the topotest needs few more test extensions before committing to master.

Thanks you so much for this prompt answer.

If we do:

```
bfd
 profile foo
  shutdown
```

The session is dropped, but immediately established again because we don't
have a proper check on BFD.

If BFD is administratively shutdown, ignore starting the session.

Fixes: FRRouting#16186

Signed-off-by: Donatas Abraitis <[email protected]>
@ton31337 ton31337 force-pushed the fix/bfd_profile_shutdown branch from 83c22b6 to 1fb48f5 Compare June 12, 2024 05:39
@riw777 riw777 merged commit e9e8a4b into FRRouting:master Jun 18, 2024
10 checks passed
@ton31337 ton31337 deleted the fix/bfd_profile_shutdown branch June 18, 2024 14:15
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.

4 participants