On Wed, Feb 12, 2025 at 04:17:25PM +0100, [email protected] wrote: > 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)
If it is helpful i am completely fine with that. Felix > > Martin. > _______________________________________________ > dev mailing list > [email protected] > https://mail.openvswitch.org/mailman/listinfo/ovs-dev _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
