← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1990229] Re: [OVN] Stop using external_ids from SB Chassis table

 

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

commit f8fa909444cf87d55819fa9dbc45ed77cff7b845
Author: Rodolfo Alonso Hernandez <ralonsoh@xxxxxxxxxx>
Date:   Fri Sep 23 09:11:25 2022 +0200

    Since OVN 20.06, config is stored in "Chassis.other_config"
    
    Since OVN 20.06 [1], the OVN configuration is stored in
    "Chassis.other_config".
    
    Since OVN 22.09, the "Chassis" configuration stored in
    "Chassis.other_config" will not be replicated to
    "Chassis.external_ids".
    
    This patch replaces any reference to "external_ids" when
    retrieving the Chassis configuration. That means Neutron
    no longer supports OVN versions below 20.06.
    
    [1]https://github.com/ovn-org/ovn/commit/74d90c2223d0a8c123823fb849b4c2de58c296e4
    [2]https://github.com/ovn-org/ovn/commit/51309429cc3032a0cb422603e7bbda4905ca01ae
    
    Closes-Bug: #1990229
    Change-Id: If379cefb66a262318858e1cea6503aa3a56ba956


** 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/1990229

Title:
  [OVN] Stop using external_ids from SB Chassis table

Status in neutron:
  Fix Released

Bug description:
  Since the release of ovn v22.09.0 the content of SB Chassis table
  column other_config is no longer copied to the external_ids.

  Neutron still relies on the external_ids e.g. in neutron-agent, mech-
  driver etc.

  All code pieces that use the chassis.external_ids column need to be
  updated.

  The duplication of the column was introduced in v20.06.0.

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



References