On 2/25/25 4:54 PM, Dumitru Ceara wrote:
> In order to achieve that the first 2 patches in the series improve the
> infrastructure used for advertising routes in northd. The last patch of
> the series changes the type of data the en_dynamic_routes I-P node
> computes so that it stores a map of 'struct ar_entry' instead of 'struct
> parsed_route'. That allows us to avoid reparsing LB/NAT IPs.
>
> Fixes: cd4ad2f56179 ("northd: Redistribution of NAT/LB routes.")
> Fixes: f2deb24c5c43 ("northd: Sync Advertised_Route to sb.")
> Fixes: 93f541f342f9 ("northd: Allow announcing individual host routes.")
> CC: Felix Huettner <felix.huettner@stackit.cloud>
> CC: Martin Kalcok <martin.kal...@canonical.com>
>
Really cc Felix and Martin this time.
> V2:
> - dropped first two patches from v1 (they were already applied to the
> tree)
> - added acks from reviewers on the first two patches
> - patch 3:
> - addressed Felix's comments:
> - added OVS_NOT_REACHED() for cases that shouldn't happen.
> - split the functionality of should_advertise_route() to make it
> more clear
> - addressed Martin's comments:
> - removed stale TODO item
> - renamed functions that build dynamic routes
>
> Dumitru Ceara (3):
> northd: Move functions that announce LB/NAT IPs.
> northd/en-advertised-route-sync: Store parsed data instead of IDL
> pointers.
> northd/en-advertised-route-sync: Avoid reparsing NAT/LB IPs.
>
> TODO.rst | 7 -
> northd/en-advertised-route-sync.c | 661 ++++++++++++++++++++++--------
> northd/northd.c | 231 -----------
> northd/northd.h | 17 +-
> 4 files changed, 486 insertions(+), 430 deletions(-)
>
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
