On 12/12/24 1:53 PM, Felix Huettner wrote: > On Wed, Dec 11, 2024 at 04:06:55PM +0100, Dumitru Ceara wrote: >> On 12/10/24 3:12 PM, Felix Huettner via dev wrote: >>> previously connected routes (so routes based on the networks of the >>> LRPs) where handled separately from static routes. >>> >>> In order to later announce routes from ovn to the fabric we need to have >>> a central overview over all routes we want to announce. This includes >>> routes based on Logical_Router_Static_Routes as well as the networks on >>> Logical_Router_Port. >>> >>> With this change the "routes" engine node will output a complete list of >>> all routes in the system. This also serves as a future integration point >>> to integrate routes learned from outside of OVN. >>> >>> Signed-off-by: Felix Huettner <felix.huettner@stackit.cloud> >>> --- >> Hi Felix, >> >> I didn't dig into it too much but this patch introduces a crash in >> northd. E.g. on the ovsrobot runs: > Hi Dumitru, > > i found the issue and it is caused by a double free. > Once i have time i will send a new version. > > The fix will be part of "northd: Store outport of parsed_route." since > it fitted there best. >
OK, I'll review the first two patches in the series then. Thanks, Dumitru _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
