On 2/12/25 5:12 PM, Felix Huettner via dev wrote:
> When adding the "source" field to "struct ecmp_groups_node" we missed to
> also take this field into account in ecmp_groups_find.
> This could lead to northd building an ecmp group based on different
> routes from different sources (e.g. static and learned routes).
> As these routes are designed to have different priorities we need to
> ensure that these priorities are still respected for ECMP routes.
>
> Fixes: f8924740f26e ("northd: Move connected routes to route engine.")
> Signed-off-by: Felix Huettner <[email protected]>
> ---
> northd/northd.c | 3 +-
> tests/ovn-northd.at | 131 ++++++++++++++++++++++++++++++++++++++++++++
> 2 files changed, 133 insertions(+), 1 deletion(-)
>
Thanks, Felix, for the fix and also thanks for adding a test for this case!
Applied to main.
Regards,
Dumitru
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
