Hi Numan,

I tried to reproduce the issue with your way but didn't get the warning
again by restarting the controller of one chassis. Then I reviewed your
last response and seen the warning log.

The mac_binding entry that it tries to create is learned from external as
seen in the last warning log. I have then applied below patch of neutron.

https://opendev.org/openstack/neutron/commit/a278c5ba789c014ec777a75fc9538179d6707202

The patch is making always_learn_from_arp_request to false while creating
the router. This has reduced the SB DB size as well.



On Thu, Oct 21, 2021 at 1:29 AM Numan Siddique <num...@ovn.org> wrote:

> On Mon, Oct 18, 2021 at 7:55 AM Daniel Alvarez Sanchez
> <dalva...@redhat.com> wrote:
> >
> >
> >
> > On Mon, Oct 18, 2021 at 1:12 PM Ammad Syed <syedamma...@gmail.com>
> wrote:
> >>
> >> Hi Brendan,
> >>
> >> Not sure but this could be related to the patch below in neutron that
> was recently released.
> >>
> >>
> https://opendev.org/openstack/neutron/commit/f6c35527698119ee6f73a6a3613c9beebb563840
> >
> >
> > Not really, as this commit that you refer to is to reduce the memory
> footprint in the neutron-server process itself. Neutron never inserts
> anything into the MAC_Binding table.
> > However, we landed another patch [0] that will likely minimize this
> issue as it will reduce the MAC_Binding insertions, limiting them only to
> MAC addresses that need to be reached out by the overlay.
> >
> > Thanks!
> > daniel
> >
> > [0] https://review.opendev.org/c/openstack/neutron/+/813610
> >
> >
> >>
> >>
> >> Ammad
> >>
> >> On Mon, Oct 18, 2021 at 3:40 PM Brendan Doyle <brendan.do...@oracle.com>
> wrote:
> >>>
> >>>
> >>> I too am seeing many entries in the ovn-controller log like these:
> >>>
> >>>
> ovn/ovn-controller.log:2021-10-18T09:50:46.984Z|00164|ovsdb_idl|WARN|transaction
> error: {"details":"Transaction causes multiple rows in \"MAC_Binding\"
> table to have identical values (lr_vcn6727324-ls_vcn6727324_external_ugw
> and \"253.255.80.18\") for index on columns \"logical_port\" and \"ip\".
> First row, with UUID 614509d1-5dfc-4268-94b7-6190c1bd8c58, was inserted by
> this transaction.  Second row, with UUID
> 5b56044f-7e62-4e6b-b559-2bca31ad5ab0, existed in the database before this
> transaction and was not modified by the transaction.","error":"constraint
> violation"}
> >>>
> >>>
> >>> So with my limited understanding of the MAC_Binding table I'm
> wondering are these benign?
> >
> >
> > I think it's benign
> >>>
> >>>
> >>> In that is it that we for what ever reason have decided that we need
> to do an ARP to discover
> >>> a MAC, then ovn-controller tries to add the discovered MAC to the
> MAC_Binding only to find
> >>> that it is already there. In which case the question is why was an ARP
> request sent when
> >>> we already had a MAC binding.
>
> FYI,  ovn-controller also learns from the ARP requests received from
> external which can be
> turned off if desired with the option - always_learn_from_arp_request
> set in the logical router's options
> column.
>
> I don't think ovn-controller will generate an ARP request if its
> already learnt.  Looks to me
> multiple ovn-controller try to learn from an ARP request originated
> from external and ofcourse
> only one will be able to write to the mac_binding entry and others would
> fail.
>
> Thanks
> Numan
>
>
>
> >>>
> >>> For me these are only ever seen on Distributed router Port Gateways.
> >>>
> >>>
> >>> On 12/10/2021 13:06, Ammad Syed wrote:
> >>>
> >>> Hi,
> >>>
> >>> I am using openstack with ml2/ovn. I have two gateway chassis whenever
> I shutdown one chassis and bring it back online, I see below error in
> ovn-controller logs.
> >>>
> >>> Can you please advise the way to fix it ?
> >>>
> >>>
> 2021-10-12T11:49:22.124Z|00511|binding|INFO|cr-lrp-4c882760-15b1-4cf4-b680-53e88880e928:
> Claiming fa:16:3e:55:12:5a 101.53.244.97/24
> >>> 2021-10-12T11:49:22.126Z|00512|binding|INFO|Releasing lport
> cr-lrp-4c882760-15b1-4cf4-b680-53e88880e928 from this chassis.
> >>> 2021-10-12T11:50:32.032Z|00513|ovsdb_idl|WARN|transaction error:
> {"details":"Transaction causes multiple rows in \"MAC_Binding\" table to
> have identical values (lrp-4c882760-15b1-4cf4-b680-53e88880e928 and
> \"fe80::6a4f:64ff:fef7:8c0\") for index on columns \"logical_port\" and
> \"ip\".  First row, with UUID cbc847e6-75e4-4d73-9906-4fef221cad38, was
> inserted by this transaction.  Second row, with UUID
> bcac2e3e-5f32-411a-afcf-3249b85700f4, existed in the database before this
> transaction and was not modified by the transaction.","error":"constraint
> violation"}
> >>>
> >>> Ammad
> >>>
> >>> _______________________________________________
> >>> discuss mailing list
> >>> disc...@openvswitch.org
> >>>
> https://urldefense.com/v3/__https://mail.openvswitch.org/mailman/listinfo/ovs-discuss__;!!ACWV5N9M2RV99hQ!eo4L8XZbywCy0zb_p7-7LEVv9xl6g9V8JEf6oZTAO21scCtNAudRSGcK9mYnI7H-cyA$
> >>>
> >>>
> >>> _______________________________________________
> >>> discuss mailing list
> >>> disc...@openvswitch.org
> >>> https://mail.openvswitch.org/mailman/listinfo/ovs-discuss
> >>
> >>
> >>
> >> --
> >> Regards,
> >>
> >>
> >> Syed Ammad Ali
> >> _______________________________________________
> >> discuss mailing list
> >> disc...@openvswitch.org
> >> https://mail.openvswitch.org/mailman/listinfo/ovs-discuss
> >
> > _______________________________________________
> > discuss mailing list
> > disc...@openvswitch.org
> > https://mail.openvswitch.org/mailman/listinfo/ovs-discuss
>


-- 
Regards,


Syed Ammad Ali
_______________________________________________
discuss mailing list
disc...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-discuss

Reply via email to