mbox series

[ovs-dev,v3,ovn,0/2] Make ovn-northd recover from NB/SB inconsistencies.

Message ID 20200430183131.12099.85379.stgit@dceara.remote.csb
Headers show
Series Make ovn-northd recover from NB/SB inconsistencies. | expand

Message

Dumitru Ceara April 30, 2020, 6:31 p.m. UTC 
The patches in this series address the following issues:
- patch 1: ovn-northd generates transactions that trigger referential
  integrity violations due to stale Datapath_Binding records in
  the SB database.
- patch 2: ovn-northd generates transactions that trigger index uniqueness
  constraint violations due to reusing of stale tunnel_key values in
  Port_Binding table records.

Reported-by: Dan Williams <dcbw@redhat.com>
Reported-at: https://bugzilla.redhat.com/1828637
Signed-off-by: Dumitru Ceara <dceara@redhat.com>

Dumitru Ceara (2):
      ovn-northd: Clear SB records depending on stale datapaths.
      ovn-northd: Fix tunnel_key allocation for SB Port_Bindings.


 northd/ovn-northd.c |   51 ++++++++++++++++++++++-----------
 tests/ovn-northd.at |   80 +++++++++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 115 insertions(+), 16 deletions(-)


---
V3:
- reworked patch #2 to delete/add the stale PB.
- removed patch #3 as it's not needed anymore.
- merged HA_Chassis_Group unit test in patch #2.
V2:
- add patch #3 to handle stale HA_Chassis_Group references.
- add unit tests.
- call cleanup_mac_bindings() also when stale Port_Bindings
  are reused.
- rephrase commit messages.