[ovs-dev] [PATCH ovn] ovn-northd: Warn for >1 IPv4 in build_lrouter_force_snat_flows_op().

Numan Siddique numans at ovn.org
Mon Mar 1 13:25:55 UTC 2021


On Thu, Feb 25, 2021 at 11:43 PM Ben Pfaff <blp at ovn.org> wrote:
>
> On Thu, Feb 25, 2021 at 11:21:26PM +0530, Numan Siddique wrote:
> > On Thu, Feb 25, 2021 at 4:57 AM Ben Pfaff <blp at ovn.org> wrote:
> > >
> > > The comments here point out that more than one IPv4 address is a
> > > problem (versus more than 2 IPv6 addresses), but the code doesn't match.
> > >
> >
> > Oops. My mistake. Thanks for catching this.
> >
> > Looks like you have addressed fixing the logic in the ddlog patch
> > itself and this patch is not
> > required :).
>
> Oh, I apologize.  That should not have been folded into my ddlog patch
> since it's a separate issue.  I made a mistake.

No worries.

Thanks
Numan

>
> Thanks for verifying that it's correct!
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>


More information about the dev mailing list