yahoo-eng-team team mailing list archive

[Doug Goldstein <2116960@xxxxxxxxxxxxxxxxxx>]

Public bug reported:

The code at
https://opendev.org/openstack/neutron/src/commit/a8b8a646482fa807776c4ab2b4ccb06067e384c1/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_db_sync.py#L1451
only syncs the segment host mappings for OVN_CONTROLLER_AGENT but this
means that OVN_CONTROLLER_GW_AGENT is skipped. It seems like both
OVN_CONTROLLER_GW_AGENT and OVN_CONTROLLER_AGENT should be included in
the sync here of segment host mappings to properly populate the DB.

The reason I say this is the code
https://opendev.org/openstack/neutron/src/commit/a8b8a646482fa807776c4ab2b4ccb06067e384c1/neutron/plugins/ml2/drivers/ovn/mech_driver/mech_driver.py#L1354-L1394
which is part of the OVN mechanism does not explicitly filter out the
OVN_CONTROLLER_GW_AGENT from updating the segment host mapping.

And in fact includes OVN_CONTROLLER_GW_AGENT in
https://opendev.org/openstack/neutron/src/commit/a8b8a646482fa807776c4ab2b4ccb06067e384c1/neutron/plugins/ml2/drivers/ovn/mech_driver/mech_driver.py#L1389
so it appears the DB sync operation is incorrect.

** Affects: neutron
     Importance: Undecided
         Status: New

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

Title:
  neutron ovn db sync skips chassis enabled as gateway

Status in neutron:
  New

Bug description:
  The code at
  https://opendev.org/openstack/neutron/src/commit/a8b8a646482fa807776c4ab2b4ccb06067e384c1/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_db_sync.py#L1451
  only syncs the segment host mappings for OVN_CONTROLLER_AGENT but this
  means that OVN_CONTROLLER_GW_AGENT is skipped. It seems like both
  OVN_CONTROLLER_GW_AGENT and OVN_CONTROLLER_AGENT should be included in
  the sync here of segment host mappings to properly populate the DB.

  The reason I say this is the code
  https://opendev.org/openstack/neutron/src/commit/a8b8a646482fa807776c4ab2b4ccb06067e384c1/neutron/plugins/ml2/drivers/ovn/mech_driver/mech_driver.py#L1354-L1394
  which is part of the OVN mechanism does not explicitly filter out the
  OVN_CONTROLLER_GW_AGENT from updating the segment host mapping.

  And in fact includes OVN_CONTROLLER_GW_AGENT in
  https://opendev.org/openstack/neutron/src/commit/a8b8a646482fa807776c4ab2b4ccb06067e384c1/neutron/plugins/ml2/drivers/ovn/mech_driver/mech_driver.py#L1389
  so it appears the DB sync operation is incorrect.

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