yahoo-eng-team team mailing list archive

[OpenStack Infra <2105383@xxxxxxxxxxxxxxxxxx>]

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

commit db030749278324b52b6d09f4d55a5d1e288d0abe
Author: Rodolfo Alonso Hernandez <ralonsoh@xxxxxxxxxx>
Date:   Fri Mar 28 03:33:07 2025 +0000

    Revert the network IDs filter in ``_gen_router_port_options``
    
    In [1], a change in the ``network_ids`` filter was introduced. This
    filter is used to retrieve the networks belonging to a router. However
    this is used in two places:
    * To set the gateway MTU. Here all the networks attached to the router
      must be present.
    * To calculate the "redirect-type" value in the Logical_Router_Port.
      Only the internal interfaces must be present; the gateway port should
      be discarded.
    
    NOTE: patch [1] is present since 2025.1 (Epoxy).
    
    [1]https://review.opendev.org/c/openstack/neutron/+/937026
    
    Closes-Bug: #2105383
    Related-Bug: #2032817
    Change-Id: I9cd6e77e47ce16dcf2fc46fc076f1d7ee7a4f9d3


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

Title:
  [OVN] The filter to add the MTU to a LRP changes the options
  dictionary population

Status in neutron:
  Fix Released

Bug description:
  This bug is related to [1][2].

  In [2], a change in the Logical_Router_Port options is introduced to
  always set the MTU. The MTU is calculated finding out the minimum MTU
  of all networks attached to the router, including the gateway port.

  However, in order to calculate the "redirect-type", it is only needed
  to retrieve the router ports excluding the gateway port.

  [1]https://bugs.launchpad.net/neutron/+bug/2032817
  [2]https://review.opendev.org/c/openstack/neutron/+/937026

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