yahoo-eng-team team mailing list archive

[OpenStack Infra <2002993@xxxxxxxxxxxxxxxxxx>]

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

commit 463c3df4cff0a5764beef3f83a7f3bf26d688470
Author: Frode Nordahl <frode.nordahl@xxxxxxxxxxxxx>
Date:   Mon Jan 16 17:20:13 2023 +0100

    [ovn] Do not create empty default route when empty gateway_ip
    
    Before this patch, if a OVN router was associated with an
    external network whose subnet had an empty `gateway_ip`, a
    default route with an empty dst-ip is created in the OVN
    database.
    
    As documented in the Neutron API [0], it is allowed to create
    a subnet without a gateway_ip.
    
    0: https://docs.openstack.org/api-ref/network/v2/index.html?expanded=create-subnet-detail#create-subnet
    
    Closes-Bug: #2002993
    Signed-off-by: Frode Nordahl <frode.nordahl@xxxxxxxxxxxxx>
    Change-Id: Ica76e0821d753af883444d2a449283e9e69ba03f


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

Title:
  [ovn] default gateway populated with empty dst-ip when gateway
  explicitly disabled on subnet

Status in neutron:
  Fix Released

Bug description:
  As documented in the Neutron API [0], it is allowed to create a subnet
  without a gateway_ip.

  This may be useful if you want to control the routes using the
  extraroutes API [1].

  At present the Neutron OVN driver will create a default route with an
  empty dst-ip in the OVN database in case of the gateway_ip not being
  set.

  I believe this is a bug and that the Neutron OVN driver should omit
  creating the route when no gateway_ip is set.

  0: https://docs.openstack.org/api-ref/network/v2/index.html?expanded=create-subnet-detail#create-subnet
  1: https://docs.openstack.org/api-ref/network/v2/index.html?expanded=create-subnet-detail,add-extra-routes-to-router-detail,remove-interface-from-router-detail#add-extra-routes-to-router

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