openstack
/
openstack
Code Issues Proposed changes

Update git submodules 

* Update charm-neutron-dynamic-routing from branch 'master'
  to b97646f6652ada059fb6416d94b8db5da42d9772
  - Add OVN + neutron-dynamic-routing bundles
    
    * OVN + NDR testing is possible as of Neutron 22.0.0 (2023.1 or
      Antelope), see [1][2].
    * Vault is added to non-OVN bundles as the tests.yaml file is shared
      and uses a configure step for Vault
    
    Important note: at the time of writing the OVN integration code in
    Neutron does not have any use for `distributed` and `ha` router extra
    attributes and whether distributed or centralized FIPs are used in OVN
    is governed by the ovn/enable_distributed_floating_ip option which does
    not operate at the per-router granularity and is global.
    
    The docs have not been updated and still say [3] that OVN routers do not
    expose `ha` or `distributed` attributes which are now there as of [1].
    
    At the same time, the NDR implementation uses the `distributed`
    attribute to determine whether to use the gateway port IP and treat FIPs
    as centralized [4] and add a next-hop to the gateway port IP or not in
    the advertised /32 routes. For routers with `distributed == True`,
    NDR only attempts [5] to find ports with device owner
    `'floatingip_agent_gateway'`. Therefore, when distributed FIPs are
    enabled via ovn/enable_distributed_floating_ip, the `distributed` extra
    attribute on a router is set to `False` and /32 routes are advertised
    as centralized.
    
    The distributed FIP case is currently not exercised by the current
    bundles neither for ML2/ovs nor ML2/ovn. The current bundles only test
    the control plane and route advertisement, not packet forwarding via
    advertised next-hops.
    
    [1] https://review.opendev.org/c/openstack/neutron/+/864051
    [2] 4d1a7bd0bc
    [3] cd66232c2b/doc/source/ovn/faq/index.rst (L16-L29)
    [4] 513ea649be/neutron_dynamic_routing/db/bgp_db.py (L564)
    [5] 513ea649be/neutron_dynamic_routing/db/bgp_db.py (L567-L580)
    
    Change-Id: I21ba5d79ff100b839874a65e4ddd32f20dfb1311
This commit is contained in:
Dmitrii Shcherbakov 2023-05-31 08:50:37 +03:00 committed by Gerrit Code Review
parent 95bcefef6a
commit ad241538b1
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
charm-neutron-dynamic-routing

@ -1 +1 @@
Subproject commit 5963ea4fc32ef269d93f2fc8e05b363815397c43
Subproject commit b97646f6652ada059fb6416d94b8db5da42d9772