Re: [ovs-dev] ovn: Handling of arp/nd learning on logical switches

2023-03-03 Thread Olaf Seibert via dev
Tangentially related to this: I was trying to detect when the problem with the "too many resubmits" occurs, for alerting purposes. Some light examination of the source of ovs suggested that the coverage counter "drop_action_too_many_resubmit" corresponds to the error

[ovs-dev] [PATCH ovn] northd: Fix memory leak.

2022-08-17 Thread Olaf Seibert via dev
Commit 7b56f69580e1f390d9c6753a2cb8f0dbfbb4c467 cloned the external_ids to ids, then passed them to sbrec_port_binding_set_external_ids(). The intermediate step is unneeded (and caused a memory leak). Pass the external_ids directly. Signed-off-by: Olaf Seibert --- northd/northd.c | 4 +--- 1

[ovs-dev] [ovn] Memory leak in ovn-northd: which fix is preferred?

2022-08-16 Thread Olaf Seibert via dev
Hi! We discovered a memory leak in ovn-northd, which grows quickly, caused by commit 7b56f69580e1f390d9c6753a2cb8f0dbfbb4c467 (dated Wed Feb 23 09:30:00 2022 +). We also found a fix for it. I am however not familiar with this code, so when I looked a bit further, I wondered if a different

[ovs-dev] [ovn] Memory leak in ovn-northd: which fix is preferred?

2022-08-16 Thread Olaf Seibert via dev
Hi! We discovered a memory leak in ovn-northd, which grows quickly, caused by commit 7b56f69580e1f390d9c6753a2cb8f0dbfbb4c467 (dated Wed Feb 23 09:30:00 2022 +). We also found a fix for it. I am however not familiar with this code, so when I looked a bit further, I wondered if a different