Merged
Conversation
mattiaswal
force-pushed
the
new-frr
branch
8 times, most recently
from
fde6af0 to
10374a3
Compare
mattiaswal
force-pushed
the
new-frr
branch
2 times, most recently
from
7d396a3 to
dd09c99
Compare
troglobit
requested changes
Feb 6, 2026
Contributor
troglobit
left a comment
troglobit
left a comment
There was a problem hiding this comment.
Wow, this was an amazing achievement! 🤯 It will be a huge step forward and also very useful for our under-powered boards that don't need a full blown Frr just to set static routes.
mattiaswal
force-pushed
the
new-frr
branch
5 times, most recently
from
3e857e4 to
1e4b327
Compare
This includes adding a netd-daemon since FRR has gone towards all daemons should be configured from mgmtd using YANG, netd uses gRPC to reconfigure in mgmtd Signed-off-by: Enable grpc
You can use it to inspect python applications, to inspect executiontime.
Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
With the latest backported fixes to Finit, we no longer need to remove /run/netd.pid to trigger a reload (restart) of staticd. Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
In the ongoing campaign against load-bearing sleep() and pure luck. Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
Not all systems are equally quick, let's give it while to set before bailing out because we test too soon. Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
Also, update test description a bit. Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
Drop backported fixes. Details at https://github.com/finit-project/finit/releases/tag/4.16-rc1 Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
This is a simpler approach than both grpc and watchfrr. Since we only really need dynamic handling of static routes (for DHCP), we can track changes made and maintain a "diff engine" for the resulting frr.conf. Signed-off-by: Joachim Wiberg <troglobit@gmail.com>
troglobit
approved these changes
Feb 24, 2026
Contributor
troglobit
left a comment
troglobit
left a comment
There was a problem hiding this comment.
Have tested this extensively, and also done some refactoring on this branch. The new approach, which so far seems the most stable, is to use vtysh -b + using netd to calculate a "route diff" to inject into /etc/frr/frr.conf.
Contributor
Author
|
Long terms tests show ✔️ Rock solid! |
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Upgrade FRR to 10.5.1
This includes adding a netd-daemon since FRR has gone towards
all daemons should be configured from mgmtd using YANG,
netd uses gRPC to reconfigure in mgmtd.
The netd implmentation also support to to disable FRR buildtime and
still have route management using kernel routes. This is feature should
be treated as pretty untested and exprimental.
This fix #1029
Description
Checklist
Tick relevant boxes, this PR is-a or has-a: