yahoo-eng-team team mailing list archive

[OpenStack Infra <2073613@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://review.opendev.org/c/openstack/neutron/+/924515
Committed: https://opendev.org/openstack/neutron/commit/836fd550739a819e15b68c9bbb022bfb1f9f0221
Submitter: "Zuul (22348)"
Branch:    master

commit 836fd550739a819e15b68c9bbb022bfb1f9f0221
Author: Rodolfo Alonso Hernandez <ralonsoh@xxxxxxxxxx>
Date:   Fri Jul 19 14:56:34 2024 +0000

    [OVN] Remove maintenance method "remove_duplicated_chassis_registers"
    
    Since OVN 21.03.0 [1], ovn-controller no longer will try to reuse the
    stale Chassis/Chassis_Private records but will try to create new ones.
    If the stale records have no been manually deleted and ovn-controller
    tries to register a new Chassis with the same values, the transaction will
    fail.
    
    [1]https://github.com/ovn-org/ovn/commit/fc359bfe934290baeeeed1bc78a3f2a919421fa3
    
    Closes-Bug: #2073613
    Change-Id: I2f14558b0910fed03fe0243c4daa33947feeedcb


** Changed in: neutron
       Status: In Progress => Fix Released

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

Title:
  [OVN] Maintenance method "remove_duplicated_chassis_registers" is no
  longer needed

Status in neutron:
  Fix Released

Bug description:
  Since [1], present in OVN 21.03.0, the maintenance method "remove_duplicated_chassis_registers" is no longer needed. If ovn-controller is not stopped correctly, it will leave Chassis/Chassis_private in the SB database. However, ovn-controller no longer will try to reuse the stale records but will try to create new ones. If the stale records have no been manually deleted and ovn-controller tries to register a new Chassis with the same values, the DB will throw the following error:
  """
  ovsdb_idl|WARN|transaction error: {"details":"Transaction causes multiple rows in \"Encap\" table to have identical values (geneve and \"172.19.0.100\") for index on columns \"type\" and \"ip\".  First row, with UUID ed386b5a-8900-4815-9cc2-3cd7154a3799, was inserted by this transaction.  Second row, with UUID de9b4bc7-eedb-489f-849f-d3dd55e14350, existed in the database before this transaction and was not modified by the transaction.","error":"constraint violation"}
  """

  Because the minimum version tested and supported is 22.03, we can drop
  this method as it won't be possible anymore to have two
  Chassis/Chassis_Private registers pointing to the same host.

  [1]https://github.com/ovn-
  org/ovn/commit/fc359bfe934290baeeeed1bc78a3f2a919421fa3

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