[ovs-dev] [PATCH v3 2/3] ovn: Gratuitous ARP for centralized NAT rules on a distributed router
Mickey Spiegel
mickeys.dev at gmail.com
Fri Mar 17 22:29:17 UTC 2017
On Fri, Mar 17, 2017 at 12:47 PM, Guru Shetty <guru at ovn.org> wrote:
>
>
> On 2 February 2017 at 20:48, Mickey Spiegel <mickeys.dev at gmail.com> wrote:
>
>> This patch extends gratuitous ARP support for NAT addresses so that it
>> applies to centralized NAT rules on a distributed router, in addition to
>> the existing gratuitous ARP support for NAT addresses on gateway routers.
>>
>> Gratuitous ARP packets for centralized NAT rules on a distributed router
>> are only generated on the redirect-chassis. This is achieved by extending
>> the syntax for "options:nat-addresses" in the southbound database,
>> allowing the condition 'is_chassis_resident("LPORT_NAME")' to be appended
>> after the MAC and IP addresses. This condition is automatically inserted
>> by ovn-northd when the northbound "options:nat-addresses" is set to
>> "router" and the peer is a distributed gateway port.
>>
>> A separate patch will be required to support gratuitous ARP for
>> distributed NAT rules that specify logical_port and external_mac. Since
>> the MAC address differs and the logical port often resides on a different
>> chassis from the redirect-chassis, these addresses cannot be included in
>> the same "nat-addresses" string as for centralized NAT rules.
>>
>> Signed-off-by: Mickey Spiegel <mickeys.dev at gmail.com>
>>
>
> Would you please mind re-spinning this? It does not apply anymore.
>
In addition to the rebase, in the next patch for distributed NAT rules:
- updated ovn-sb.xml references to OVS version 2.8 rather than OVS version
2.7.
- added a NEWS item.
- streamlined code to remove an unnecessary malloc that I had added in
ovn-northd.c ovn_port_update_sbrec.
Mickey
>
>> ---
>> ovn/controller/pinctrl.c | 104 ++++++++++++++++++++++++++++++
>> ++++++++++++++---
>> ovn/lib/ovn-util.c | 38 ++++++++++++++---
>> ovn/lib/ovn-util.h | 2 +
>> ovn/northd/ovn-northd.c | 52 +++++++++++++++++-------
>> ovn/ovn-nb.xml | 33 ++++++++++++---
>> ovn/ovn-sb.xml | 31 ++++++++++----
>> tests/ovn.at | 70 +++++++++++++++++++++++++++++++
>> 7 files changed, 289 insertions(+), 41 deletions(-)
>>
>
More information about the dev
mailing list