Skip to content

frr: T8021: add pending upstream patch to fix bgpd crash - #1072

Closed
c-po wants to merge 1 commit into
vyos:currentfrom
c-po:frr-bgpd
Closed

frr: T8021: add pending upstream patch to fix bgpd crash#1072
c-po wants to merge 1 commit into
vyos:currentfrom
c-po:frr-bgpd

Conversation

@c-po

@c-po c-po commented Dec 1, 2025

Copy link
Copy Markdown
Member

Change summary

This addresses FRRouting/frr#19549 which we have a pending upstream PR FRRouting/frr#20078. Until this is merged or we upgrade to FRR 10.5 we need this fix.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes)
  • Migration from an old Vyatta component to vyos-1x, please link to related PR inside obsoleted component
  • Other (please describe):

Related Task(s)

Related PR(s)

Checklist:

  • I have read the CONTRIBUTING document
  • I have linked this PR to one or more Phabricator Task(s)
  • My commit headlines contain a valid Task id
  • My change requires a change to the documentation
  • I have updated the documentation accordingly

This addresses FRRouting/frr#19549 which we have a
pending upstream PR FRRouting/frr#20078. Until this is
merged or we upgrade to FRR 10.5 we need this fix.
@github-actions

github-actions Bot commented Dec 1, 2025

Copy link
Copy Markdown

👍
No issues in PR Title / Commit Title

@github-actions github-actions Bot added the current VyOS rolling release label Dec 1, 2025

@sever-sever sever-sever left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Some topotests fails on Debian-arm and Ubuntu https://ci1.netdef.org/browse/FRR-PULLREQ3-TOPO9U2204AMD64-12674
I'm unfamiliar with those tests.
@c-po Is it OK?

@c-po
c-po requested a review from sever-sever December 2, 2025 15:38
@c-po

c-po commented Dec 2, 2025

Copy link
Copy Markdown
Member Author

Some topotests fails on Debian-arm and Ubuntu https://ci1.netdef.org/browse/FRR-PULLREQ3-TOPO9U2204AMD64-12674 I'm unfamiliar with those tests. @c-po Is it OK?

I think we can ignore this.

Local build of FRR and all smoketests passed.

image

@c-po
c-po enabled auto-merge December 2, 2025 15:40

@sever-sever sever-sever left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Add upstream patch to fix BGPd crashes
Tested by community, I have not tested it.

@tjjh89017

Copy link
Copy Markdown
Contributor

FRRouting/frr#20078

Upstream merged the PR.
We may rebuild FRR and disable this!
Thanks a lot @c-po for help!!!

@c-po c-po closed this Dec 3, 2025
auto-merge was automatically disabled December 3, 2025 14:20

Pull request was closed

@c-po

c-po commented Dec 3, 2025

Copy link
Copy Markdown
Member Author

Package rebuilds complete for circinus and current.

@c-po c-po reopened this Dec 3, 2025
@c-po c-po closed this Dec 3, 2025
@c-po
c-po deleted the frr-bgpd branch December 3, 2025 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

current VyOS rolling release

Development

Successfully merging this pull request may close these issues.

3 participants