On 11/28/25 1:56 PM, Dumitru Ceara wrote: > On 11/28/25 1:16 PM, Rukomoinikova Aleksandra wrote: >> Hi! Thank you so much for the review. I'd be glad if you merged it after >> the squash. Thank you so much for the edits you made. There really are a >> lot of them. I'll take a look at them and keep them in mind for the future. >> > > Hi Alexandra, > > No worries! I squashed in the suggested minor changes and pushed the > patch to main. >
Hi Alexandra, It seems we missed a case. Since your fix got merged the following system test is failing: 22: multinode.at:4024 IPv6 NA received on non resident chassis https://github.com/ovn-org/ovn/actions/runs/19806853423/job/56742547105#step:19:4564 It was added by Xavier with https://github.com/ovn-org/ovn/commit/d76d6fe I wonder if we don't need to extend Xavier's fix now to ARP as well. Would you happen to have some time to look into this please? Thanks, Dumitru > Regards, > Dumitru > > > [...] > >>> The changes I'm suggesting are minor, here's the full diff (some more style >>> issues >>> are addressed in it): >>> >>> https://github.com/dceara/ovn/commit/a2307c9 >>> >>> If it looks good to you too I can squash it in and push the patch to main. >>> Please let me know what you think. >>> >>> Regards, >>> Dumitru >>> > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
