yahoo-eng-team team mailing list archive

[OpenStack Infra <2006735@xxxxxxxxxxxxxxxxxx>]

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

commit d83e243c6d54f29ded7a0bea6e5538772fff3a8e
Author: Rodolfo Alonso Hernandez <ralonsoh@xxxxxxxxxx>
Date:   Thu Feb 9 12:57:25 2023 +0100

    [OVN] Add missing LSP device_owner info in trunk driver
    
    The ``OVNTrunkHandler`` class updates the port binding profile of the
    trunk subports. The methods ``_set_binding_profile`` and
    ``_unset_binding_profile`` update both the OVN LSP register and the
    Neutron DB port register. This patch adds the missing field
    "neutron:device_owner" from the LSP external_ids dictionary.
    
    This patch also updates ``OvsdbNbOvnIdl.set_lswitch_port`` API method.
    The method now accepts "external_ids_update" kwarg. This dictionary
    allows to update (or add) individually each LSP.external_ids
    dictionary key, instead of overwritting the whole variable.
    
    NOTE: ``set_lswitch_port`` is not used outside Neutron now so this is
    safe to change the API method signature.
    
    Closes-Bug: #2006735
    Change-Id: I985f3294b2ca7ab5989022ec1b904c8e29354e07


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

Title:
   [OVN] OVN trunk driver is not setting the OVN LSP
  "external_ids:neutron:device_owner" value

Status in neutron:
  Fix Released

Bug description:
  The methods ``OVNTrunkHandler._set_binding_profile`` and
  ``OVNTrunkHandler._unset_binding_profile`` are not updating the OVN
  LSP device owner value (stored in "external_ids") when setting or
  removing the binding profile.

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