[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