On Wed, 2025-02-12 at 15:20 +0100, Felix Huettner wrote: > > > +/* XXX: This function is, in nature, very similar to how > > + * publish_host_routes_lrp looks up neighboring host routes. I > > really wanted > > + * to reuse it, but it's designed to work with already existing > > parsed_routes > > + * when creating Advertised_Route records. And that doesn't work > > in following > > + * scenario: > > + * > > + * R1 --- ls_int --- R2 > > + * > > + * If R1 and R2 don't have IPv4/6 configured on their LRPs plugged > > to > > + * ls_int, and you set "connected-as-host", no parsed_routes will > > be generated > > + * (makes sense). But as a consequence, publish_host_routes_lrp is > > never > > + * executed. > > there is a parsed_route generated for the link local address but we > have > been filtering it out before processing it further. > However in the case of connected-as-host i see no reason to keep this > limitation. > > I have built a small patch on top of current main to share this. > Not sure if that is exactly what you are looking for. > https://github.com/felixhuettner/ovn/commit/5326af4e0fb0b054a7f023ed8d15ae5b2969928a > >
As I mentioned in my previous email, this may suit our use-case. Would you be OK with me including this patch in the next version and signing it off on your behalf? (with you as an author of course) Martin. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
