On Wed, Aug 10, 2022 at 8:17 PM Ales Musil <[email protected]> wrote:
>
> On Wed, Aug 10, 2022 at 10:04 AM Dumitru Ceara <[email protected]> wrote:
>
> > It doesn't really need the load balancer applied on the logical router.
> > As a matter of fact, load balancers are not supported on distributed
> > routers without distributed gateway ports:
> >
> >
> > https://github.com/ovn-org/ovn/blob/4dc4bc7fdb848bcc626becbd2c80ffef8a39ff9a/ovn-nb.xml#L2223

Unrelated to this patch -
Looks like this section needs some fixes in documentation.  We now
support multiple distributed gateway ports.

> >
> > Signed-off-by: Dumitru Ceara <[email protected]>
> > ---
> >  tests/system-ovn.at | 1 -
> >  1 file changed, 1 deletion(-)
> >
> > diff --git a/tests/system-ovn.at b/tests/system-ovn.at
> > index 2ba71df0f..851117f55 100644
> > --- a/tests/system-ovn.at
> > +++ b/tests/system-ovn.at
> > @@ -5513,7 +5513,6 @@ ovn-nbctl lb-add lb0 "30.0.0.10:80" "20.0.0.4:80"
> >
> >  ovn-nbctl ls-lb-add sw1 lb0
> >  ovn-nbctl ls-lb-add sw2 lb0
> > -ovn-nbctl lr-lb-add lr0 lb0
> >
> >  ADD_NAMESPACES(sw0-p1-lbc)
> >  ADD_VETH(sw0-p1-lbc, sw0-p1-lbc, br-int, "10.0.0.3/24",
> > "10:54:00:00:00:03", \
> > --
> > 2.31.1
> >
> > _______________________________________________
> > dev mailing list
> > [email protected]
> > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
> >
> >
> Makes sense.
>
> Acked-by: Ales Musil <[email protected]>

Thanks.  Applied to main.

Numan

>
> Regards,
> Ales
>
> --
>
> Ales Musil
>
> Senior Software Engineer - OVN Core
>
> Red Hat EMEA <https://www.redhat.com>
>
> [email protected]    IM: amusil
> <https://red.ht/sig>
> _______________________________________________
> 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

Reply via email to