yahoo-eng-team team mailing list archive

[OpenStack Infra <1991509@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://review.opendev.org/c/openstack/ovn-octavia-provider/+/858363
Committed: https://opendev.org/openstack/ovn-octavia-provider/commit/d889812f3c2deaf630892d65c90cdbafd4435d21
Submitter: "Zuul (22348)"
Branch:    master

commit d889812f3c2deaf630892d65c90cdbafd4435d21
Author: Luis Tomas Bolivar <ltomasbo@xxxxxxxxxx>
Date:   Fri Sep 30 20:13:49 2022 +0200

    Ensure lbs are properly configured for router gateway set/unset
    
    Before adding support for lbs with VIPs on the provider networks,
    there was no need to react to gateway chassis creation events, and
    nothing was done for its deletion. However, after adding the support
    for that, there is a need of properly handling the creation event
    for that type of port.
    
    For the loadbalancer VIPs on the provider network, processing the
    event and triggering the lb_creat_lrp_assoc_handler means that the
    information about the logical router will be added, i.e., the router
    is added to the loadbalancer external_ids as a lr_ref, while the
    loadbalancer is also added into the logical_router object
    (loadbalancer entry). In addition, the lb is also added to the logical
    switches connected to the router.
    
    For the loadbalancer VIPs on tenant networks (which should not be
    affected by the gateway port creation/deletion), this patch ensures
    the lb is not added to the logical switch representing the provider
    network that is connected to the router. Therefore differentiating
    between lrp ports which has gateway_chassis and the ones that don't,
    i.e., adding the lb to the switch when the lrp port is the one
    connecting a subnet with the router, and not doing so when it is the
    gateway port for the router to the provider network.
    
    Closes-Bug: #1991509
    
    Change-Id: Iddd96fd9015230b3dd75aa2182055cf43eb608c1


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

Title:
  [ovn-octavia-provider] router gateway unset + set breaks ovn lb
  connectivity

Status in neutron:
  Fix Released

Bug description:
  The LogicalRouterPortEvent for gateway_chassis port are skip [1],
  however if ovn lb VIPs are on provider network, the create event needs
  to be handle so that the loadbalancer gets properly configured and
  added to the router

  
  [1] https://opendev.org/openstack/ovn-octavia-provider/src/commit/acbf6e7f3e223c088582390475c84464bc27227d/ovn_octavia_provider/event.py#L39

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