[ovs-dev] [PATCH ovn 2/2] ovn-northd: Fix IPv6 ECMP symmetric reply flows
Ilya Maximets
i.maximets at ovn.org
Thu May 20 18:53:19 UTC 2021
On 5/20/21 6:30 PM, Mark Gray wrote:
> When adding ECMP routes with symmetric replies, ovn-northd
> adds a priority 100 flow in the table "lr_in_ip_routing" which bypasses
> the ECMP routes for replies by using the ct.rpl field. Lower priority
> flows are also added in this table for each route to IPv4/6 network. These
> flows have a priority that is calculated by build_route_match() as a function
> of the length of the prefix of the route. For long prefixes the priority of
> these flows exceeds that of the symmetric reply flow causing unexpected
> behaviour. This is particularly seen with IPv6 due to the long length of IPv6
> prefixes.
>
> This patch changes the priority of the symmetric reply flow to 300
> which will ensure it will have a greater priority than the route
> flows even at the maximum prefix length of 128.
>
> This patch also adds additional tests for this condition.
>
> Reported-at: https://bugzilla.redhat.com/1959008
> Fixes: 4fdca656857d ("Add ECMP symmetric replies.")
> Signed-off-by: Mark Gray <mark.d.gray at redhat.com>
> ---
> northd/ovn-northd.8.xml | 2 +-
> northd/ovn-northd.c | 2 +-
> tests/ovn.at | 127 +++++++++++++++++++++++++++++++
> tests/system-ovn.at | 162 ++++++++++++++++++++++++++++++++++++++++
> 4 files changed, 291 insertions(+), 2 deletions(-)
Shouldn't there be a corresponding ddlog part?
Best regards, Ilya Maximets.
More information about the dev
mailing list