From 1d68253f263b5b3b27c6b786ba8116eeb33ac794 Mon Sep 17 00:00:00 2001 From: Will Szumski Date: Mon, 10 Jun 2024 13:44:14 +0100 Subject: [PATCH] Correct logic error when associating FIP with OVN LB Fixes a logic error which meant that we didn't iterate over all logical switches when associating a FIP to an OVN loadbalancer. The symptom was that the FIP would show in neutron, but would not exist in OVN. Closes-Bug: #2068644 Change-Id: I6d1979dfb4d6f455ca419e64248087047fbf73d7 Co-Authored-By: Brian Haley (cherry picked from commit d8a4ad9167afd824a3f823d86a8fd33fb67c4abd) --- .../plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_client.py | 2 +- ...fix-issue-with-ovn-loadbalancer-fip-4e4bda00cf019f71.yaml | 5 +++++ 2 files changed, 6 insertions(+), 1 deletion(-) create mode 100644 releasenotes/notes/fix-issue-with-ovn-loadbalancer-fip-4e4bda00cf019f71.yaml diff --git a/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_client.py b/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_client.py index acd4c41a0e8..96abc466e64 100644 --- a/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_client.py +++ b/neutron/plugins/ml2/drivers/ovn/mech_driver/ovsdb/ovn_client.py @@ -989,7 +989,7 @@ class OVNClient(object): if lb in item.load_balancer] if not ls_linked: - return + continue # Find out IP addresses and subnets of configured members. members_to_verify = [] diff --git a/releasenotes/notes/fix-issue-with-ovn-loadbalancer-fip-4e4bda00cf019f71.yaml b/releasenotes/notes/fix-issue-with-ovn-loadbalancer-fip-4e4bda00cf019f71.yaml new file mode 100644 index 00000000000..943aa596d1e --- /dev/null +++ b/releasenotes/notes/fix-issue-with-ovn-loadbalancer-fip-4e4bda00cf019f71.yaml @@ -0,0 +1,5 @@ +--- +fixes: + - | + Fixes an issue when associating floating IPs to OVN load balancers. See `LP#2068644 + `__ for more details.