Hi Ales Thanks for the patch - this will make our lives easier in the future!
Looks good to me with a small nit in system-ovn.at, which can be addressed in this patch or the previous one : - We have this comment "Check entries in table 76 and 77 expires w/o traffic" which should be upgraded to their table name - (table=OFTABLE_ECMP_NH_MAC and table=OFTABLE_ECMP_NH_MAC) - "table 33" is referenced twice in system-ovn, but it seems incredibly old, so I doubt it is correct. Therefore, I suggest deleting the table number completely. Acked-by: Xavier Simonart <[email protected]> Thanks Xavier On Tue, Jun 16, 2026 at 12:08 PM 0-day Robot <[email protected]> wrote: > Bleep bloop. Greetings Ales Musil, I am a robot and I have tried out your > patch. > Thanks for your contribution. > > I encountered some error that I wasn't expecting. See the details below. > > > checkpatch: > WARNING: Line is 87 characters long (recommended limit is 79) > #616 FILE: ovn-architecture.7.xml:1446: > 91, then its actions set the logical flow flag > <code>MLF_LOOKUP_MAC</code>. > > Lines checked: 974, Warnings: 1, Errors: 0 > > > Please check this out. If you feel there has been an error, please email > [email protected] > > Thanks, > 0-day Robot > _______________________________________________ > 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
