On Thu, Jun 3, 2021 at 2:50 PM Mark Michelson 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.
>
> Originally, this was a two patch series but it has bloomed into a 5
> patch series. After v7, a new approach was suggested and it's now 6
> patches.
>
> Patch 1 fixes the scenario where a VM attempts to reach a floating IP on
> the directly connected router. This has been part of this patch series
> since v1.
>
> Patch 2 is an incidental fix that removes a redundant paragraph from
> documenttion.
>
> Patch 3 is a small cleanup in ovn-northd.c to factor out peer retrieval
> into its own function.
>
> Patch 4 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 5 recognizes that patch 4 may not always be desired, so it makes
> the behavior opt-in.
>
> Finally, patch 6 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.
> ---
> v7 -> v8: First 2 patches are the same as they have been in previous
> versions. Patch 6 is nearly identical to patch 5 from previous versions.
> Patches 3, 4, and 5 represent a completely new approach to solving the
> issue from before. And that's also why previous version are not
> documented here.
> ---
> Mark Michelson (6):
> northd: Swap src and dst eth addresses in router egress loop.
> ovn-sb: Remove redundant "nat-addresses" information from
> Port_Binding.
> 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.
Hi Mark,
I applied the first 2 patches of this series to master and backported
the first patch to branch-21.06.
I think the first patch is required to address the issue reported by
Brendan Doyle.
The first patch doesn't apply cleanly to other branches. Can you
please take a look and try to backport
to other branches ? I think this fix is required.
I provided a few comments to patch 4 earlier.
Thanks
Numan
>
> northd/ovn-northd.8.xml | 8 +
> northd/ovn-northd.c | 380 +---
> northd/ovn_northd.dl| 180 ---
> ovn-nb.xml | 29 ++-
> ovn-sb.xml | 10 --
> tests/ovn-nbctl.at | 3 +
> tests/ovn-northd.at | 345
> tests/system-ovn.at | 215 ++-
> utilities/ovn-nbctl.c | 25 ++-
> 9 files changed, 1043 insertions(+), 152 deletions(-)
>
> --
> 2.31.1
>
> ___
> dev mailing list
> d...@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev