[ovs-dev] [PATCH] OVN: do not distribute traffic for local FIP

[Numan Siddique]

On Mon, May 27, 2019 at 2:31 PM Lorenzo Bianconi <
lorenzo.bianconi at redhat.com> wrote:

> Do not send traffic for local FIP through the overlay tunnels but
> manage it in the local hypervisor
>
> Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi at redhat.com>
> ---
>  ovn/northd/ovn-northd.c | 13 +++++++++++++
>  1 file changed, 13 insertions(+)
>
> diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn-northd.c
> index de0c06d4b..b233deef2 100644
> --- a/ovn/northd/ovn-northd.c
> +++ b/ovn/northd/ovn-northd.c
> @@ -5222,6 +5222,19 @@ add_distributed_nat_routes(struct hmap *lflows,
> const struct ovn_port *op)
>              continue;
>          }
>
> +        ds_put_format(&match, "inport == %s && "
> +                      "ip4.src == %s && ip4.dst == %s",
> +                       op->json_key, nat->logical_ip, nat->external_ip);
> +        ds_put_format(&actions, "outport = %s; eth.dst = %s; "
> +                      REGBIT_DISTRIBUTED_NAT" = 1; "
> +                      REGBIT_NAT_REDIRECT" = 0; next;",
> +                      op->od->l3dgw_port->json_key,
> +                      nat->external_mac);
> +        ovn_lflow_add(lflows, op->od, S_ROUTER_IN_IP_ROUTING, 400,
> +                      ds_cstr(&match), ds_cstr(&actions));
> +        ds_clear(&match);
> +        ds_clear(&actions);
> +
>          for (size_t j = 0; j < op->od->nbr->n_nat; j++) {
>              const struct nbrec_nat *nat2 = op->od->nbr->nat[j];
>
>
Hi Lorenzo,

The changes looks fine to me. Can you please also update the documentation
in ovn-northd-8.xml ?

Thanks
Numan

-- 
> 2.21.0
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>