[ovs-dev] [PATCH ovn v11 0/4] ARP and Floating IP Fixes

[Mark Michelson]

On 7/8/21 12:04 PM, Numan Siddique wrote:
> On Thu, Jul 8, 2021 at 10:44 AM Mark Michelson <mmichels at redhat.com> wrote:
>>
>> This patch series aims to fix issues seen in OpenStack deployments when
>> floating IPs were assigned to routers, and those floating IPs were not
>> part of any subnet configured on that router.
>>
>> This series has gone through several transformations, but this
>> incarnation is a four patch series:
>>
>> Patch 1 is a small cleanup in ovn-northd.c to factor out peer retrieval
>> into its own function.
>>
>> Patch 2 alters northd to install logical flows to make it so that
>> routers can reach NAT and load balancer addresses on their neighbors
>> without the need to configure static routes or MAC bindings.
>>
>> Patch 3 recognizes that patch 2 may not always be desired, so it makes
>> the behavior opt-in.
>>
>> Finally, patch 4 addresses the situation for when the pre-allocated
>> logical flows cannot be used. For this situation, we will flood the ARP
>> request if the TPA is for a configured IP address that is outside the
>> connected routers' subnets.
>> ---
>> v10 -> v11:
>> * Fixed memory leak in patch 2
> 
> Thanks.
> 
> For the entire series:
> 
> Acked-by: Numan Siddique <numans at ovn.org>
> 
> Numan

THanks for the review Numan. I merged this to master.

> 
>> ---
>>
>> Mark Michelson (4):
>>    northd: Factor peer retrieval into its own function.
>>    northd: Add IP routing and ARP resolution flows for NAT/LB addresses.
>>    northd: Add options to automatically add routes for NATs and LBs.
>>    northd: Flood ARPs to routers for "unreachable" addresses.
>>
>>   northd/ovn-northd.8.xml |  15 +-
>>   northd/ovn-northd.c     | 416 +++++++++++++++++++++++++++++-----------
>>   northd/ovn_northd.dl    | 169 +++++++++++++---
>>   ovn-nb.xml              |  33 +++-
>>   tests/ovn-nbctl.at      |   3 +
>>   tests/ovn-northd.at     | 345 +++++++++++++++++++++++++++++++++
>>   tests/system-ovn.at     | 102 ++++++++++
>>   utilities/ovn-nbctl.c   |  27 ++-
>>   8 files changed, 969 insertions(+), 141 deletions(-)
>>
>> --
>> 2.31.1
>>
>> _______________________________________________
>> dev mailing list
>> dev at openvswitch.org
>> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>>
>