> Remove the flow in lr_in_gw_redirect stage:
>
> A priority-150 logical flow with match
> <code>outport == <var>GW</var> &&
> eth.dst == 00:00:00:00:00:00</code> has actions
> <code>outport = <var>CR</var>; next;</code>, where
> <var>GW</var> is the logical router distributed gateway
> port and <var>CR</var> is the <code>chassisredirect</code>
> port representing the instance of the logical router
> distributed gateway port on the
> <code>redirect-chassis</code>.
>
> The commit c0bf32d ("Manage ARP process locally in a DVR scenario") updated
> the priority-100 flow in this stage to priority 200, which makes this
> priority-150 flow useless, because whatever packets matching this flow
> would also match the priority-50 flow.
>
> Cc: Lorenzo Bianconi <[email protected]>
> Signed-off-by: Han Zhou <[email protected]>
Thanks Han
Tested-by: Lorenzo Bianconi <[email protected]>
> ---
> northd/ovn-northd.8.xml | 12 ------------
> northd/ovn-northd.c | 11 -----------
> 2 files changed, 23 deletions(-)
>
> diff --git a/northd/ovn-northd.8.xml b/northd/ovn-northd.8.xml
> index 1f81742..8f224b0 100644
> --- a/northd/ovn-northd.8.xml
> +++ b/northd/ovn-northd.8.xml
> @@ -2909,18 +2909,6 @@ icmp4 {
>
> <ul>
> <li>
> - A priority-150 logical flow with match
> - <code>outport == <var>GW</var> &&
> - eth.dst == 00:00:00:00:00:00</code> has actions
> - <code>outport = <var>CR</var>; next;</code>, where
> - <var>GW</var> is the logical router distributed gateway
> - port and <var>CR</var> is the <code>chassisredirect</code>
> - port representing the instance of the logical router
> - distributed gateway port on the
> - <code>redirect-chassis</code>.
> - </li>
> -
> - <li>
> For each NAT rule in the OVN Northbound database that can
> be handled in a distributed manner, a priority-200 logical
> flow with match <code>ip4.src == <var>B</var> &&
> diff --git a/northd/ovn-northd.c b/northd/ovn-northd.c
> index 3f03e41..b0344d7 100644
> --- a/northd/ovn-northd.c
> +++ b/northd/ovn-northd.c
> @@ -10156,17 +10156,6 @@ build_lrouter_flows(struct hmap *datapaths, struct
> hmap *ports,
> ovn_lflow_add_with_hint(lflows, od, S_ROUTER_IN_GW_REDIRECT, 50,
> ds_cstr(&match), ds_cstr(&actions),
> stage_hint);
> -
> - /* If the Ethernet destination has not been resolved,
> - * redirect to the central instance of the l3dgw_port.
> - * Such traffic will be replaced by an ARP request or ND
> - * Neighbor Solicitation in the ARP request ingress
> - * table, before being redirected to the central instance.
> - */
> - ds_put_format(&match, " && eth.dst == 00:00:00:00:00:00");
> - ovn_lflow_add_with_hint(lflows, od, S_ROUTER_IN_GW_REDIRECT, 150,
> - ds_cstr(&match), ds_cstr(&actions),
> - stage_hint);
> }
>
> /* Packets are allowed by default. */
> --
> 2.1.0
>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
