yahoo-eng-team team mailing list archive

[Lucas Alvares Gomes <2040166@xxxxxxxxxxxxxxxxxx>]

Public bug reported:

The neutron_sync_mode configuration option is ignored when syncing the
OVN Southbound database, that class assumes that it's always in "repair"
mode.

neutron_sync_mode can have 3 values:

* off, which should just turn off sync
* log, which should just log the inconsistencies found between the Neutron and OVN databases
* repair, which should repair all inconsistencies found between Neutron and OVN databases

But, right now only the Northbound DB sync class checks for these
values:
https://github.com/openstack/neutron/blob/dad8c3fecc4e97764a92f558d9df510fa2516877/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_db_sync.py#L98-L100

The Southbound DB sync does not even receive the sync mode as a
parameter and assumes it's always in repair mode (even tho the default
value for that option is "log"):
https://github.com/openstack/neutron/blob/dad8c3fecc4e97764a92f558d9df510fa2516877/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_db_sync.py#L1330

** Affects: neutron
     Importance: High
     Assignee: Lucas Alvares Gomes (lucasagomes)
         Status: Confirmed

** Changed in: neutron
       Status: New => Confirmed

** Changed in: neutron
     Assignee: (unassigned) => Lucas Alvares Gomes (lucasagomes)

** Changed in: neutron
   Importance: Undecided => High

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to neutron.
https://bugs.launchpad.net/bugs/2040166

Title:
  [OVN] The neutron_sync_mode conf is ignored for SB DB sync

Status in neutron:
  Confirmed

Bug description:
  The neutron_sync_mode configuration option is ignored when syncing the
  OVN Southbound database, that class assumes that it's always in
  "repair" mode.

  neutron_sync_mode can have 3 values:

  * off, which should just turn off sync
  * log, which should just log the inconsistencies found between the Neutron and OVN databases
  * repair, which should repair all inconsistencies found between Neutron and OVN databases

  But, right now only the Northbound DB sync class checks for these
  values:
  https://github.com/openstack/neutron/blob/dad8c3fecc4e97764a92f558d9df510fa2516877/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_db_sync.py#L98-L100

  The Southbound DB sync does not even receive the sync mode as a
  parameter and assumes it's always in repair mode (even tho the default
  value for that option is "log"):
  https://github.com/openstack/neutron/blob/dad8c3fecc4e97764a92f558d9df510fa2516877/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_db_sync.py#L1330

To manage notifications about this bug go to:
https://bugs.launchpad.net/neutron/+bug/2040166/+subscriptions