Page MenuHomeFreeBSD

Fix per-FIB gateway support in rc.d/routing
ClosedPublic

Authored by andrew.fengler_scaleengine.com on Dec 13 2022, 5:30 PM.
Tags
None
Referenced Files
Unknown Object (File)
Jan 25 2024, 11:06 PM
Unknown Object (File)
Dec 20 2023, 6:02 AM
Unknown Object (File)
Dec 12 2023, 10:20 AM
Unknown Object (File)
Aug 30 2023, 12:11 PM
Unknown Object (File)
Aug 26 2023, 5:06 AM
Unknown Object (File)
Jul 11 2023, 1:08 AM
Unknown Object (File)
Jul 8 2023, 2:07 PM
Unknown Object (File)
Jul 5 2023, 1:59 AM
Subscribers

Details

Summary

See: https://reviews.freebsd.org/D22706
There was a mistake in the previous diff. I'm not sure how that ended up that way, but there were a few mistakes in variable names. This patch will complete what was originally written and tested.

It changes 3 variable names to correctly use $_fib instead of $fib, and to prepend any ipv6 static routes, instead of ipv4 static routes to the the ipv6 route list.

Sorry for the mess.

Test Plan

This is what i originally tested in D22706.
It's running in production on several machines running 13.x

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Dec 13 2022, 6:13 PM

This fix should be MFCed into stable/13, as it fixes D22706 which has been cherry-picked into stable/13.

I'd suggest to add commit meta message MFC after to have mail reminders.

Can this please be MFCed to stable/13? As mentioned, this fixes a previous incorrect MFC.

Can this please be MFCed to stable/13? As mentioned, this fixes a previous incorrect MFC.

No response from the committer for a long time.

I'll test and MFC if no objections.

MFCed into stable/13 via commit e8deaa3c6b8b .

FreeBSD's Phabricator has some lag and does not reflect the commit.