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

Add frr gateway support #849

Draft
wants to merge 4 commits into
base: dev
Choose a base branch
from

Conversation

jbemmel
Copy link
Collaborator

@jbemmel jbemmel commented Jun 29, 2023

There are still some issues to be solved:

/ # arping -I eth1 172.16.0.1 # VRRP VIP
ARPING 172.16.0.1 from 172.16.0.5 eth1
Unicast reply from 172.16.0.1 [aa:c1:ab:95:31:67] 0.022ms
Unicast reply from 172.16.0.1 [aa:c1:ab:69:50:ee] 0.030ms
Unicast reply from 172.16.0.1 [00:00:5e:00:01:01] 0.034ms
Unicast reply from 172.16.0.1 [00:00:5e:00:01:01] 0.012ms
Unicast reply from 172.16.0.1 [00:00:5e:00:01:01] 0.011ms
^CSent 3 probe(s) (1 broadcast(s))
Received 5 response(s) (0 request(s), 0 broadcast(s))
/ # arping -I eth1 172.16.0.3 # interface IP
ARPING 172.16.0.3 from 172.16.0.5 eth1
Unicast reply from 172.16.0.3 [aa:c1:ab:95:31:67] 0.006ms
Unicast reply from 172.16.0.3 [aa:c1:ab:95:31:67] 0.009ms
Unicast reply from 172.16.0.3 [aa:c1:ab:95:31:67] 0.010ms
^CSent 3 probe(s) (1 broadcast(s))

The above is the vrrp case, 3 different interfaces respond to ARP. For anycast I was able to remove 1 response, but ARP requests are still sourced from the wrong MAC

The PR creates a macvtap device on top of the Linux bridge for a vlan, in bridge mode. It could be that this model needs to be changed (the example at https://docs.frrouting.org/en/latest/vrrp.html uses a physical interface)

@ipspace
Copy link
Owner

ipspace commented Jun 30, 2023

Does "There are still some issues to be solved" mean "this should be a draft PR, wait with merging till I sort them out"?

@jbemmel jbemmel marked this pull request as draft June 30, 2023 06:27
@jbemmel
Copy link
Collaborator Author

jbemmel commented Jun 30, 2023

Does "There are still some issues to be solved" mean "this should be a draft PR, wait with merging till I sort them out"?

I am sharing what I have in the hope that others could look at it and chime in. I've been pushing it forward, but it may very well be that there are better ways to do this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants