peer addresses and unconnected interfaces #23
Conversation
choppsv1
force-pushed
the
main
branch
2 times, most recently
from
91b4758 to
eb56767
Compare
eklabn
force-pushed
the
ekinzie/ip-p2p
branch
from
8ede74e to
226f45a
Compare
choppsv1
force-pushed
the
main
branch
2 times, most recently
from
741d071 to
7a41244
Compare
eklabn
force-pushed
the
ekinzie/ip-p2p
branch
4 times, most recently
from
983e795 to
e5b6180
Compare
For p2p links with each endpoint in a different IP subnet, as would be found with IPv4 addresses and 32-bit subnet masks, configure the peer address. In Linux, adding an interface route with the peer's address as the destination is not sufficient; no other route will be able to use the peer's address as a next-hop.
An interface with no connection is currently omitted from the target network namespace. Instead, add a Linux "dummy" interface and assign IP addresses.
eklabn
force-pushed
the
ekinzie/ip-p2p
branch
from
e5b6180 to
e365d1d
Compare
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #23 +/- ##
==========================================
- Coverage 59.19% 59.15% -0.05%
==========================================
Files 19 19
Lines 5833 5912 +79
==========================================
+ Hits 3453 3497 +44
- Misses 2380 2415 +35 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
In line with some of my comments, can you split these two commits into two separate PRs?
I am happy to push through the /32 address support changes, but don't yet want to merge the unconnected interface changes until I can work on a proper solution for configuring parallel p2p connections.
EDIT: Ignore my previous comment, you can keep both of these commits in this PR.
| and ifaddr.network != oifaddr.network | ||
| ): | ||
| con.cmd_raises( | ||
| f"ip addr add {ifaddr} peer {oifaddr.ip} dev {ifname}" |
There was a problem hiding this comment.
You drop the peer's network mark here, which is the opposite of what you did in _set_p2p_addr (where you instead kept the peer's network mask and instead dropped the address's mask). Is this intentional?
| and ifaddr.network != oifaddr.network | ||
| ): | ||
| con.cmd_raises( | ||
| f"ip addr add {ifaddr} peer {oifaddr.ip} dev {ifname}" |
| @@ -0,0 +1,28 @@ | |||
| version: 1 | |||
There was a problem hiding this comment.
please move this test to the tests/config directory
| @@ -0,0 +1,31 @@ | |||
| version: 1 | |||
There was a problem hiding this comment.
please move this test to the tests/config directory
2 participants
Here are a couple of changes to (1) handle interface addresses with /32 masks and (2) create a dummy for each unconnected interface.