On Mon, Jan 27, 2025 at 03:48:27PM +0100, Dumitru Ceara wrote:
> On 1/27/25 3:38 PM, Felix Huettner wrote:
> > On Wed, Jan 22, 2025 at 09:29:21AM +0100, Dumitru Ceara wrote:
> >> On 1/22/25 1:34 AM, Numan Siddique wrote:
> >>> On Thu, Jan 16, 2025 at 8:27 AM Dumitru Ceara <dce...@redhat.com> wrote:
> >>>>
> >>>> On 1/15/25 12:13 PM, Dumitru Ceara wrote:
> >>>>> On 1/2/25 4:19 PM, Felix Huettner via dev wrote:
> >>>>>> Hi everyone,
> >>>>>>
> >>>>>> this is the ovn-controller part of the OVN Fabric integration series
> >>>>>> https://mail.openvswitch.org/pipermail/ovs-dev/2024-November/418554.html.
> >>>>>>
> >>>>>> By the end of the series ovn-controller can announce routes from the
> >>>>>> sb Advertised_Route table by writing them to a linux vrf.
> >>>>>> Ovn-controller can also learn routes by reading them from the linux 
> >>>>>> vrf and
> >>>>>> writing them to the sb Learned_Route table.
> >>>>>>
> >>>>>> This patch series has two prerequisites and will therefor probably 
> >>>>>> fail CI
> >>>>>> tests for now. Therefor i marked this series as RFC.
> >>>>>>
> >>>>>> One prerequisite is including Frodes changes in the ovs submodule:
> >>>>>> https://patchwork.ozlabs.org/project/openvswitch/list/?series=437419
> >>>>>>
> >>>>>> Another prerequisite is my changes to northd. They are not required for
> >>>>>> compilation but to ensure that tests can run:
> >>>>>> https://patchwork.ozlabs.org/project/ovn/list/?series=437305
> >>>>>>
> >>>>>> Major changes v2->v3:
> >>>>>> * Ensure that we do not try write to non existing sb tables during
> >>>>>>   upgrades.
> >>>>>>
> >>>>>> Felix Huettner (8):
> >>>>>>   controller: Introduce route node.
> >>>>>>   controller: Introduce route-exchange-netlink.
> >>>>>>   controller: Announce routes via route-exchange.
> >>>>>>   controller: Support learning routes.
> >>>>>>   controller: Support learning routes per iface.
> >>>>>>   controller: Prioritize host routes.
> >>>>>>   controller: Watch for route changes.
> >>>>>>   controller: Cleanup routes on stop.
> >>>>>
> >>>>> Isn't a significant chunk of this based on Frode's
> >>>>> https://patchwork.ozlabs.org/project/ovn/list/?series=416659&state=* ?
> >>>>>
> >>>>> Frode, Felix, it would be nice if you guys could have a look and agree
> >>>>> on the proper authorship attributions of these patches.
> >>>>>
> >>>>
> >>>> I did a first full review iteration of this series and shared some
> >>>> comments/findings/questions on each of the individual patches.
> >>>>
> >>>> However, I didn't do full e2e testing yet.  I'll try to also review the
> >>>> "OVN Fabric integration: active-active." [0] series before doing more
> >>>> thorough testing.
> >>>>
> >>>> [0] https://patchwork.ozlabs.org/project/ovn/list/?series=437377&state=*
> >>>
> >>> Hi Felix,
> >>>
> >>> For the v4 version,  can you possible  add a multinode system test(s)
> >>> (in tests/ovn-multinode.at)
> >>> instantiating frr daemons ?
> >>>
> >>> This would help in understanding the feature end-to-end and of course
> >>> to test the functionality end-to-end ?
> >>>
> >>
> >> +1 that's a very good idea!  Felix, let me know if you need help with
> >> doing that.
> > 
> > Hi Numan, Hi Dumitru,
> > 
> > sounds like a good idea.
> > However i am not sure if i will have the time to actually get this done
> > before the feature freeze for 25.03.
> > 
> > I would try to send out v4 as soon as i have it ready and then build
> > this afterwards, just to give reviewers more time.
> > 
> > However if anyone else could help out there that would be really great.
> > 
> 
> I'll see if I manage to come up with something.  If we have time it
> would be nice to get that into our code base before 25.03 release.

Thanks a lot.

> 
> Regards,
> Dumitru
> 
> > Thanks a lot,
> > Felix
> > 
> >>
> >> Thanks,
> >> Dumitru
> >>
> >>> Thanks
> >>> Numan
> >>>
> >>>>
> >>>> Regards,
> >>>> Dumitru
> >>>>
> >>>> _______________________________________________
> >>>> dev mailing list
> >>>> d...@openvswitch.org
> >>>> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> >>>
> >>
> > 
> 
> _______________________________________________
> dev mailing list
> d...@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to